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

Issue 9477018: retry r123804 - ntp theme fixes (Closed)

Created:
8 years, 10 months ago by Evan Stade
Modified:
8 years, 9 months ago
CC:
chromium-reviews, dbeam+watch-ntp_chromium.org, estade+watch_chromium.org, arv (Not doing code reviews), nkostylev+watch_chromium.org
Visibility:
Public.

Description

retry r123804 - ntp theme fixes 1. fix where the theme background is displayed for default (0) alignment -- it should be center center. This matches what the bookmar 1a. fix tests for alignment conversions 2. Fix the bookmark bar for vertical center alignment. 3. update the ntp background when the bookmark bar is attached/detached. TODO: transition (3) so that it looks good on windows and mac. BUG=115594 TEST=unit tests Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124014

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -46 lines) Patch
M chrome/browser/ntp_background_util.cc View 3 chunks +12 lines, -3 lines 0 comments Download
M chrome/browser/resources/chromeos/guest_session_tab.html View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/resources/incognito_tab.html View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/resources/ntp4/new_tab.html View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/resources/ntp4/new_tab.js View 2 chunks +13 lines, -5 lines 0 comments Download
M chrome/browser/resources/ntp4/new_tab_theme.css View 2 chunks +5 lines, -2 lines 0 comments Download
M chrome/browser/themes/theme_service.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/themes/theme_service.cc View 3 chunks +4 lines, -7 lines 0 comments Download
M chrome/browser/themes/theme_service_unittest.cc View 1 chunk +11 lines, -10 lines 1 comment Download
M chrome/browser/ui/webui/ntp/new_tab_ui.h View 2 chunks +3 lines, -0 lines 0 comments Download
M chrome/browser/ui/webui/ntp/new_tab_ui.cc View 3 chunks +12 lines, -6 lines 0 comments Download
M chrome/browser/ui/webui/ntp/ntp_resource_cache.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/theme_source.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/theme_source_unittest.cc View 2 chunks +5 lines, -4 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Stade
http://codereview.chromium.org/9477018/diff/1/chrome/browser/themes/theme_service_unittest.cc File chrome/browser/themes/theme_service_unittest.cc (right): http://codereview.chromium.org/9477018/diff/1/chrome/browser/themes/theme_service_unittest.cc#newcode21 chrome/browser/themes/theme_service_unittest.cc:21: EXPECT_EQ("center top", ThemeService::AlignmentToString(alignment)); we can't make it spit out ...
8 years, 10 months ago (2012-02-28 02:17:23 UTC) #1
Dan Beam
lgtm with trybots (which you seem to have, sans windows)
8 years, 10 months ago (2012-02-28 02:28:51 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/estade@chromium.org/9477018/1
8 years, 10 months ago (2012-02-28 02:42:04 UTC) #3
commit-bot: I haz the power
8 years, 10 months ago (2012-02-28 03:13:41 UTC) #4
Try job failure for 9477018-1 (retry) on linux_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

Powered by Google App Engine
This is Rietveld 408576698