New issue
Advanced search Search tips

Issue 618426 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

content layer should snap to the toplevel on chromeos

Project Member Reported by osh...@chromium.org, Jun 8 2016

Issue description

looks like this is regressed in r371676.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/894e134acd22e115b55fe0b309dcb2dcf5786974

commit 894e134acd22e115b55fe0b309dcb2dcf5786974
Author: oshima <oshima@chromium.org>
Date: Thu Jun 09 03:53:11 2016

Snap content to the top level window on chromeos

BUG= 618426 
TEST=manual

Review-Url: https://codereview.chromium.org/2056503002
Cr-Commit-Position: refs/heads/master@{#398774}

[modify] https://crrev.com/894e134acd22e115b55fe0b309dcb2dcf5786974/content/browser/renderer_host/render_widget_host_view_aura.cc

Project Member

Comment 2 by sheriffbot@chromium.org, Jun 9 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Type-Bug Merge-Request-52 M-52 Type-Bug-Regression

Comment 4 by tin...@google.com, Jun 10 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)
Project Member

Comment 5 by sheriffbot@chromium.org, Jun 13 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by osh...@chromium.org, Jun 13 2016

Sorry I was OOO last Friday. Will merge soon.

Comment 7 by osh...@chromium.org, Jun 13 2016

Labels: -Merge-Approved-52 merge-merged-2743
Status: Fixed (was: Started)
I screw up commit message, my bad. The CL was merged at

https://chromium.googlesource.com/chromium/src.git/+/1a5840e8c54cb8995daadbba7300128fd0fc43b0



Status: Verified (was: Fixed)

Sign in to add a comment