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

Issue 8953004: Now that the launcher is sized to the width of the root window we need (Closed)

Created:
9 years ago by sky
Modified:
9 years ago
CC:
chromium-reviews, dhollowa+watch_chromium.org, Paweł Hajdan Jr., sadrul, ben+watch_chromium.org
Visibility:
Public.

Description

Now that the launcher is sized to the width of the root window we need to tell it the size of the launcher so that it doesn't layout on top of the status. BUG=106432 TEST=covered by tests. R=ben@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114658

Patch Set 1 #

Patch Set 2 : Tweaks #

Unified diffs Side-by-side diffs Delta from patch set Stats (+165 lines, -31 lines) Patch
M ui/aura_shell/aura_shell.gyp View 1 chunk +1 line, -0 lines 0 comments Download
M ui/aura_shell/launcher/launcher.h View 2 chunks +9 lines, -0 lines 0 comments Download
M ui/aura_shell/launcher/launcher.cc View 3 chunks +88 lines, -3 lines 0 comments Download
A ui/aura_shell/launcher/launcher_unittest.cc View 1 chunk +31 lines, -0 lines 0 comments Download
M ui/aura_shell/launcher/launcher_view.cc View 3 chunks +0 lines, -24 lines 0 comments Download
M ui/aura_shell/shelf_layout_manager.h View 1 1 chunk +6 lines, -4 lines 0 comments Download
M ui/aura_shell/shelf_layout_manager.cc View 2 chunks +4 lines, -0 lines 0 comments Download
M ui/aura_shell/shelf_layout_manager_unittest.cc View 2 chunks +26 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
sky
9 years ago (2011-12-15 01:09:34 UTC) #1
Ben Goodger (Google)
lgtm
9 years ago (2011-12-15 02:27:31 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sky@chromium.org/8953004/2001
9 years ago (2011-12-15 16:23:37 UTC) #3
commit-bot: I haz the power
9 years ago (2011-12-15 17:38:56 UTC) #4
Change committed as 114658

Powered by Google App Engine
This is Rietveld 408576698