Use dev().info for prod logging (#16731)

* Use dev().info for prod logging.

* Add comment explaining AMP.setLogLevel().

extensions/amp-bind/0.1/bind-impl.js

@@ -231,7 +231,7 @@ export class Bind {
       user().error(TAG, 'Failed to merge result from AMP.setState().', e);
     }
 
-    dev().fine(TAG, 'state:', this.state_);
+    dev().info(TAG, 'state:', this.state_);
 
     if (opt_skipEval) {
       return Promise.resolve();
@@ -306,7 +306,7 @@ export class Bind {
    * @return {!Promise}
    */
   setStateWithExpression(expression, scope) {
-    dev().fine(TAG, 'setState:', `"${expression}"`);
+    dev().info(TAG, 'setState:', `"${expression}"`);
     this.setStatePromise_ = this.evaluateExpression_(expression, scope)
         .then(result => this.setState(result))
         .then(() => {
@@ -329,7 +329,7 @@ export class Bind {
    * @return {!Promise}
    */
   pushStateWithExpression(expression, scope) {
-    dev().fine(TAG, 'pushState:', expression);
+    dev().info(TAG, 'pushState:', expression);
     return this.evaluateExpression_(expression, scope).then(result => {
       // Store the current values of each referenced variable in `expression`
       // so that we can restore them on history-pop.
@@ -368,7 +368,7 @@ export class Bind {
    * @return {!Promise}
    */
   scanAndApply(added, removed, timeout = 2000) {
-    dev().fine(TAG, 'rescan:', added, removed);
+    dev().info(TAG, 'rescan:', added, removed);
     const promise = this.removeThenAdd_(added, removed)
         .then(bindingsAdded => {
           // Don't reevaluate/apply if there are no bindings.
@@ -403,12 +403,12 @@ export class Bind {
    * @private
    */
   initialize_(root) {
-    dev().fine(TAG, 'init');
+    dev().info(TAG, 'init');
     let promise = Promise.all([
       this.addMacros_(),
       this.addBindingsForNodes_([root])]
     ).then(results => {
-      dev().fine(TAG, '⤷', 'Δ:', results);
+      dev().info(TAG, '⤷', 'Δ:', results);
       // Listen for DOM updates (e.g. template render) to rescan for bindings.
       root.addEventListener(AmpEvents.DOM_UPDATE, e => this.onDomUpdate_(e));
     });
@@ -797,7 +797,7 @@ export class Bind {
         // Throw to reject promise.
         throw this.reportWorkerError_(error, `${TAG}: Expression eval failed.`);
       } else {
-        dev().fine(TAG, '⤷', result);
+        dev().info(TAG, '⤷', result);
         return result;
       }
     });
@@ -824,7 +824,7 @@ export class Bind {
           reportError(userError, elements[0]);
         }
       });
-      dev().fine(TAG, 'bindings:', results);
+      dev().info(TAG, 'bindings:', results);
       return results;
     });
   }
@@ -942,7 +942,7 @@ export class Bind {
       return this.applyBoundElement_(results, boundElement);
     });
     return Promise.all(promises).then(() => {
-      dev().fine(TAG, 'updated:', promises.length, 'elements');
+      dev().info(TAG, 'updated:', promises.length, 'elements');
     });
   }
 
@@ -962,7 +962,7 @@ export class Bind {
       });
     });
     return Promise.all(promises).then(() => {
-      dev().fine(TAG, 'updated:', promises.length, 'elements');
+      dev().info(TAG, 'updated:', promises.length, 'elements');
     });
   }
 
@@ -1211,7 +1211,7 @@ export class Bind {
     if (parent && parent.tagName == 'AMP-LIST') {
       return;
     }
-    dev().fine(TAG, 'dom_update:', target);
+    dev().info(TAG, 'dom_update:', target);
     this.removeThenAdd_([target], [target]).then(() => {
       this.dispatchEventForTesting_(BindEvents.RESCAN_TEMPLATE);
     });
@@ -1234,7 +1234,7 @@ export class Bind {
         })
         .then(added => {
           sum += added;
-          dev().fine(TAG, '⤷', 'Δ:', sum, ', ∑:', this.numberOfBindings());
+          dev().info(TAG, '⤷', 'Δ:', sum, ', ∑:', this.numberOfBindings());
         });
   }
 

extensions/amp-list/0.1/amp-list.js

@@ -130,7 +130,7 @@ export class AmpList extends AMP.BaseElement {
 
   /** @override */
   mutatedAttributesCallback(mutations) {
-    dev().fine(TAG, 'mutate:', mutations);
+    dev().info(TAG, 'mutate:', mutations);
     const src = mutations['src'];
     const state = mutations['state'];
     if (src !== undefined) {
@@ -250,7 +250,7 @@ export class AmpList extends AMP.BaseElement {
    * @private
    */
   scheduleRender_(items) {
-    dev().fine(TAG, 'schedule:', items);
+    dev().info(TAG, 'schedule:', items);
     const deferred = new Deferred();
     const {promise, resolve: resolver, reject: rejecter} = deferred;
     // If there's nothing currently being rendered, schedule a render pass.
@@ -269,7 +269,7 @@ export class AmpList extends AMP.BaseElement {
   doRenderPass_() {
     const current = this.renderItems_;
     dev().assert(current, 'Nothing to render.');
-    dev().fine(TAG, 'pass:', current);
+    dev().info(TAG, 'pass:', current);
     const scheduleNextPass = () => {
       // If there's a new `renderItems_`, schedule it for render.
       if (this.renderItems_ !== current) {
@@ -336,7 +336,7 @@ export class AmpList extends AMP.BaseElement {
    * @private
    */
   render_(elements) {
-    dev().fine(TAG, 'render:', elements);
+    dev().info(TAG, 'render:', elements);
 
     this.mutateElement(() => {
       removeChildren(dev().assertElement(this.container_));

src/log.js

@@ -600,6 +600,7 @@ export function resetLogConstructorForTesting() {
  *  1. Not disabled using `#log=0`.
  *  2. Development mode is enabled via `#development=1` or logging is explicitly
  *     enabled via `#log=D` where D >= 1.
+ *  3. AMP.setLogLevel(D) is called, where D >= 1.
  *
  * @param {!Element=} opt_element
  * @return {!Log}
@@ -644,6 +645,7 @@ function getUserLogger(suffix) {
  * Enabled in the following conditions:
  *  1. Not disabled using `#log=0`.
  *  2. Logging is explicitly enabled via `#log=D`, where D >= 2.
+ *  3. AMP.setLogLevel(D) is called, where D >= 2.
  *
  * @return {!Log}
  */