bookmarks: Bring Bookmark Bubble closer to the mocks.

bookmarks: Bring Bookmark Bubble closer to the mocks.

This changes the text and color of "Bookmark Added" to match with what is
shown in the mocks.

BUG=231694
TEST=go to any page, press Ctrl+D or press the star button in the omnibox.
Observe the Bookmkar Bubble, the text should read "Bookmark added!" and should
be in black color.
R=msw@chromium.org, sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203325

[tfarina, 2013-05-26 15:14:53]

Mike, I didn't address the link->button yet. I'm with very limited access to
internet connection this week.

I think this should be a good start.

Screenshot attached to the bug.

Please, take a look.

Thanks,

[msw, 2013-05-27 20:47:31]

Restore the margins or fix the control layout, the bubble looks odd.
I suppose the title color and casing change seems good on its own.

Tips on posting WIP CL screenshots:
1) Add a caption; describe and/or link to the CL (and patch set as needed).
2) Reduce useless image area (shrink the window first, or crop the picture).
3) Include some before/after or mock comparison as needed (ideally one pic).
4) Don't spam intermediate WIP shots, be patient and post polished work.

https://codereview.chromium.org/15808006/diff/1/chrome/browser/ui/views/bookm...
File chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc (right):

https://codereview.chromium.org/15808006/diff/1/chrome/browser/ui/views/bookm...
chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc:256:
set_margins(gfx::Insets(20, 20, 20, 20));
The margins look absurd without control size/placement/spacing changes.
I'd recommend not changing this without changing those.

[tfarina, 2013-05-29 01:48:31]

Restored the margins.

Please, take another look.

[msw, 2013-05-29 02:10:27]

Update the CL description and this LGTM.

[tfarina, 2013-05-29 02:11:14]

Done.

[tfarina, 2013-05-29 21:13:47]

Scott, ping?

[sky, 2013-05-29 23:36:18]

LGTM

[tfarina, 2013-05-31 03:40:44]

Committed patchset #2 manually as r203325 (presubmit successful).