Make command-zero reset page scale in addition to zoom level

chrome/browser/ui/browser_command_controller.cc

 // Copyright (c) 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_command_controller.h"
 
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 1125 matching lines @@
   // Update the zoom commands when an active tab is selected.
   UpdateCommandsForZoomState();
 }
 
 void BrowserCommandController::UpdateCommandsForZoomState() {
   WebContents* contents =
       browser_->tab_strip_model()->GetActiveWebContents();
   if (!contents)
     return;
   command_updater_.UpdateCommandEnabled(IDC_ZOOM_PLUS, CanZoomIn(contents));
-  command_updater_.UpdateCommandEnabled(IDC_ZOOM_NORMAL, ActualSize(contents));
   command_updater_.UpdateCommandEnabled(IDC_ZOOM_MINUS, CanZoomOut(contents));
+  // Reset zoom is always enabled because it also resets page scale.
+  command_updater_.UpdateCommandEnabled(IDC_ZOOM_NORMAL, true);

[Avi (use Gerrit), 2015/02/26 20:16:58]

Can we have it be enabled only if _either_ there is a custom zoom or scale? If
there is no zoom or scale, can we have it disabled?

[ccameron, 2015/02/26 20:23:06]

That information is really deep the renderer and can be changed continuously
with pinch-zoom. It would take an enormous amount of IPC traffic to keep them in
sync (not to mention bug-prone plumbing). Is there a downside to having this
command be enabled?

[Nico, 2015/02/26 20:35:22]

You only need to send "is at no pinch" and "is at pinch". That shouldn't need
much traffic?

 }
 
 void BrowserCommandController::UpdateCommandsForContentRestrictionState() {
   int restrictions = GetContentRestrictions(browser_);
 
   command_updater_.UpdateCommandEnabled(
       IDC_COPY, !(restrictions & CONTENT_RESTRICTION_COPY));
   command_updater_.UpdateCommandEnabled(
       IDC_CUT, !(restrictions & CONTENT_RESTRICTION_CUT));
   command_updater_.UpdateCommandEnabled(
@@ skipping 195 matching lines @@
 
 BrowserWindow* BrowserCommandController::window() {
   return browser_->window();
 }
 
 Profile* BrowserCommandController::profile() {
   return browser_->profile();
 }
 
 }  // namespace chrome