ampproject · rsimha · Mar 30, 2020 · Mar 24, 2020 · Mar 30, 2020
diff --git a/3p/3d-gltf/index.js b/3p/3d-gltf/index.js
@@ -22,8 +22,8 @@ import {user} from '../../src/log';
 
 import GltfViewer from './viewer';
 
-const seq = (taskA, taskB) => cb => taskA(() => taskB(cb));
-const parallel = (taskA, taskB) => cb => {
+const seq = (taskA, taskB) => (cb) => taskA(() => taskB(cb));
+const parallel = (taskA, taskB) => (cb) => {
   let n = 0;
   const finish = () => {
     n++;
@@ -36,8 +36,8 @@ const parallel = (taskA, taskB) => cb => {
 };
 
 const loadThree = (global, cb) => {
-  const loadScriptCb = url => cb => loadScript(global, url, cb);
-  const loadThreeExample = examplePath =>
+  const loadScriptCb = (url) => (cb) => loadScript(global, url, cb);
+  const loadThreeExample = (examplePath) =>
     loadScriptCb(
       'https://cdn.jsdelivr.net/npm/three@0.91/examples/js/' + examplePath
     );
@@ -62,7 +62,7 @@ export function gltfViewer(global) {
       onload: () => {
         nonSensitiveDataPostMessage('loaded');
       },
-      onprogress: e => {
+      onprogress: (e) => {
         if (!e.lengthComputable) {
           return;
         }
@@ -74,7 +74,7 @@ export function gltfViewer(global) {
           })
         );
       },
-      onerror: err => {
+      onerror: (err) => {
         user().error('3DGLTF', err);
         nonSensitiveDataPostMessage(
           'error',
@@ -84,7 +84,7 @@ export function gltfViewer(global) {
         );
       },
     });
-    listenParent(global, 'action', msg => {
+    listenParent(global, 'action', (msg) => {
       viewer.actions[msg['action']](msg['args']);
     });
     nonSensitiveDataPostMessage('ready');

diff --git a/3p/3d-gltf/viewer.js b/3p/3d-gltf/viewer.js
@@ -128,7 +128,7 @@ export default class GltfViewer {
     let oldW = null;
     let oldH = null;
     /** @param {JsonObject} box */
-    const setSize = box => {
+    const setSize = (box) => {
       const w = box['width'];
       const h = box['height'];
       if (oldW === w && oldH === h) {
@@ -250,9 +250,9 @@ export default class GltfViewer {
     loader.load(
       this.options_['src'],
       /** @param {{scene: !THREE.Scene}} gltfData */
-      gltfData => {
+      (gltfData) => {
         this.setupCameraForObject_(gltfData.scene);
-        gltfData.scene.children.slice().forEach(child => {
+        gltfData.scene.children.slice().forEach((child) => {
           this.model_.add(child);
         });
 

diff --git a/3p/3p.js b/3p/3p.js
@@ -197,7 +197,7 @@ export function computeInMasterFrame(global, taskId, work, cb) {
   if (!global.context.isMaster) {
     return; // Only do work in master.
   }
-  work(result => {
+  work((result) => {
     for (let i = 0; i < cbs.length; i++) {
       cbs[i].call(null, result);
     }
@@ -254,7 +254,7 @@ export function validateData(data, mandatoryFields, opt_optionalFields) {
  */
 function validateExactlyOne(data, alternativeFields) {
   userAssert(
-    alternativeFields.filter(field => data[field]).length === 1,
+    alternativeFields.filter((field) => data[field]).length === 1,
     '%s must contain exactly one of attributes: %s.',
     data.type,
     alternativeFields.join(', ')

diff --git a/3p/ampcontext.js b/3p/ampcontext.js
@@ -129,7 +129,7 @@ export class AbstractAmpContext {
     this.client_.makeRequest(
       MessageType.SEND_EMBED_STATE,
       MessageType.EMBED_STATE,
-      data => {
+      (data) => {
         this.hidden = data['pageHidden'];
         this.dispatchVisibilityChangeEvent_();
       }
@@ -155,7 +155,7 @@ export class AbstractAmpContext {
    *    every time we receive a page visibility message.
    */
   onPageVisibilityChange(callback) {
-    return this.client_.registerCallback(MessageType.EMBED_STATE, data => {
+    return this.client_.registerCallback(MessageType.EMBED_STATE, (data) => {
       callback({hidden: data['pageHidden']});
     });
   }
@@ -171,7 +171,7 @@ export class AbstractAmpContext {
     return this.client_.makeRequest(
       MessageType.SEND_INTERSECTIONS,
       MessageType.INTERSECTION,
-      intersection => {
+      (intersection) => {
         callback(intersection['changes']);
       }
     );
@@ -230,7 +230,7 @@ export class AbstractAmpContext {
    *    request succeeds.
    */
   onResizeSuccess(callback) {
-    this.client_.registerCallback(MessageType.EMBED_SIZE_CHANGED, obj => {
+    this.client_.registerCallback(MessageType.EMBED_SIZE_CHANGED, (obj) => {
       callback(obj['requestedHeight'], obj['requestedWidth']);
     });
   }
@@ -243,7 +243,7 @@ export class AbstractAmpContext {
    *    request is denied.
    */
   onResizeDenied(callback) {
-    this.client_.registerCallback(MessageType.EMBED_SIZE_DENIED, obj => {
+    this.client_.registerCallback(MessageType.EMBED_SIZE_DENIED, (obj) => {
       callback(obj['requestedHeight'], obj['requestedWidth']);
     });
   }

diff --git a/3p/beopinion.js b/3p/beopinion.js
@@ -25,7 +25,7 @@ import {setStyles} from '../src/style';
  * @param {!Window} global
  */
 function getBeOpinion(global) {
-  loadScript(global, 'https://widget.beopinion.com/sdk.js', function() {});
+  loadScript(global, 'https://widget.beopinion.com/sdk.js', function () {});
 }
 
 /**
@@ -87,20 +87,20 @@ function createContainer(global, data) {
  */
 function getBeOpinionAsyncInit(global, accountId) {
   const {context} = global;
-  return function() {
-    context.onResizeDenied(function(requestedHeight, requestedWidth) {
+  return function () {
+    context.onResizeDenied(function (requestedHeight, requestedWidth) {
       context.requestResize(requestedWidth, requestedHeight);
     });
     global.BeOpinionSDK.init({
       account: accountId,
-      onContentReceive: function(hasContent) {
+      onContentReceive: function (hasContent) {
         if (hasContent) {
           context.renderStart();
         } else {
           context.noContentAvailable();
         }
       },
-      onHeightChange: function(newHeight) {
+      onHeightChange: function (newHeight) {
         const c = global.document.getElementById('c');
         const boundingClientRect = c./*REVIEW*/ getBoundingClientRect();
         context.requestResize(boundingClientRect.width, newHeight);

diff --git a/3p/bodymovinanimation.js b/3p/bodymovinanimation.js
@@ -38,7 +38,7 @@ function getBodymovinAnimationSdk(global, renderer, cb) {
     renderer === 'svg'
       ? 'https://cdnjs.cloudflare.com/ajax/libs/bodymovin/4.13.0/bodymovin_light.min.js'
       : 'https://cdnjs.cloudflare.com/ajax/libs/bodymovin/4.13.0/bodymovin.min.js';
-  loadScript(global, scriptToLoad, function() {
+  loadScript(global, scriptToLoad, function () {
     cb(global.bodymovin);
   });
 }
@@ -83,7 +83,7 @@ export function bodymovinanimation(global) {
   const shouldLoop = dataLoop != 'false';
   const loop = !isNaN(dataLoop) ? dataLoop : shouldLoop;
   const renderer = dataReceived['renderer'];
-  getBodymovinAnimationSdk(global, renderer, function(bodymovin) {
+  getBodymovinAnimationSdk(global, renderer, function (bodymovin) {
     animationHandler = bodymovin.loadAnimation({
       container: animatingContainer,
       renderer,

diff --git a/3p/embedly.js b/3p/embedly.js
@@ -64,7 +64,7 @@ export const CardOptions = {
  * @visibleForTesting
  */
 function getEmbedly(global, callback) {
-  loadScript(global, EMBEDLY_SDK_URL, function() {
+  loadScript(global, EMBEDLY_SDK_URL, function () {
     callback();
   });
 }
@@ -98,15 +98,15 @@ export function embedly(global, data) {
 
   container.appendChild(card);
 
-  getEmbedly(global, function() {
+  getEmbedly(global, function () {
     // Given by the parent frame.
     delete data.width;
     delete data.height;
 
     global.window['embedly']('card', card);
 
     // Use embedly SDK to listen to resize event from loaded card
-    global.window['embedly']('on', RESIZE_EVENT_NAME, function(iframe) {
+    global.window['embedly']('on', RESIZE_EVENT_NAME, function (iframe) {
       context.requestResize(
         iframe./*OK*/ width,
         parseInt(iframe./*OK*/ height, 10) + /* margin */ 5

diff --git a/3p/environment.js b/3p/environment.js
@@ -73,8 +73,8 @@ function manageWin_(win) {
 function instrumentDocWrite(parent, win) {
   const doc = win.document;
   const {close} = doc;
-  doc.close = function() {
-    parent.ampManageWin = function(win) {
+  doc.close = function () {
+    parent.ampManageWin = function (win) {
       manageWin(win);
     };
     if (!parent.ampSeen) {
@@ -93,7 +93,7 @@ function instrumentDocWrite(parent, win) {
  */
 function instrumentSrcdoc(parent, iframe) {
   let srcdoc = iframe.getAttribute('srcdoc');
-  parent.ampManageWin = function(win) {
+  parent.ampManageWin = function (win) {
     manageWin(win);
   };
   srcdoc += '<script>window.parent.ampManageWin(window)</script>';
@@ -159,7 +159,7 @@ function installObserver(win) {
   if (!window.MutationObserver) {
     return;
   }
-  const observer = new MutationObserver(function(mutations) {
+  const observer = new MutationObserver(function (mutations) {
     for (let i = 0; i < mutations.length; i++) {
       maybeInstrumentsNodes(win, mutations[i].addedNodes);
     }
@@ -177,14 +177,14 @@ function installObserver(win) {
 function instrumentEntryPoints(win) {
   // Change setTimeout to respect a minimum timeout.
   const {setTimeout} = win;
-  win.setTimeout = function(fn, time) {
+  win.setTimeout = function (fn, time) {
     time = minTime(time);
     arguments[1] = time;
     return setTimeout.apply(this, arguments);
   };
   // Implement setInterval in terms of setTimeout to make
   // it respect the same rules
-  win.setInterval = function(fn) {
+  win.setInterval = function (fn) {
     const id = intervalId++;
     const args = Array.prototype.slice.call(arguments);
     /**
@@ -195,7 +195,8 @@ function instrumentEntryPoints(win) {
       next();
       if (typeof fn == 'string') {
         // Handle rare and dangerous string arg case.
-        return (0, win.eval /*NOT OK but whatcha gonna do.*/).call(win, fn); // lgtm [js/useless-expression]
+        return (0, win.eval) /*NOT OK but whatcha gonna do.*/
+          .call(win, fn); // lgtm [js/useless-expression]
       } else {
         return fn.apply(this, arguments);
       }
@@ -211,7 +212,7 @@ function instrumentEntryPoints(win) {
     return id;
   };
   const {clearInterval} = win;
-  win.clearInterval = function(id) {
+  win.clearInterval = function (id) {
     clearInterval(id);
     win.clearTimeout(intervals[id]);
     delete intervals[id];
@@ -235,11 +236,11 @@ function blockSyncPopups(win) {
   }
   try {
     win.alert = maybeThrow;
-    win.prompt = function() {
+    win.prompt = function () {
       maybeThrow();
       return '';
     };
-    win.confirm = function() {
+    win.confirm = function () {
       maybeThrow();
       return false;
     };
@@ -269,7 +270,7 @@ function minTime(time) {
  * Installs embed state listener.
  */
 export function installEmbedStateListener() {
-  listenParent(window, 'embed-state', function(data) {
+  listenParent(window, 'embed-state', function (data) {
     inViewport = data['inViewport'];
   });
 }
diff --git a/3p/facebook.js b/3p/facebook.js
@@ -228,12 +228,12 @@ export function facebook(global, data) {
 
   getFacebookSdk(
     global,
-    FB => {
+    (FB) => {
       // Dimensions are given by the parent frame.
       delete data.width;
       delete data.height;
 
-      FB.Event.subscribe('xfbml.resize', event => {
+      FB.Event.subscribe('xfbml.resize', (event) => {
         context.updateDimensions(
           parseInt(event.width, 10),
           parseInt(event.height, 10) + /* margins */ 20

diff --git a/3p/github.js b/3p/github.js
@@ -29,7 +29,7 @@ import {writeScript} from './3p';
  * @param {function()} cb
  */
 function getGistJs(global, scriptSource, cb) {
-  writeScript(global, scriptSource, function() {
+  writeScript(global, scriptSource, function () {
     cb();
   });
 }
@@ -52,7 +52,7 @@ export function github(global, data) {
     gistUrl += '?file=' + encodeURIComponent(data.file);
   }
 
-  getGistJs(global, gistUrl, function() {
+  getGistJs(global, gistUrl, function () {
     // Dimensions are given by the parent frame.
     delete data.width;
     delete data.height;

diff --git a/3p/iframe-messaging-client.js b/3p/iframe-messaging-client.js
@@ -62,7 +62,7 @@ export class IframeMessagingClient {
   getData(requestType, payload, callback) {
     const responseType = requestType + CONSTANTS.responseTypeSuffix;
     const messageId = this.nextMessageId_++;
-    const unlisten = this.registerCallback(responseType, result => {
+    const unlisten = this.registerCallback(responseType, (result) => {
       if (result[CONSTANTS.messageIdFieldName] === messageId) {
         unlisten();
         callback(result[CONSTANTS.contentFieldName]);
@@ -100,7 +100,7 @@ export class IframeMessagingClient {
    * @return {*} TODO(#23582): Specify return type
    */
   requestOnce(requestType, responseType, callback) {
-    const unlisten = this.registerCallback(responseType, event => {
+    const unlisten = this.registerCallback(responseType, (event) => {
       unlisten();
       callback(event);
     });
@@ -191,7 +191,7 @@ export class IframeMessagingClient {
    * @private
    */
   setupEventListener_() {
-    listen(this.win_, 'message', event => {
+    listen(this.win_, 'message', (event) => {
       // If we have set a host window, strictly check that it's from it.
       if (this.hostWindow_ && event.source != this.hostWindow_) {
         return;

diff --git a/3p/iframe-transport-client.js b/3p/iframe-transport-client.js
@@ -62,7 +62,7 @@ export class IframeTransportClient {
     this.iframeMessagingClient_.makeRequest(
       MessageType.SEND_IFRAME_TRANSPORT_EVENTS,
       MessageType.IFRAME_TRANSPORT_EVENTS,
-      eventData => {
+      (eventData) => {
         const events =
           /**
            * @type
@@ -77,7 +77,7 @@ export class IframeTransportClient {
           events.length,
           'Received empty events list in ' + this.win_.location.href
         );
-        events.forEach(event => {
+        events.forEach((event) => {
           try {
             devAssert(
               event.creativeId,

diff --git a/3p/integration.js b/3p/integration.js
@@ -638,7 +638,7 @@ export function draw3p(win, data, configCallback) {
     win.context.tagName
   );
   if (configCallback) {
-    configCallback(data, data => {
+    configCallback(data, (data) => {
       userAssert(data, 'Expected configuration to be passed as first argument');
       run(type, win, data);
     });
@@ -659,7 +659,7 @@ export function draw3p(win, data, configCallback) {
  * @param {!Array<string>=} opt_allowedEmbeddingOrigins List of domain suffixes
  *     that are allowed to embed this frame.
  */
-window.draw3p = function(
+window.draw3p = function (
   opt_configCallback,
   opt_allowed3pTypes,
   opt_allowedEmbeddingOrigins