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

Issue 2757453002: [Merge to M58] cros: Fix shelf overlaps bottom of maximized window in guest session (Closed)

Created:
3 years, 9 months ago by Qiang(Joe) Xu
Modified:
3 years, 9 months ago
Reviewers:
msw, James Cook
CC:
chromium-reviews, kalyank, sadrul
Target Ref:
refs/pending/branch-heads/3029
Project:
chromium
Visibility:
Public.

Description

[Merge to M58] cros: Fix shelf overlaps bottom of maximized window in guest session Changes: CL in https://codereview.chromium.org/2688353005 makes change that leave the shelf bottom locked and hidden until user prefs are loaded. When ChromeLauncherController::SetShelfBehaviorsFromPrefs(), it will be blocked on https://cs.chromium.org/chromium/src/ash/common/shelf/shelf_controller.cc?l=96, which leaves shelf alignment bottom locked in guest session. We don't set display work area change for bottom locked, that is why bug happens. Solution is to delete the CanChangeShelfAlignment() check as it should be enforced on shelf context menu already. TBR=msw@chromium.org, jamescook@chromium.org BUG=699661 TEST=emulator test saw reporter's bug fixed Review-Url: https://codereview.chromium.org/2748983004 Cr-Commit-Position: refs/heads/master@{#457246} (cherry picked from commit 5859fcdeb7fa7508f902d80d401958c956cb964f) Review-Url: https://codereview.chromium.org/2757453002 . Cr-Commit-Position: refs/branch-heads/3029@{#221} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} Committed: https://chromium.googlesource.com/chromium/src/+/05f8c49347384a74e7081d0b1947a61406f96677

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -3 lines) Patch
M ash/common/shelf/shelf_controller.cc View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
Qiang(Joe) Xu
Committed patchset #1 (id:1) manually as 05f8c49347384a74e7081d0b1947a61406f96677 (presubmit successful).
3 years, 9 months ago (2017-03-15 23:32:58 UTC) #2
msw
3 years, 9 months ago (2017-03-15 23:35:38 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698