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

Issue 623467 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Scroll bar doesn't work after clicking on 'hide google chrome' option

Reported by vku...@etouch.net, Jun 27 2016

Issue description

Chrome Version:53.0.2780.0 (Official Build)5453dc6abc2d7c797c7f9f16df8aed8e6358a217-refs/heads/master@{#402053} (64-bit)
OS:Mac(10.10.5,10.11.4), Retina(10.11.4)

What steps will reproduce the problem?
(1)Launch chrome and enter any text to search for e.g 'test' in omnibox
(2)Click on 'hide google chrome' from chrome option of toolbar.
(3)Now click on chrome icon from dock and try to scroll the page.

Actual: Scroll bar doesn't work after clicking on 'hide google chrome' option.

Expected: Scroll bar should work after clicking on 'hide google chrome' option and launching chrome.

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



 

Comment 1 by vku...@etouch.net, Jun 27 2016

Labels: hasbisect
Owner: enne@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: Regression:Scroll bar doesn't work after clicking on 'hide google chrome' option (was: Regression:Scroll bar doesn't work after clicking on 'hide google chrome' option.)
Manual regression range:
Good Build: 53.0.2778.0
Bad Build:  53.0.2779.0

Narrow bisect:
https://chromium.googlesource.com/chromium/src/+log/3c51f5db7293b1e6100d44e71497428cde6d520a..e39df25f0accb5148a6b9249bac1ec900556e904?pretty=fuller&n=50

Suspecting: r401796 ?
Kindly help to re-assign, if your changes are not cause for this issue.

Note: 
1.Issue not seen on Win & Linux OS.
2.Issue is seen for all internal pages and webpages.
Actual_Scroll.mov
2.3 MB Download
Expected_Scroll.mov
2.5 MB Download

Comment 2 Deleted

Comment 3 Deleted

The issue can be seen on all web pages.
Thank you enne@ for the revert (https://codereview.chromium.org/2100203002/).
Project Member

Comment 6 by bugdroid1@chromium.org, Jun 27 2016

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

commit 7ea3a9cd847d7478598d22804d04c32c156c14f4
Author: enne <enne@chromium.org>
Date: Mon Jun 27 21:38:43 2016

Revert of Turn on enable begin frame scheduling by default (patchset #11 id:200001 of https://codereview.chromium.org/1939253002/ )

Reason for revert:
Causes smoothness regressions, other bugs

BUG= 623174 , 623467 , 623490 

Original issue's description:
> Turn on enable begin frame scheduling by default
>
> This turns on --enable-begin-frame-scheduling[1] for all[2] platforms.
> This was already on for Android so should only be a real change
> for desktop / ChromeOS platforms.
>
> Lots of cleanup can follow from this like removing all commit vsync /
> authoritative vsync / CompositorVSyncManager things, but this is a
> smaller patch to suss out any performance regressions.
>
> [1] In this case, "begin frame scheduling" means browser->renderer
> begin frame ticks instead of sending vsync information and having
> a synthetic source on the renderer side.
>
> [2] MUS is not hooked up to begin frame scheduling yet, but
> mojo:mash_session in an "oxygen" build still works with this patch
> applied.  Blimp also doesn't use begin frame scheduling and will
> eventually just be transitioned to a synthetic begin frame source
> for its engine half.
>
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/f2d7f5e1891703ec4384ededd80f896816921204
> Cr-Commit-Position: refs/heads/master@{#401796}

TBR=boliu@chromium.org,piman@chromium.org,skyostil@chromium.org,sunnyps@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

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

[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/android_webview/lib/main/aw_main_delegate.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/cc/base/switches.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/cc/base/switches.h
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/chrome/browser/chromeos/login/chrome_restart_request.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/content/browser/android/content_startup_flags.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/content/browser/compositor/browser_compositor_output_surface.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/content/renderer/gpu/render_widget_compositor.cc
[modify] https://crrev.com/7ea3a9cd847d7478598d22804d04c32c156c14f4/ui/compositor/compositor.cc

Comment 7 by enne@chromium.org, Jun 27 2016

Cc: ccameron@chromium.org
Status: Fixed (was: Assigned)
Should be fixed via the revert.  ccameron's https://codereview.chromium.org/2093113003/ appears to fix the issue with this patch applied, so will wait to reland until that goes in.

Sign in to add a comment