Chromium Code Reviews
DescriptionMerge 21480 - Adds the first run banner.
First run is determined by checking if there are no thumbnails. The check for no thumbnails is done on the backend since I need to add code to add some pre populated thumbnails and therefore the most visited pages will not be empty on the client. There is one edge case where the banner can show up again and that is if the user blacklists all his thumbnails and restarts chrome.
Also, updates the look of the fillers to match the latest mocks.
BUG=15079
TEST=Start with a new empty profile. You should see the blue first run banner. Then navigate to some link and come back the NTP. The banner should not be shown again. Also, test that this did not break the yellow notification banner shown when a thumbnail is removed.
Review URL: http://codereview.chromium.org/159304
TBR=arv@google.com
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21737
Patch Set 1 #
Messages
Total messages: 1 (0 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||