Comment #2 on issue 27210 by [email protected]: "Always Show Bookmarks
Bar" checkmark out of sync on 2 menus
http://code.google.com/p/chromium/issues/detail?id=27210
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=31654
------------------------------------------------------------------------
r31654 | [email protected] | 2009-11-11 00:19:13 -0800 (Wed, 11 Nov 2009)
| 11 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/chromeos/status_area_view.cc?r1=31654&r2=31653
M
http://src.chromium.org/viewvc/chrome/trunk/src/views/controls/menu/native_menu_gtk.cc?r1=31654&r2=31653
M
http://src.chromium.org/viewvc/chrome/trunk/src/views/controls/menu/native_menu_gtk.h?r1=31654&r2=31653
Fix for issue 27210, show bookmarks bar menu is out of sync with the
browser's status.
* Update the states of menu before showing menu like Window's menu does.
* Don't fire activate signal when the state is chagned via API.
* Eliminated UpdateStateData and replace it with the NativeMenuGtk and the
position
stored in the gtk widget.
* Fixed StateAreaView to check bookmark bar status.
BUG=27210
TEST=Run linux builds (gtk/views/chromeos) check if the bookmark bar menu's
status is in sync with the bookmark bar.
Review URL: http://codereview.chromium.org/383012
------------------------------------------------------------------------
--
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
