New issue
Advanced search Search tips

Issue 859461 link

Starred by 2 users
Status: Fixed
Owner:
chrishtr@chromium.org
Closed: Jul 4
Cc:
f...@opera.com
chrishtr@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

SVG foreignObject event issue when scrolled

Reported by genome.p...@googlemail.com, Jul 2 
UserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36

Steps to reproduce the problem:
1. Have a div in a foreignObject which is bigger than the browser window
2. Register an event handler on the div
3. Scroll the container

What is the expected behavior?
Events should get fired on all parts of the div

What went wrong?
Scroll position influences event triggering on parts (lower right edges) of the div although it is displayed correctly

Did this work before? Yes Worked with Chrome 65

Does this work in other browsers? Yes

Chrome version: 67.0.3396.99  Channel: stable
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: 

-Issue also on iOS 10.13
-Also with Dev version 69.0.3472.3
bug.html
959 bytes View Download

Comment 1 by schenney@chromium.org, Jul 2

Owner: chrishtr@chromium.org
Status: Assigned (was: Unconfirmed)
This seems familiar, but I thought the issues with hit testing foreign object were resolved and had rolled out.

Any insight on where to send it?

Comment 2 by genome.p...@googlemail.com, Jul 4 

Where to send what?
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 4 

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

commit f7c82afda798daf9cb95891a572a931b55460230
Author: Chris Harrelson <chrishtr@chromium.org>
Date: Wed Jul 04 16:08:39 2018

Fix root cull rect for hit testing under <foreignObject>.

It was incorrectly set to the view rect of the containing LayoutView.
Also fixed a similar accessibility bug along the way.

Bug:  859461 

Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I443bb56b2e8fac06a6af2efe4e9d3ec3ff6a9556
Reviewed-on: https://chromium-review.googlesource.com/1125510
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#572596}
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/core/layout/layout_view.cc
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/core/layout/svg/layout_svg_foreign_object.cc
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/core/layout/svg/layout_svg_foreign_object_test.cc
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/core/paint/paint_layer.cc
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/core/paint/paint_layer.h
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/core/paint/paint_layer_test.cc
[modify] https://crrev.com/f7c82afda798daf9cb95891a572a931b55460230/third_party/blink/renderer/modules/accessibility/ax_layout_object.cc

Comment 4 by chrishtr@chromium.org, Jul 4

Status: Fixed (was: Assigned)

Comment 5 by genome.p...@googlemail.com, Jul 5 

In which version was this issue fixed? Because I have the same problem with dev version 69.0.3472.3

Comment 6 by genome.p...@googlemail.com, Jul 5 

Ok, now I have tried with version 69.0.3482.0 (Canary) and it seems to be fixed there. So only one question remains: How long will it take to hit release?

Comment 7 by chrishtr@chromium.org, Jul 6 

The fix will reach the stable channel in about 7-8 weeks.

Comment 8 by chrishtr@chromium.org, Sep 28

Cc: f...@opera.com chrishtr@chromium.org
 Issue 866297  has been merged into this issue.
►

Sign in to add a comment