Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEeDwAJ

BUG=463116
Committed: https://crrev.com/b515424de0aa3d62c4d64f4a73cf0bff137539c8
Cr-Commit-Position: refs/heads/master@{#377283}

[philipj_slow, 2016-02-23 17:28:42]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[philipj_slow, 2016-02-23 17:30:05]

philipj@opera.com changed reviewers:
+ chrishtr@chromium.org, timvolodine@chromium.org

[philipj_slow, 2016-02-23 17:30:06]

PTAL. I haven't let the dry run finish, but am pretty sure that if any test
fails, it'll be something trivial.

[commit-bot: I haz the power, 2016-02-23 17:34:28]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1726743002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1726743002/1

[chrishtr, 2016-02-23 17:41:51]

Please be prepared to roll this back and re-land in one more release if a lot of
authors complain. We've been beating up on SVG developers a bit lately (e.g.
SVGPathSeg), and we should be sensitive to the cost of updating sites when user
agents break them.

[chrishtr, 2016-02-23 17:42:03]

lgtm

[philipj_slow, 2016-02-23 17:44:05]

On 2016/02/23 17:41:51, chrishtr wrote:
> Please be prepared to roll this back and re-land in one more release if a lot
of
> authors complain. We've been beating up on SVG developers a bit lately (e.g.
> SVGPathSeg), and we should be sensitive to the cost of updating sites when
user
> agents break them.

Yeah, if some usage surfaces which is either widespread or non-trivial to fix,
we should delay this by a release cycle or two.

[timvolodine, 2016-02-23 18:14:28]

android_webview/tools/WebViewShell/test/webexposed/global-interface-listing-expected.txt
-- lgtm

[commit-bot: I haz the power, 2016-02-23 19:15:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-23 19:15:56]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[philipj_slow, 2016-02-24 03:03:54]

Description was changed from

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

event-dispatching.js is updated to use getBoundingClientRect()

BUG=463116
==========

to

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

Tests were updated to use getBoundingClientRect(). One test,
smil-scheduled-in-inactive-document-crash.html, didn't actually run the
problematic code because of a early finishJSTest(), which was fixed.

BUG=463116
==========

[philipj_slow, 2016-02-24 03:20:21]

Description was changed from

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

Tests were updated to use getBoundingClientRect(). One test,
smil-scheduled-in-inactive-document-crash.html, didn't actually run the
problematic code because of a early finishJSTest(), which was fixed.

BUG=463116
==========

to

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

BUG=463116
==========

[philipj_slow, 2016-02-24 04:56:44]

Some more tests needed to be updated, so in order to make this easier to revert
I moved all of the test changes that could be moved into
https://codereview.chromium.org/1729073002/ and will land that first.

[philipj_slow, 2016-02-24 10:55:51]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2016-02-24 10:55:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, timvolodine@chromium.org
Link to the patchset: https://codereview.chromium.org/1726743002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-02-24 10:57:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1726743002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1726743002/60001

[commit-bot: I haz the power, 2016-02-24 13:40:49]

Description was changed from

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

BUG=463116
==========

to

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

BUG=463116
==========

[commit-bot: I haz the power, 2016-02-24 13:40:50]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-02-24 13:41:57]

Description was changed from

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

BUG=463116
==========

to

==========
Remove SVGElement.offsetParent/offsetTop/offsetLeft/offsetWidth/offsetHeight

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/jjwLLSG_hGY/Ovi-nvEe...

BUG=463116

Committed: https://crrev.com/b515424de0aa3d62c4d64f4a73cf0bff137539c8
Cr-Commit-Position: refs/heads/master@{#377283}
==========

[commit-bot: I haz the power, 2016-02-24 13:41:59]

Patchset 4 (id:??) landed as
https://crrev.com/b515424de0aa3d62c4d64f4a73cf0bff137539c8
Cr-Commit-Position: refs/heads/master@{#377283}