Revert gatherer changes

GoogleChrome · Jul 28, 2016 · b313795 · b313795
1 parent a8c5602
commit b313795
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 18 deletions.
diff --git a/lighthouse-core/driver/gatherers/screenshots.js b/lighthouse-core/driver/gatherers/screenshots.js
@@ -46,10 +46,9 @@ class ScreenshotFilmstrip extends Gather {
   }
 
   afterPass(options, tracingData) {
-    return this.getScreenshots(tracingData.traceContents)
-      .then(screenshots => {
-        this.artifact = screenshots;
-      });
+    return this.getScreenshots(tracingData.traceContents).then(screenshots => {
+      this.artifact = screenshots;
+    });
   }
 }
 

diff --git a/lighthouse-core/driver/gatherers/speedline.js b/lighthouse-core/driver/gatherers/speedline.js
@@ -20,15 +20,15 @@ const Gather = require('./gather');
 const speedline = require('speedline');
 
 class Speedline extends Gather {
+
   afterPass(options, tracingData) {
-    return speedline(tracingData.traceContents)
-      .then(results => {
-        this.artifact = results;
-      }).catch(err => {
-        this.artifact = {
-          debugString: err.message
-        };
-      });
+    return speedline(tracingData.traceContents).then(results => {
+      this.artifact = results;
+    }).catch(err => {
+      this.artifact = {
+        debugString: err.message
+      };
+    });
   }
 }
 

diff --git a/lighthouse-core/test/driver/gatherers/screenshots.js b/lighthouse-core/test/driver/gatherers/screenshots.js
@@ -27,9 +27,7 @@ describe('Screenshot gatherer', () => {
     // Currently this test must rely on knowing the phase hook for the gatherer.
     // A little unfortunate, but we need a "run scheduler with this gatherer, this mocked driver,
     // and this trace" test class to do that right
-    return screenshotsGather.afterPass(undefined, {
-      traceContents: traceData
-    }).then(_ => {
+    return screenshotsGather.afterPass(undefined, {traceContents: traceData}).then(_ => {
       assert.ok(Array.isArray(screenshotsGather.artifact));
       assert.equal(screenshotsGather.artifact.length, 7);
 

diff --git a/lighthouse-core/test/driver/gatherers/speedline.js b/lighthouse-core/test/driver/gatherers/speedline.js
@@ -24,9 +24,7 @@ describe('Speedline gatherer', () => {
   it('returns an error debugString on faulty trace data', done => {
     const speedlineGather = new SpeedlineGather();
 
-    speedlineGather.afterPass({}, {
-      traceContents: {boo: 'ya'}
-    }).then(_ => {
+    speedlineGather.afterPass({}, {traceContents: {boo: 'ya'}}).then(_ => {
       assert.ok(speedlineGather.artifact.debugString);
       assert.ok(speedlineGather.artifact.debugString.length);
       done();