Introduce ChromeZoomLevelPref, make zoom level prefs independent of profile prefs.

chrome/browser/chromeos/login/signin/merge_session_load_page.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/login/signin/merge_session_load_page.h"
 
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "base/i18n/rtl.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/chromeos/login/signin/oauth2_login_manager_factory.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/renderer_preferences_util.h"
 #include "chrome/browser/tab_contents/tab_util.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/browser_resources.h"
 #include "chrome/grit/generated_resources.h"
 #include "content/public/browser/browser_thread.h"
+#include "content/public/browser/host_zoom_map.h"
 #include "content/public/browser/interstitial_page.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_icon_set.h"
 #include "net/base/escape.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/webui/jstemplate_builder.h"
@@ skipping 56 matching lines @@
   base::StringPiece html(
       ResourceBundle::GetSharedInstance().GetRawDataResource(
           IDR_MERGE_SESSION_LOAD_HTML));
   return webui::GetI18nTemplateHtml(html, &strings);
 }
 
 void MergeSessionLoadPage::OverrideRendererPrefs(
       content::RendererPreferences* prefs) {
   Profile* profile = Profile::FromBrowserContext(
       web_contents_->GetBrowserContext());
+  // TODO(wjmaclean): Convert this to use the HostZoomMap for the WebContents
+  // when HostZoomMap moves to StoragePartition.
+  prefs->default_zoom_level = content::HostZoomMap::GetDefaultForBrowserContext(
+                                  profile)->GetDefaultZoomLevel();

[Peter Kasting, 2014/09/11 00:45:47]

What about
ZoomController::FromWebContents(web_contents_)->GetDefaultZoomLevel()?  That's
what you use in chrome_page_zoom.cc.

Same comment applies elsewhere you do this.

[wjmaclean, 2014/09/11 21:00:51]

Good catch, better to keep it consistent (and this eliminates the need for the
TODO as well).

Done.

   renderer_preferences_util::UpdateFromSystemSettings(prefs, profile);
 }
 
 void MergeSessionLoadPage::OnProceed() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   proceeded_ = true;
   NotifyBlockingPageComplete();
 }
 
 void MergeSessionLoadPage::OnDontProceed() {
@@ skipping 50 matching lines @@
                   " NOT " : "")
            << " be blocking now, "
            << state;
   if (!manager->ShouldBlockTabLoading()) {
     manager->RemoveObserver(this);
     interstitial_page_->Proceed();
   }
 }
 
 }  // namespace chromeos