Comment #5 on issue 24379 by bugdro...@chromium.org: Remove menu  
integration from browser actions
http://code.google.com/p/chromium/issues/detail?id=24379

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=29095

------------------------------------------------------------------------
r29095 | a...@chromium.org | 2009-10-14 23:07:38 -0700 (Wed, 14 Oct 2009) | 8  
lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/chrome_dll_resource.h?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.cc?r1=29095&r2=29094
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/browser_action_apitest.cc
    D /trunk/src/chrome/browser/extensions/browser_action_test.cc
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_apitest.cc?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_apitest.h?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_browser_actions_api.cc?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_browser_actions_api.h?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_actions_toolbar_gtk.cc?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/browser_actions_container.cc?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/toolbar_view.cc?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/toolbar_view.h?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.gyp?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/extension_action.cc?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/extension_action.h?r1=29095&r2=29094
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/resources/extension_process_bindings.js?r1=29095&r2=29094
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action/background.html
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action/icon.png
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action/icon2.png
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action/manifest.json
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action/update.html
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action_no_icon
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action_no_icon/background.html
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action_no_icon/manifest.json
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/browser_action_no_icon/update.html

Remove the implicit wrench menu items for browser actions.

Also, allow browser actions with no initial icons, and add
some better tests.

BUG=24379,24671

Review URL: http://codereview.chromium.org/276010
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to