🏗 Update closure compiler to February's version (#32525)

extensions/amp-playbuzz/0.1/utils.js

@@ -54,7 +54,6 @@ export function debounce(func, wait, immediate) {
 /**
  *
  * Gets an element creator using a given document to create elements.
- * @export getElementCreator
  * @param {Document} document
  * @return {!Function}
  */

extensions/amp-user-notification/0.1/amp-user-notification.js

@@ -35,7 +35,6 @@ const TAG = 'amp-user-notification';
 const SERVICE_ID = 'userNotificationManager';
 
 /**
- * @export
  * @typedef {{
  *   showNotification: boolean
  * }}

package.json

@@ -37,7 +37,7 @@
   "devDependencies": {
     "0x": "4.10.2",
     "@ampproject/filesize": "4.2.0",
-    "@ampproject/google-closure-compiler": "20201207.0.0",
+    "@ampproject/google-closure-compiler": "20210202.0.0",
     "@ampproject/remapping": "1.0.1",
     "@babel/core": "7.12.13",
     "@babel/eslint-parser": "7.12.13",

src/impression.js

@@ -40,7 +40,7 @@ const DEFAULT_APPEND_URL_PARAM = ['gclid', 'gclsrc'];
  * (but somewhat more stringent) to the one described in the [3p/README.md](
  * https://github.com/ampproject/amphtml/blob/master/3p/README.md)
  *
- * @export {!Array<!RegExp>}
+ * @type {!Array<!RegExp>}
  */
 const TRUSTED_REFERRER_HOSTS = [
   /**

src/polyfills/fetch.js

@@ -304,8 +304,6 @@ export class Response extends FetchResponse {
 
 /**
  * Installs fetch and Response polyfill
- *
- * @export
  * @param {Window} win
  */
 export function install(win) {

src/polyfillstub/intersection-observer-stub.js

@@ -195,7 +195,6 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
    * @return {?Element}
    */
   get root() {
@@ -206,7 +205,6 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
    * @return {*}
    */
   get rootMargin() {
@@ -217,7 +215,6 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
    * @return {*}
    */
   get thresholds() {
@@ -228,7 +225,7 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
+   * @return {undefined}
    */
   disconnect() {
     if (this.inst_) {
@@ -239,7 +236,6 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
    * @return {!Array}
    */
   takeRecords() {
@@ -250,7 +246,6 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
    * @param {!Element} target
    */
   observe(target) {
@@ -264,7 +259,6 @@ export class IntersectionObserverStub {
   }
 
   /**
-   * @export
    * @param {!Element} target
    */
   unobserve(target) {

src/polyfillstub/resize-observer-stub.js

@@ -120,7 +120,7 @@ export class ResizeObserverStub {
   }
 
   /**
-   * @export
+   * @return {*}
    */
   disconnect() {
     if (this.inst_) {
@@ -131,7 +131,6 @@ export class ResizeObserverStub {
   }
 
   /**
-   * @export
    * @param {!Element} target
    */
   observe(target) {
@@ -145,7 +144,6 @@ export class ResizeObserverStub {
   }
 
   /**
-   * @export
    * @param {!Element} target
    */
   unobserve(target) {

src/runtime.js

@@ -171,7 +171,6 @@ function adoptShared(global, callback) {
    * @param {function(string,?string=,number=)} unusedFn
    * @param {function()=} opt_flush
    * @deprecated
-   * @export
    */
   global.AMP.setTickFunction = (unusedFn, opt_flush) => {};
 

src/service/resource.js

@@ -137,10 +137,10 @@ export class Resource {
     /** @private {number} */
     this.id_ = id;
 
-    /** @export @const {!AmpElement} */
+    /** @const {!AmpElement} */
     this.element = element;
 
-    /** @export @const {string} */
+    /** @const {string} */
     this.debugid = element.tagName.toLowerCase() + '#' + id;
 
     /** @const {!Window} */

src/service/resources-interface.js

@@ -48,7 +48,6 @@ export class ResourcesInterface {
   /**
    * Returns a list of resources.
    * @return {!Array<!./resource.Resource>}
-   * @export
    */
   get() {}
 

src/service/viewer-interface.js

@@ -36,7 +36,6 @@ export class ViewerInterface {
    * exported. Need to make sure it's not used externally.
    * @param {string} name
    * @return {?string}
-   * @export
    */
   getParam(name) {}
 
@@ -168,7 +167,6 @@ export class ViewerInterface {
    * @param {!JsonObject} data
    * @param {boolean} unusedAwaitResponse
    * @return {(!Promise<*>|undefined)}
-   * @export
    */
   receiveMessage(eventType, data, unusedAwaitResponse) {}
 
@@ -177,7 +175,6 @@ export class ViewerInterface {
    * messages to the viewer.
    * @param {function(string, (?JsonObject|string|undefined), boolean):(!Promise<*>|undefined)} deliverer
    * @param {string} origin
-   * @export
    */
   setMessageDeliverer(deliverer, origin) {}
 

src/style.js

@@ -25,7 +25,6 @@ let propertyNameCache;
 const vendorPrefixes = ['Webkit', 'webkit', 'Moz', 'moz', 'ms', 'O', 'o'];
 
 /**
- * @export
  * @param {string} camelCase camel cased string
  * @return {string} title cased string
  */
@@ -55,7 +54,6 @@ function getVendorJsPropertyName_(style, titleCase) {
  * Returns the possibly prefixed JavaScript property name of a style property
  * (ex. WebkitTransitionDuration) given a camelCase'd version of the property
  * (ex. transitionDuration).
- * @export
  * @param {!Object} style
  * @param {string} camelCase the camel cased version of a css property name
  * @param {boolean=} opt_bypassCache bypass the memoized cache of property

third_party/subscriptions-project/swg.js

@@ -2458,7 +2458,6 @@ const defaultStyles = {
 };
 
 /**
- * @export
  * @param {string} camelCase camel cased string
  * @return {string} title cased string
  */
@@ -2488,7 +2487,6 @@ function getVendorJsPropertyName_(style, titleCase) {
  * Returns the possibly prefixed JavaScript property name of a style property
  * (ex. WebkitTransitionDuration) given a camelCase'd version of the property
  * (ex. transitionDuration).
- * @export
  * @param {!Object} style
  * @param {string} camelCase the camel cased version of a css property name
  * @param {boolean=} bypassCache bypass the memoized cache of property
@@ -15090,7 +15088,7 @@ class PaymentsWebActivityDelegate {
    */
   isVerticalCenterExperimentEnabled_(paymentDataRequest) {
     return (
-      null  
+      null
     );
   }
 
@@ -15276,7 +15274,7 @@ class UpiHandler {
   loadPaymentData(paymentDataRequest, upiPaymentMethod, onResultCallback) {
     const parameters = upiPaymentMethod['parameters'];
     const transactionInfo = paymentDataRequest['transactionInfo'];
-    const supportedInstruments = 
+    const supportedInstruments =
         [{
           'supportedMethods': ['https://tez.google.com/pay'],
           'data': {
@@ -15720,7 +15718,6 @@ class PaymentsAsyncClient {
    * @param {!IsReadyToPayRequest} isReadyToPayRequest
    * @return {!Promise} The promise will contain the boolean result and error
    *     message when possible.
-   * @export
    */
   isReadyToPay(isReadyToPayRequest) {
     // Merge with paymentOptions, preferring values from isReadyToPayRequest
@@ -15888,7 +15885,6 @@ class PaymentsAsyncClient {
    *
    * @param {!PaymentDataRequest} paymentDataRequest Provides necessary
    *     information to support a payment.
-   * @export
    */
   prefetchPaymentData(paymentDataRequest) {
     /** @type {?string} */
@@ -15919,7 +15915,6 @@ class PaymentsAsyncClient {
    *
    * @param {!PaymentDataRequest} paymentDataRequest Provides necessary
    *     information to support a payment.
-   * @export
    */
   loadPaymentData(paymentDataRequest) {
     PayFrameHelper.postMessage({
@@ -15996,7 +15991,6 @@ class PaymentsAsyncClient {
    *
    * @param {!ButtonOptions=} options
    * @return {!Element}
-   * @export
    */
   createButton(options = {}) {
     const button = null;
@@ -17287,7 +17281,7 @@ const REGWALL_HTML = `
     font-size: 16px;
     margin: 0 0 8px;
   }
-  
+
   .gaa-metering-regwall--description {
     color: #646464;
     display: block;