Remove SVGCursorElement

Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG=650598
Committed: https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171
Cr-Commit-Position: refs/heads/master@{#438116}

[fs, 2016-11-21 13:13:24]

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

[commit-bot: I haz the power, 2016-11-21 13:13:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-21 15:26:34]

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

[commit-bot: I haz the power, 2016-11-21 15:26:34]

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

[fs, 2016-11-21 16:40:26]

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

[commit-bot: I haz the power, 2016-11-21 16:41:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-21 18:27:06]

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

[commit-bot: I haz the power, 2016-11-21 18:27:07]

Dry run: This issue passed the CQ dry run.

[fs, 2016-12-02 14:47:11]

Description was changed from

==========
Remove SVGCursorElement

BUG=650598
==========

to

==========
Remove SVGCursorElement

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

BUG=650598
==========

[fs, 2016-12-08 11:47:38]

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

[commit-bot: I haz the power, 2016-12-08 11:47:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fs, 2016-12-08 11:53:01]

Description was changed from

==========
Remove SVGCursorElement

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

BUG=650598
==========

to

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

BUG=650598
==========

[commit-bot: I haz the power, 2016-12-08 13:16:29]

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

[commit-bot: I haz the power, 2016-12-08 13:16:30]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[fs, 2016-12-08 13:30:59]

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

[commit-bot: I haz the power, 2016-12-08 13:31:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-08 15:18:26]

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

[commit-bot: I haz the power, 2016-12-08 15:18:27]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[fs, 2016-12-08 21:37:11]

fs@opera.com changed reviewers:
+ pdr@chromium.org, timloh@chromium.org, timvolodine@chromium.org

[fs, 2016-12-08 21:37:12]

timvolodine: android_webview/
timloh: CSSCursorImageValue changes
timpdr: rest

[fs, 2016-12-08 21:37:23]

Description was changed from

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

BUG=650598
==========

to

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

BUG=650598
==========

[pdr., 2016-12-09 05:51:02]

pdr@chromium.org changed reviewers:
+ sashab@chromium.org

[pdr., 2016-12-09 05:51:40]

pdr@chromium.org changed reviewers:
- sashab@chromium.org

[pdr., 2016-12-09 05:53:42]

On 2016/12/08 at 21:37:12, fs wrote:
> timvolodine: android_webview/
> timloh: CSSCursorImageValue changes
> timpdr: rest

LOL :)

LGTM! So much cleanup. I couldn't find any code this missed.



+cc sashab just FYI as you recently touched this area
(https://codereview.chromium.org/2018923002).

[sashab, 2016-12-12 02:00:17]

sashab@chromium.org changed reviewers:
+ sashab@chromium.org

[sashab, 2016-12-12 02:00:18]

Wow cool!! This patch: \o/

[Timothy Loh, 2016-12-12 02:01:58]

On 2016/12/08 21:37:12, fs wrote:
> timvolodine: android_webview/
> timloh: CSSCursorImageValue changes
> timpdr: rest

lgtm!

[fs, 2016-12-12 11:56:28]

fs@opera.com changed reviewers:
+ foolip@chromium.org

[fs, 2016-12-12 11:56:29]

+timfoolip for */webexposed/*

(Will TBR timvolodine, since he appears to be OOO.)

[foolip, 2016-12-12 13:00:06]

*/webexposed/* lgtm

[fs, 2016-12-12 13:06:58]

Description was changed from

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

BUG=650598
==========

to

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG=650598
==========

[fs, 2016-12-12 13:07:12]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-12-12 13:07:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 13:09:19]

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

[commit-bot: I haz the power, 2016-12-12 13:09:21]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[fs, 2016-12-12 14:13:18]

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

[commit-bot: I haz the power, 2016-12-12 14:13:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 15:34:07]

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

[commit-bot: I haz the power, 2016-12-12 15:34:08]

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

[fs, 2016-12-12 17:16:36]

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

[commit-bot: I haz the power, 2016-12-12 17:17:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-13 01:51:58]

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

[commit-bot: I haz the power, 2016-12-13 01:51:59]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[fs, 2016-12-13 10:13:13]

The CQ bit was checked by fs@opera.com

[fs, 2016-12-13 10:13:14]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org,
timloh@chromium.org, foolip@chromium.org
Link to the patchset: https://codereview.chromium.org/2522443002/#ps120001
(title: "Rebase")

[commit-bot: I haz the power, 2016-12-13 10:13:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-13 10:20:00]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1481623993743390,
"parent_rev": "c32f8fc270f5896905cf29b3601ff51b29d129b1", "commit_rev":
"2220cae41c0d2579feef20693eb9b1c708184bbe"}

[commit-bot: I haz the power, 2016-12-13 10:20:36]

Description was changed from

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG=650598
==========

to

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG=650598

Review-Url: https://codereview.chromium.org/2522443002
==========

[commit-bot: I haz the power, 2016-12-13 10:20:39]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-12-13 10:23:59]

Description was changed from

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG=650598

Review-Url: https://codereview.chromium.org/2522443002
==========

to

==========
Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG=650598

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

[commit-bot: I haz the power, 2016-12-13 10:24:00]

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