Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(90)

Issue 12390052: gtk/bookmarks: Display bookmark folder icon on Bookmarks menu. (Closed)

Created:
7 years, 9 months ago by tfarina
Modified:
7 years, 9 months ago
Reviewers:
Elliot Glaysher
CC:
chromium-reviews
Visibility:
Public.

Description

gtk/bookmarks: Display bookmark folder icon on Bookmarks menu. BUG=172594 TEST= 1- launch chrome 2- bookmark some folders on bookmarks bar either through bookmarks bar or bookmark manager. 3- go to wrench (hotdogs) menu -> bookmarks menu. 4- now you should see the bookmark folder icon for folders as well. R=erg@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186189

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M chrome/browser/ui/gtk/bookmarks/bookmark_sub_menu_model_gtk.cc View 2 chunks +5 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
tfarina
Hi Elliot, could you review this to me?
7 years, 9 months ago (2013-03-04 00:19:13 UTC) #1
Elliot Glaysher
lgtm
7 years, 9 months ago (2013-03-04 19:10:55 UTC) #2
tfarina
7 years, 9 months ago (2013-03-05 17:02:18 UTC) #3
Message was sent while issue was closed.
Committed manually as r186189 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698