Page actions behave as disableable browser actions when script badges are enabled.

chrome/browser/ui/views/extensions/browser_action_overflow_menu_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/views/extensions/browser_action_overflow_menu_controller.h"
 
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_context_menu_model.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 69 matching lines @@
     // command.
     MessageLoop::current()->DeleteSoon(FROM_HERE, this);
   }
   return true;
 }
 
 void BrowserActionOverflowMenuController::CancelMenu() {
   menu_->Cancel();
 }
 
+bool BrowserActionOverflowMenuController::IsCommandEnabled(int id) const {
+  BrowserActionView* view = (*views_)[start_index_ + id - 1];
+  return view->button()->IsEnabled(owner_->GetCurrentTabId());

[not at google - send to devlin, 2012/07/12 01:07:29]

Same comment here and in the GTK code:

ExtensionToolbarModel already knows to ignore events when browser actions are
disabled. Do we need this too? It's just less code to let the model deal with
it.

(otoh if this kind of thing changes the appearance of buttons, or doesn't play a
sound, or is used by some accessibility tool, or whatever...)

[Yoyo Zhou, 2012/07/12 01:33:27]

I seem to recall this changes the appearance of menu items to make them
disabled.

+}
+
 void BrowserActionOverflowMenuController::ExecuteCommand(int id) {
   BrowserActionView* view = (*views_)[start_index_ + id - 1];
   owner_->OnBrowserActionExecuted(view->button());
 }
 
 bool BrowserActionOverflowMenuController::ShowContextMenu(
     views::MenuItemView* source,
     int id,
     const gfx::Point& p,
     bool is_mouse_gesture) {
@@ skipping 113 matching lines @@
 
 BrowserActionView* BrowserActionOverflowMenuController::ViewForId(
     int id, size_t* index) {
   // The index of the view being dragged (GetCommand gives a 1-based index into
   // the overflow menu).
   size_t view_index = owner_->VisibleBrowserActions() + id - 1;
   if (index)
     *index = view_index;
   return owner_->GetBrowserActionViewAt(view_index);
 }