ResizeObserver calculates wrong content rectangle of <svg> elements
Reported by
que....@gmail.com,
Aug 26 2016
|
|||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.0 Safari/537.36 Steps to reproduce the problem: 1. Take a look at this example: http://jsbin.com/kosujulefo/edit?html,js,console What is the expected behavior? As for the spec (https://wicg.github.io/ResizeObserver/#content-rect-h) "top" and "left" properties of a content rectangle of an <svg> element need to be 0 even if element has paddings. What went wrong? Padding values of content rectangle are non-zero. Did this work before? No Chrome version: 54.0.2840.0 Channel: canary OS Version: 10.0 Flash Version: Shockwave Flash 22.0 r0
,
Aug 29 2016
,
Aug 30 2016
Considering this is a Feature request and marking it as untriaged.
,
Aug 31 2016
,
Sep 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4 commit 879fbd0230e2bf90bd19a6b03c22f20fd31f46e4 Author: atotic <atotic@google.com> Date: Fri Sep 09 00:31:06 2016 Fix svg contentrect location I was not computing entry.contetRect location correctly for <svg> elements. It should be 0,0 BUG= 641364 Review-Url: https://codereview.chromium.org/2305893002 Cr-Commit-Position: refs/heads/master@{#417450} [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/LayoutTests/resize-observer/svg.html [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/Source/core/observer/ResizeObservation.cpp [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/Source/core/observer/ResizeObservation.h [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/Source/core/observer/ResizeObserver.cpp [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/Source/core/observer/ResizeObserverEntry.cpp [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/Source/core/observer/ResizeObserverEntry.h [modify] https://crrev.com/879fbd0230e2bf90bd19a6b03c22f20fd31f46e4/third_party/WebKit/Source/web/tests/ResizeObserverTest.cpp
,
Sep 9 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by kapishnikov@chromium.org
, Aug 26 2016