Remove isJSON() method from Module

Reviewed By: cpojer

Differential Revision: D6408739

fbshipit-source-id: 3f1b76ca54937fbc433d4e7a804a9703a3a2ccb7

packages/metro-bundler/src/DeltaBundler/__tests__/DeltaCalculator-test.js

@@ -65,9 +65,6 @@ describe('DeltaCalculator', () => {
       isAsset() {
         return !!isAsset;
       },
-      isJSON() {
-        return !!isAsset;
-      },
     };
   }
 

packages/metro-bundler/src/DeltaBundler/__tests__/__snapshots__/traverseDependencies-integration-test.js.snap

@@ -78,7 +78,6 @@ Array [
     ],
     "id": "index",
     "isAsset": false,
-    "isJSON": false,
     "isPolyfill": false,
     "path": "/root/index.js",
     "resolution": undefined,
@@ -87,7 +86,6 @@ Array [
     "dependencies": Array [],
     "id": "a",
     "isAsset": false,
-    "isJSON": false,
     "isPolyfill": false,
     "path": "/root/a.js",
     "resolution": undefined,
@@ -96,7 +94,6 @@ Array [
     "dependencies": Array [],
     "id": "b",
     "isAsset": false,
-    "isJSON": false,
     "isPolyfill": false,
     "path": "/root/b.js",
     "resolution": undefined,

packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-test.js

@@ -33,7 +33,7 @@ function deferred(value) {
   return {promise, resolve: () => resolve(value)};
 }
 
-function createModule({path, name, isAsset, isJSON}) {
+function createModule({path, name, isAsset}) {
   return {
     path,
     name,
@@ -43,9 +43,6 @@ function createModule({path, name, isAsset, isJSON}) {
     isAsset() {
       return !!isAsset;
     },
-    isJSON() {
-      return !!isAsset;
-    },
   };
 }
 

packages/metro-bundler/src/node-haste/AssetModule.js

@@ -61,10 +61,6 @@ class AssetModule extends Module {
     return `AssetModule : ${this.path}`;
   }
 
-  isJSON() {
-    return false;
-  }
-
   isAsset() {
     return true;
   }

packages/metro-bundler/src/node-haste/Module.js

@@ -407,10 +407,6 @@ class Module {
     return `Module : ${this.path}`;
   }
 
-  isJSON() {
-    return extname(this.path) === '.json';
-  }
-
   isAsset() {
     return false;
   }

packages/metro-bundler/src/node-haste/Polyfill.js

@@ -47,10 +47,6 @@ class Polyfill extends Module {
     return this._dependencies;
   }
 
-  isJSON() {
-    return false;
-  }
-
   isPolyfill() {
     return true;
   }