Fix pkg.browser mappings issue by specifying a value of false

src/index.js

@@ -39,6 +39,8 @@ function cachedIsFile (file, cb) {
 
 const resolveIdAsync = (file, opts) => new Promise((fulfil, reject) => resolveId(file, opts, (err, contents) => err ? reject(err) : fulfil(contents)));
 
+const FALSE = {};
+
 export default function nodeResolve ( options = {} ) {
 	const useModule = options.module !== false;
 	const useMain = options.main !== false;
@@ -82,6 +84,7 @@ export default function nodeResolve ( options = {} ) {
 
 			const basedir = importer ? dirname( importer ) : process.cwd();
 
+			// https://github.com/defunctzombie/package-browser-field-spec
 			if (options.browser && browserMapCache[importer]) {
 				const resolvedImportee = resolve( basedir, importee );
 				const browser = browserMapCache[importer];
@@ -157,16 +160,28 @@ export default function nodeResolve ( options = {} ) {
 				importee,
 				Object.assign( resolveOptions, customResolveOptions )
 			)
-				.catch(() => false)
+				.catch(() => FALSE)
 				.then(resolved => {
-					if (options.browser && packageBrowserField) {
-						if (packageBrowserField[ resolved ]) {
-							resolved = packageBrowserField[ resolved ];
+					if (resolved !== FALSE && options.browser && packageBrowserField) {
+						let l;
+						const hasLink = packageBrowserField.hasOwnProperty(resolved);
+						if (hasLink) {
+							l = packageBrowserField[ resolved ];
+						}
+						browserMapCache[l || resolved] = packageBrowserField;
+						if (hasLink) {
+							resolved = l;
+						}
+						if (resolved === false) {
+							return ES6_BROWSER_EMPTY;
 						}
-						browserMapCache[resolved] = packageBrowserField;
 					}
 
-					if ( !disregardResult && resolved !== false ) {
+					if (resolved === FALSE) {
+						return null;
+					}
+
+					if ( !disregardResult ) {
 						if ( !preserveSymlinks && resolved && fs.existsSync( resolved ) ) {
 							resolved = fs.realpathSync( resolved );
 						}
@@ -190,7 +205,7 @@ export default function nodeResolve ( options = {} ) {
 					if ( resolved && options.modulesOnly ) {
 						return readFileAsync( resolved, 'utf-8').then(code => isModule( code ) ? resolved : null);
 					} else {
-						return resolved === false ? null : resolved;
+						return resolved;
 					}
 				});
 		}