Remove SVGViewElement.viewTarget attribute

Remove SVGViewElement.viewTarget attribute

As per SVG2.0, SVGViewElement.viewTarget is removed.
https://github.com/w3c/svgwg/commit/c1690ae1b99aa95c648a304ec7a8f7dec46d7c83

Intent to Deprecate and Remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/X3kyDbj9xlA

BUG=633908
Committed: https://crrev.com/335c071421e78b900d0108a7fa6e621fb2b9d941
Cr-Commit-Position: refs/heads/master@{#432144}

[Shanmuga Pandi, 2016-11-09 13:58:38]

shanmuga.m@samsung.com changed reviewers:
+ foolip@chromium.org, fs@opera.com

[Shanmuga Pandi, 2016-11-09 13:58:38]

PTAL!!

[fs, 2016-11-09 14:35:01]

lgtm

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/svg/SVGViewElement.cpp (right):

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/SVGViewElement.cpp:25: #include
"core/svg/SVGStaticStringList.h"
Nit: Looks unused.

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/svg/SVGViewElement.h (right):

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/SVGViewElement.h:32: class
SVGStringListTearOff;
Nit: Can remove this too I think.

[foolip, 2016-11-09 15:08:58]

LGTM % nit

https://github.com/w3c/web-platform-tests/blob/master/svg/historical.html should
also test that this is removed, I'll send a PR.

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/svg/SVGViewElement.cpp (left):

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/SVGViewElement.cpp:33:
SVGStaticStringList::create(this, SVGNames::viewTargetAttr)) {
With this gone you can also drop viewTarget from SVGAttributeNames.in

[foolip, 2016-11-09 15:13:37]

Sent https://github.com/w3c/web-platform-tests/pull/4188

[Shanmuga Pandi, 2016-11-11 10:14:17]

shanmuga.m@samsung.com changed reviewers:
+ timvolodine@chromium.org

[Shanmuga Pandi, 2016-11-11 10:14:17]

timvolodine@chromium.org: Please review changes in android_webview

[timvolodine, 2016-11-11 15:00:23]

On 2016/11/11 10:14:17, Shanmuga Pandi wrote:
> mailto:timvolodine@chromium.org: Please review changes in android_webview

android_webview/.. -- lgtm

[Shanmuga Pandi, 2016-11-15 04:19:58]

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/svg/SVGViewElement.cpp (left):

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/SVGViewElement.cpp:33:
SVGStaticStringList::create(this, SVGNames::viewTargetAttr)) {
On 2016/11/09 15:08:58, foolip wrote:
> With this gone you can also drop viewTarget from SVGAttributeNames.in

Done.

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/svg/SVGViewElement.cpp (right):

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/SVGViewElement.cpp:25: #include
"core/svg/SVGStaticStringList.h"
On 2016/11/09 14:35:01, fs wrote:
> Nit: Looks unused.

Done.

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/svg/SVGViewElement.h (right):

https://codereview.chromium.org/2488703003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/SVGViewElement.h:32: class
SVGStringListTearOff;
On 2016/11/09 14:35:01, fs wrote:
> Nit: Can remove this too I think.

Done.

[Shanmuga Pandi, 2016-11-15 04:21:16]

The CQ bit was checked by shanmuga.m@samsung.com

[Shanmuga Pandi, 2016-11-15 04:21:17]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com,
timvolodine@chromium.org, foolip@chromium.org
Link to the patchset: https://codereview.chromium.org/2488703003/#ps20001
(title: "Rebased and align with comments")

[commit-bot: I haz the power, 2016-11-15 04:21:40]

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-15 04:43:24]

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

[commit-bot: I haz the power, 2016-11-15 04:43:25]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Shanmuga Pandi, 2016-11-15 05:04:18]

The CQ bit was unchecked by shanmuga.m@samsung.com

[Shanmuga Pandi, 2016-11-15 05:04:36]

The CQ bit was checked by shanmuga.m@samsung.com

[Shanmuga Pandi, 2016-11-15 05:04:36]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com,
timvolodine@chromium.org, foolip@chromium.org
Link to the patchset: https://codereview.chromium.org/2488703003/#ps40001
(title: "nits")

[commit-bot: I haz the power, 2016-11-15 05:04:59]

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

[Shanmuga Pandi, 2016-11-15 06:33:50]

The CQ bit was unchecked by shanmuga.m@samsung.com

[Shanmuga Pandi, 2016-11-15 06:36:28]

The CQ bit was checked by shanmuga.m@samsung.com

[Shanmuga Pandi, 2016-11-15 06:36:29]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com,
timvolodine@chromium.org, foolip@chromium.org
Link to the patchset: https://codereview.chromium.org/2488703003/#ps60001
(title: "fixed layout test failures in html5lib")

[commit-bot: I haz the power, 2016-11-15 06:36:45]

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-15 08:20:53]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-15 08:22:50]

Description was changed from

==========
Remove SVGViewElement.viewTarget attribute

As per SVG2.0, SVGViewElement.viewTarget is removed.
https://github.com/w3c/svgwg/commit/c1690ae1b99aa95c648a304ec7a8f7dec46d7c83

Intent to Deprecate and Remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/X3kyDbj9xlA

BUG=633908
==========

to

==========
Remove SVGViewElement.viewTarget attribute

As per SVG2.0, SVGViewElement.viewTarget is removed.
https://github.com/w3c/svgwg/commit/c1690ae1b99aa95c648a304ec7a8f7dec46d7c83

Intent to Deprecate and Remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/X3kyDbj9xlA

BUG=633908

Committed: https://crrev.com/335c071421e78b900d0108a7fa6e621fb2b9d941
Cr-Commit-Position: refs/heads/master@{#432144}
==========

[commit-bot: I haz the power, 2016-11-15 08:22:51]

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