Default zoom was never read from chrome://settings

chrome/browser/ui/browser_commands.cc

 // Copyright 2012 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/ui/browser_commands.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 32 matching lines @@
 #include "chrome/browser/ui/find_bar/find_bar.h"
 #include "chrome/browser/ui/find_bar/find_bar_controller.h"
 #include "chrome/browser/ui/find_bar/find_tab_helper.h"
 #include "chrome/browser/ui/fullscreen/fullscreen_controller.h"
 #include "chrome/browser/ui/location_bar/location_bar.h"
 #include "chrome/browser/ui/scoped_tabbed_browser_displayer.h"
 #include "chrome/browser/ui/search/search_tab_helper.h"
 #include "chrome/browser/ui/status_bubble.h"
 #include "chrome/browser/ui/tab_contents/core_tab_helper.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/zoom/chrome_zoom_level_prefs.h"
 #include "chrome/browser/ui/zoom/zoom_controller.h"
 #include "chrome/browser/upgrade_detector.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/content_restriction.h"
 #include "chrome/common/pref_names.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "components/bookmarks/browser/bookmark_utils.h"
 #include "components/google/core/browser/google_util.h"
 #include "components/translate/core/browser/language_state.h"
@@ skipping 538 matching lines @@
   return zoom_controller->GetZoomPercent() !=
       contents->GetMaximumZoomPercent() + 1;
 }
 
 bool CanZoomOut(content::WebContents* contents) {
   ZoomController* zoom_controller = ZoomController::FromWebContents(contents);
   return zoom_controller->GetZoomPercent() !=
       contents->GetMinimumZoomPercent();
 }
 
-bool ActualSize(content::WebContents* contents) {
+bool ActualSize(content::WebContents* contents, Browser* browser) {
   ZoomController* zoom_controller = ZoomController::FromWebContents(contents);
-  return zoom_controller->GetZoomPercent() != 100.0f;
+  return zoom_controller->GetZoomPercent() !=
+      zoom_controller->GetDefaultZoomPercent();

[Alexei Svitkine (slow), 2014/10/28 13:15:54]

Can't you use zoom_controller->IsAtDefaultZoom() instead of doing the same logic
here?

In fact, I find the name of that method more intuitive than this one - so maybe
rename this method to the same thing (i.e. IsAtDefaultZoom()).

 }
 
 TabStripModelDelegate::RestoreTabType GetRestoreTabType(
     const Browser* browser) {
   TabRestoreService* service =
       TabRestoreServiceFactory::GetForProfile(browser->profile());
   if (!service || service->entries().empty())
     return TabStripModelDelegate::RESTORE_NONE;
   if (service->entries().front()->type == TabRestoreService::WINDOW)
     return TabStripModelDelegate::RESTORE_WINDOW;
@@ skipping 664 matching lines @@
                                           browser->host_desktop_type()));
   app_browser->tab_strip_model()->AppendWebContents(contents, true);
 
   contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   contents->GetRenderViewHost()->SyncRendererPrefs();
   app_browser->window()->Show();
 }
 #endif  // defined(ENABLE_EXTENSIONS)
 
 }  // namespace chrome