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

Issue 597244 link

Starred by 12 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Outgoing links are not seen for any webpage on mouse hover.

Reported by vvishwak...@etouch.net, Mar 23 2016

Issue description

Version: 51.0.2688.0 (Official Build) df372032b408542d782c18cfc057ed729e46665a-refs/heads/master@{#382748} (32/64-bit)
OS: Windows

What steps will reproduce the problem?
1) Launch chrome & go to http://scoopwhoop.com/ .
2) Mouse hover on any news content and observe bottom of the page.

On mouse hover outgoing links are not seen at the bottom.

On mouse hover outgoing links should be seen at the bottom.

This is a Regression issue broken in M-51, will soon update other info

 
Labels: M-51 hasbisect
Owner: ananta@chromium.org
Status: Assigned (was: Unconfirmed)
Manual bisect:
Good build: 51.0.2686.0 
Bad build: 51.0.2687.0

Narrow bisect URL:
https://chromium.googlesource.com/chromium/src/+log/198ef342ac9036f0a05bcebf111ee5756eb7b78b..179d0904ce723736734a34ee0fdd0e567a3c4005?pretty=fuller&n=100

Suspecting: r382429 ?

Note: Issue is Windows 10 specific. It is not seen on Win 7, 8 , Linux and Mac OS.
link_actual.mp4
934 KB Download
links_expected.mp4
747 KB Download

Comment 2 by mar...@gu1dry.com, Mar 23 2016

This happens to me as well on Windows 7 Pro x64 on Chrome build 51.0.2687.0 dev-m.

Comment 3 by ananta@chromium.org, Mar 23 2016

Labels: Merge-Request-50 M-50
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 23 2016

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

commit 5a941ea1cd19d6a92d197e16ab4508652ff139d0
Author: ananta <ananta@chromium.org>
Date: Wed Mar 23 21:57:20 2016

Fix another painting regression caused by my patch to address painting problems with the fullscreen window 1 px reduction.

We are setting the size of the compositor as that returned by the DesktopWindowTreeHostWin::GetBounds call. That function
returns the window size offset by the window expansion and window enlargement rectangles, which is not the correct size.

Fix is to pass the bounds size as returned by the HWNDMessageHandler.

BUG= 597244 

Review URL: https://codereview.chromium.org/1828833002

Cr-Commit-Position: refs/heads/master@{#382934}

[modify] https://crrev.com/5a941ea1cd19d6a92d197e16ab4508652ff139d0/ui/views/widget/desktop_aura/desktop_window_tree_host_win.cc

Comment 5 by tin...@google.com, Mar 24 2016

Labels: -Merge-Request-50 Merge-Approved-50 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M50 (branch: 2661)
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 24 2016

Labels: -merge-approved-50 merge-merged-2661
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1ea30795e2a2ca1ed748ae6d677b4fc24c73e6ee

commit 1ea30795e2a2ca1ed748ae6d677b4fc24c73e6ee
Author: Anantanarayanan Iyengar <ananta@chromium.org>
Date: Thu Mar 24 21:57:36 2016

Fix another painting regression caused by my patch to address painting problems with the fullscreen window 1 px reduction.

Merging to M50

We are setting the size of the compositor as that returned by the DesktopWindowTreeHostWin::GetBounds call. That function
returns the window size offset by the window expansion and window enlargement rectangles, which is not the correct size.

Fix is to pass the bounds size as returned by the HWNDMessageHandler.

BUG= 597244 

Review URL: https://codereview.chromium.org/1828833002

Cr-Commit-Position: refs/heads/master@{#382934}
(cherry picked from commit 5a941ea1cd19d6a92d197e16ab4508652ff139d0)

Review URL: https://codereview.chromium.org/1831163002 .

Cr-Commit-Position: refs/branch-heads/2661@{#385}
Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081}

[modify] https://crrev.com/1ea30795e2a2ca1ed748ae6d677b4fc24c73e6ee/ui/views/widget/desktop_aura/desktop_window_tree_host_win.cc

This issue seems to be connected with goo.gl/TkPkur
Close as Fixed?
Cc: jialiul@chromium.org scottmg@chromium.org ananta@chromium.org ashej...@chromium.org
 Issue 597573  has been merged into this issue.
Labels: TE-Verified-M50 TE-Verified-50.0.2661.57
Tested the issue on Windows 7 using 50.0.2661.57.Observed that on mouse hover, outgoing links are seen at the bottom.
Please find attached screencast.

Marking it as TE-Verified.
597244.mp4
1.8 MB Download
Status: Fixed (was: Assigned)

Sign in to add a comment