MD Settings: Stop calling chrome.settingsPrivate.setDefaultZoomLevel on startup.

chrome/browser/resources/settings/appearance_page/appearance_page.js

Index: chrome/browser/resources/settings/appearance_page/appearance_page.js
diff --git a/chrome/browser/resources/settings/appearance_page/appearance_page.js b/chrome/browser/resources/settings/appearance_page/appearance_page.js
index 5b7462bd9315bb65beb987e947d9e0919873dba2..5789801ebc1581c7caf8b11b8f39d0558ae662c0 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_page.js
+++ b/chrome/browser/resources/settings/appearance_page/appearance_page.js
@@ -49,6 +49,9 @@ Polymer({
       value: function() {
         return {
           type: chrome.settingsPrivate.PrefType.NUMBER,
+          /* Needed such that the pref can be initialized without unnecessarily
+           * invoking chrome.settingsPrivate.setDefaultZoomPercent(). */
+          initialized: false,

[michaelpg, 2016/10/11 03:47:33]

We probably shouldn't "overload" this value more than we already do -- it'd be a
partial PrefObject with an extra |initialized| member. How about a separate
defaultZoomLevelInitialized_ property?

[dpapad, 2016/10/11 17:30:17]

Done.

         };
       },
     },
@@ -213,6 +216,14 @@ Polymer({
     // The |percent| may be undefined on startup.
     if (percent === undefined)
       return;
+
+    if (!this.defaultZoomLevel_.initialized) {

[michaelpg, 2016/10/11 03:47:33]

Since we only use this here, I wouldn't be averse to just defining it here (and
nowhere else) rather than making it part of a "type".

[dpapad, 2016/10/11 17:30:17]

What do you mean defining it only here? Here in this scope? Needs to be
persisted across different calls of  zoomLevelChanged.

[michaelpg, 2016/10/11 22:01:10]

Oh, I was just saying, this is JS, we don't have to declare all our properties.
So using this.defaultZLI_ *only* in this function is OK:

if (!this.defaultZoomLevelInitialized_) {
  // Simply mark this pref as initialized, etc....
  /** @private */this.defaultZoomLevelInitialized_ = true;
  return;
}

as a plus, you don't need the 6-line comment in the latest patch.

[dpapad, 2016/10/11 22:05:54]

Ah, got it now. Personally I don't prefer augmenting properties to an object
outside the constructor. JS Compiler enforces this if @struct is used (which I
know we are not using), but nonetheless just feels wrong. I am fine with
declaring it with @private {boolean} before using it here.

+      // Simply mark this pref as initialized, no actual change was requested by
+      // the user.
+      this.defaultZoomLevel_.initialized = true;
+      return;
+    }
+
     chrome.settingsPrivate.setDefaultZoomPercent(percent);
   },