New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 642694 link

Starred by 2 users
Status: Fixed
Owner:
michae...@chromium.org
Closed: Sep 2016
Cc:
nyerramilli@chromium.org
█ dschuyler@chromium.org
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , All , Mac
Pri: 1
Type: Bug-Regression
M54



Sign in to add a comment

Regression:Content doesn't appear after dragging drawer content from L.H.S of page.

Reported by vku...@etouch.net, Aug 31 2016 
Chrome Version:53.0.2785.89 (Official Build) a905cffb2d18baa410d0a19cc9d3941349f2d4b6-refs/branch-heads/2785@{#796} (32/64-bit) 
OS:Windows (7,8,8.1,10)

What steps will reproduce the problem?
(1)Launch chrome and navigate to chrome://md-settings/
(2)Drag and hold drawer content from R.H.S and instantly open context menu OR press middle mouse wheel,observe.


Actual: Content doesn't appear after dragging drawer content from R.H.S of page.

Expected: Content should be properly displayed even after dragging drawer content.


This is a regression issue broken in 'M53' and will soon update other info.

Comment 1 by vku...@etouch.net, Aug 31 2016

Labels: -M-53 M54 hasbisect OS-Linux OS-Mac
Owner: michae...@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: Regression:Content doesn't appear after dragging drawer content from L.H.S of page. (was: Regression:Content doesn't appear after dragging drawer content from R.H.S of page.)
Correction
This is a regression issue broken in 'M54' and below is the manual regression range:
Good Build: 54.0.2830.0
Bad Build:  54.0.2831.0

CL:
https://chromium.googlesource.com/chromium/src/+log/54.0.2830.0..54.0.2831.0?pretty=fuller&n=10000
(Unable to narrow down the range using tool)

Suspecting: 412350 ?
Kindly help to re-assign, if your changes are not cause for this issue.
Actual_Settings.mp4
256 KB View Download

Comment 2 by michae...@chromium.org, Aug 31 2016

Cc: dschuyler@chromium.org
Labels: -Pri-2 OS-All Pri-1
Status: Started (was: Assigned)
Whoops. I didn't know this peeking feature existed! Likely lazy-creating the settings-menu means it doesn't stamp until it's fully opened.
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 7 2016 

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

commit 64b7d7d026f7ef0d69c59845cc20b5649c48c8c7
Author: michaelpg <michaelpg@chromium.org>
Date: Wed Sep 07 04:32:25 2016

MD Settings menu should be visible

Currently the settings-menu is lazy-created when the app-drawer is opened.
But the app-drawer can be visible even when its opened property is false, like
when the drawer is being dragged partway open, or while it is closing. In these
cases, the settings-menu should still be stamped.

BUG= 642694 
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/64b7d7d026f7ef0d69c59845cc20b5649c48c8c7/chrome/browser/resources/settings/settings_ui/settings_ui.html
[modify] https://crrev.com/64b7d7d026f7ef0d69c59845cc20b5649c48c8c7/chrome/browser/resources/settings/settings_ui/settings_ui.js
[modify] https://crrev.com/64b7d7d026f7ef0d69c59845cc20b5649c48c8c7/chrome/test/BUILD.gn
[add] https://crrev.com/64b7d7d026f7ef0d69c59845cc20b5649c48c8c7/chrome/test/data/webui/settings/settings_ui_browsertest.js
[modify] https://crrev.com/64b7d7d026f7ef0d69c59845cc20b5649c48c8c7/ui/webui/resources/js/util.js

Comment 4 by michae...@chromium.org, Sep 20 2016

Status: Fixed (was: Started)
►

Sign in to add a comment