Viewer: dont check file type, try to parse json file (#1234)

GoogleChrome · Dec 22, 2016 · c5402c9 · c5402c9
1 parent a00240b
commit c5402c9
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lighthouse-viewer/app/src/lighthouse-report-viewer.js b/lighthouse-viewer/app/src/lighthouse-report-viewer.js
@@ -189,12 +189,16 @@ class LighthouseViewerReport {
    */
   onFileUpload(file) {
     return FileUploader.readFile(file).then(str => {
-      if (!file.type.match('json')) {
-        throw new Error('Unsupported report format. Expected JSON.');
+      let json;
+      try {
+        json = JSON.parse(str);
+      } catch(e) {
+        throw new Error('Could not parse JSON file.');
       }
+
       this._isNewReport = true;
 
-      this.replaceReportHTML(JSON.parse(str));
+      this.replaceReportHTML(json);
     }).catch(err => logger.error(err.message));
   }