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 590 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) {

[Alexei Svitkine (slow), 2014/10/29 03:44:25]

I was suggesting we rename this function to also be called IsAtDefaultZoom() -
which is a better name, imho.

   ZoomController* zoom_controller = ZoomController::FromWebContents(contents);
-  return zoom_controller->GetZoomPercent() != 100.0f;
+  return !zoom_controller->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