🐛🏗 Indentation fixes based on the new (more accurate) eslint implem…

…entation of the `indent` rule (#16736)
ampproject · Jul 14, 2018 · 920e13f · 920e13f
1 parent ec8e691
commit 920e13f
Show file tree

Hide file tree

Showing 25 changed files with 195 additions and 195 deletions.
diff --git a/3p/iframe-transport-client.js b/3p/iframe-transport-client.js
@@ -93,7 +93,7 @@ export class IframeTransportClient {
     return this.creativeIdToContext_[creativeId] ||
         (this.creativeIdToContext_[creativeId] =
             new IframeTransportContext(this.win_, this.iframeMessagingClient_,
-              creativeId, this.vendor_));
+                creativeId, this.vendor_));
   }
 
   /**

diff --git a/ads/google/a4a/experiment-utils.js b/ads/google/a4a/experiment-utils.js
@@ -54,7 +54,7 @@ export class ExperimentUtils {
   maybeSelectExperiment(
     win, element, selectionBranches, experimentName) {
     const experimentInfoMap =
-        /** @type {!Object<string, !ExperimentInfo>} */ ({});
+    /** @type {!Object<string, !ExperimentInfo>} */ ({});
     experimentInfoMap[experimentName] = {
       isTrafficEligible: () => true,
       branches: selectionBranches,

diff --git a/ads/google/a4a/line-delimited-response-handler.js b/ads/google/a4a/line-delimited-response-handler.js
@@ -77,7 +77,7 @@ export function metaJsonCreativeGrouper(callback) {
   return function(line, done) {
     if (first) {
       const metadata =
-          /** @type {!Object<string, *>} */(tryParseJson(first) || {});
+      /** @type {!Object<string, *>} */(tryParseJson(first) || {});
       const lowerCasedMetadata =
           Object.keys(metadata).reduce((newObj, key) => {
             newObj[key.toLowerCase()] = metadata[key];

diff --git a/ads/google/a4a/url-builder.js b/ads/google/a4a/url-builder.js
@@ -55,7 +55,7 @@ export function buildUrl(
     if (fullLength > capacity) {
       const truncatedValue = encodedValue
           .substr(0, capacity - encodedNameAndSep.length - 1)
-        // Don't end with a partially truncated escape sequence
+      // Don't end with a partially truncated escape sequence
           .replace(/%\w?$/, '');
       if (truncatedValue) {
         encodedParams.push(encodedNameAndSep + truncatedValue);

diff --git a/ads/nativo.js b/ads/nativo.js
@@ -106,8 +106,8 @@ export function nativo(global, data) {
       setPercentageOfadViewed(
           (((coordinates.intersectionRect
               .height * 100) / coordinates
-                .boundingClientRect
-                .height) / 100));
+              .boundingClientRect
+              .height) / 100));
       global.PostRelease.checkIsAdVisible();
     }
     // Public

diff --git a/build-system/config.js b/build-system/config.js
@@ -108,7 +108,7 @@ module.exports = {
     '**/*.js',
     '!**/*.extern.js',
     '!{node_modules,build,dist,dist.3p,dist.tools,' +
-        'third_party}/**/*.*',
+        'third_party,test/coverage}/**/*.*',
     '!examples/**/*.*',
     '!{validator/dist,validator/webui/dist,' +
         'validator/node_modules,validator/nodejs/node_modules,' +

diff --git a/extensions/amp-a4a/0.1/template-validator.js b/extensions/amp-a4a/0.1/template-validator.js
@@ -73,7 +73,7 @@ export class TemplateValidator extends Validator {
     }
 
     const parsedResponseBody =
-        /** @type {!./amp-ad-type-defs.AmpTemplateCreativeDef} */ (
+    /** @type {!./amp-ad-type-defs.AmpTemplateCreativeDef} */ (
         tryParseJson(body) || {});
     return getAmpAdTemplateHelper(context.win)
         .fetch(parsedResponseBody.templateUrl)

diff --git a/extensions/amp-ad-exit/0.1/amp-ad-exit.js b/extensions/amp-ad-exit/0.1/amp-ad-exit.js
@@ -133,7 +133,7 @@ export class AmpAdExit extends AMP.BaseElement {
           continue;
         }
         const customVar =
-            /** @type {!./config.VariableDef} */ (target.vars[customVarName]);
+        /** @type {!./config.VariableDef} */ (target.vars[customVarName]);
         if (!customVar) {
           continue;
         }

diff --git a/extensions/amp-ad-network-adsense-impl/0.1/amp-ad-network-adsense-impl.js b/extensions/amp-ad-network-adsense-impl/0.1/amp-ad-network-adsense-impl.js
@@ -237,7 +237,7 @@ export class AmpAdNetworkAdsenseImpl extends AmpA4A {
    */
   divertExperiments() {
     const experimentInfoMap =
-        /** @type {!Object<string,
+    /** @type {!Object<string,
         !../../../src/experiments.ExperimentInfo>} */ ({
         [FORMAT_EXP]: {
           isTrafficEligible: () => !this.isResponsive_() &&

diff --git a/extensions/amp-ad-network-fake-impl/0.1/amp-ad-network-fake-impl.js b/extensions/amp-ad-network-fake-impl/0.1/amp-ad-network-fake-impl.js
@@ -61,7 +61,7 @@ export class AmpAdNetworkFakeImpl extends AmpA4A {
         return null;
       }
       const {status, headers} =
-          /** @type {{status: number, headers: !Headers}} */ (response);
+      /** @type {{status: number, headers: !Headers}} */ (response);
 
       // In the convert creative mode the content is the plain AMP HTML.
       // This mode is primarily used for A4A Envelop for testing.

diff --git a/extensions/amp-analytics/0.1/iframe-transport.js b/extensions/amp-analytics/0.1/iframe-transport.js
@@ -174,22 +174,22 @@ export class IframeTransport {
     // TODO(jonkeller): Consider merging with jank-meter.js
     IframeTransport.performanceObservers_[this.type_] =
         new this.ampWin_.PerformanceObserver(entryList => {
-        if (!entryList) {
-          return;
-        }
-        entryList.getEntries().forEach(entry => {
-          if (entry && entry['entryType'] == 'longtask' &&
+          if (!entryList) {
+            return;
+          }
+          entryList.getEntries().forEach(entry => {
+            if (entry && entry['entryType'] == 'longtask' &&
               (entry['name'] == 'cross-origin-descendant') &&
               entry.attribution) {
-            entry.attribution.forEach(attrib => {
-              if (this.frameUrl_ == attrib.containerSrc &&
+              entry.attribution.forEach(attrib => {
+                if (this.frameUrl_ == attrib.containerSrc &&
                     ++this.numLongTasks_ % LONG_TASK_REPORTING_THRESHOLD == 0) {
-                user().error(TAG_, `Long Task: Vendor: "${this.type_}"`);
-              }
-            });
-          }
+                  user().error(TAG_, `Long Task: Vendor: "${this.type_}"`);
+                }
+              });
+            }
+          });
         });
-      });
     IframeTransport.performanceObservers_[this.type_].observe({
       entryTypes: ['longtask'],
     });

diff --git a/extensions/amp-analytics/0.1/vendors.js b/extensions/amp-analytics/0.1/vendors.js
@@ -2204,7 +2204,7 @@ ANALYTICS_CONFIG['infonline']['triggers']['pageview']['iframe' +
 
 ANALYTICS_CONFIG['adobeanalytics_nativeConfig']
     ['triggers']['pageLoad']['iframe' +
-      /* TEMPORARY EXCEPTION */ 'Ping'] = true;
+    /* TEMPORARY EXCEPTION */ 'Ping'] = true;
 
 ANALYTICS_CONFIG['oewa']['triggers']['pageview']['iframe' +
 /* TEMPORARY EXCEPTION */ 'Ping'] = true;

diff --git a/extensions/amp-next-page/0.1/next-page-service.js b/extensions/amp-next-page/0.1/next-page-service.js
@@ -140,7 +140,7 @@ export class NextPageService {
     this.resources_ = Services.resourcesForDoc(ampDoc);
     this.multidocManager_ =
         new MultidocManager(win, Services.ampdocServiceFor(win),
-          Services.extensionsFor(win), Services.timerFor(win));
+            Services.extensionsFor(win), Services.timerFor(win));
 
     installPositionObserverServiceForDoc(ampDoc);
     this.positionObserver_ = getServiceForDoc(ampDoc, 'position-observer');