Revert of Fix SVGDocument return title that is not a child of the root element

Revert of Fix SVGDocument return title that is not a child of the root element (patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue : https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
>
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
>
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
>
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
>
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
>
> BUG=543061
>
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=54306, 592323
Committed: https://crrev.com/b25217584cf92b17941a2db757baa2efd93a77d8
Cr-Commit-Position: refs/heads/master@{#379541}

[hyunjunekim2, 2016-03-07 07:36:55]

Created Revert of Fix SVGDocument return title that is not a child of the root
element

[commit-bot: I haz the power, 2016-03-07 07:37:03]

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

[commit-bot: I haz the power, 2016-03-07 07:37:04]

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

[commit-bot: I haz the power, 2016-03-07 07:37:05]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[hyunjunekim2, 2016-03-07 07:38:46]

Description was changed from

==========
Revert of Fix SVGDocument return title that is not a child of the root element
(patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue :https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
> 
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
> 
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
> 
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
> 
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
> 
> BUG=543061
> 
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=543061
==========

to

==========
Revert of Fix SVGDocument return title that is not a child of the root element
(patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue : https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
> 
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
> 
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
> 
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
> 
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
> 
> BUG=543061
> 
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=54306,592323
==========

[hyunjunekim2, 2016-03-07 07:38:46]

hyunjune.kim@samsung.com changed reviewers:
+ nyerramilli@chromium.org

[hyunjunekim2, 2016-03-07 10:41:51]

fs, nyerramilli

Could you revert this one?
This case has been crash.

var doc = document.implementation.createDocument(SVG_NAMESPACE, "SVG", null);
var title1 = doc.createElementNS(HTML_NAMESPACE, "title");
title1.textContent = "foo";
doc.documentElement.appendChild(title1);
title2 = doc.createElementNS(HTML_NAMESPACE, "title");
doc.documentElement.appendChild(title2);

Because when append second child called |title2|,
return null on Travalsal<SVGTitleElement>firstWithin(*this);
where is |setTitleElement|.

I am sorry. After more research, i'll make new patch.
Thank you.

[fs, 2016-03-07 11:21:41]

The CQ bit was checked by fs@opera.com

[fs, 2016-03-07 11:21:42]

lgtm

[commit-bot: I haz the power, 2016-03-07 11:21:53]

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

[commit-bot: I haz the power, 2016-03-07 12:38:49]

Description was changed from

==========
Revert of Fix SVGDocument return title that is not a child of the root element
(patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue : https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
> 
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
> 
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
> 
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
> 
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
> 
> BUG=543061
> 
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=54306,592323
==========

to

==========
Revert of Fix SVGDocument return title that is not a child of the root element
(patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue : https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
> 
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
> 
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
> 
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
> 
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
> 
> BUG=543061
> 
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=54306,592323
==========

[commit-bot: I haz the power, 2016-03-07 12:38:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-07 12:41:10]

Description was changed from

==========
Revert of Fix SVGDocument return title that is not a child of the root element
(patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue : https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
> 
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
> 
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
> 
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
> 
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
> 
> BUG=543061
> 
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=54306,592323
==========

to

==========
Revert of Fix SVGDocument return title that is not a child of the root element
(patchset #12 id:220001 of https://codereview.chromium.org/1752713002/ )

Reason for revert:
Crash in blink:Document::setTitleElement.

Issue : https://bugs.chromium.org/p/chromium/issues/detail?id=592323

Original issue's description:
> Fix SVGDocument return title that is not a child of the root element
>
> The title element should be a child of the root element.
> But currently return invalid title that is not a child of the root element.
>
> Because on getting, if the root element is an svg element in the SVG
> namespace, then let value ba a concatenation of the data of all
> the child Text nodes of the first title element in the SVG namespace
> that is a child of the root element.[1]
>
> And on setting, let element be the first title element in the SVG
> namespace that is a child of the root element, if any.
> If there isn't one, create a title element in the SVG namespace,
> insert it as the first child of the root element, and let element
> be that element.[1]
>
> [1] https://html.spec.whatwg.org/multipage/dom.html#document.title
>
> BUG=543061
>
> Committed: https://crrev.com/2764e5d8a0ec6cf13cb4d918a5032e45da691247
> Cr-Commit-Position: refs/heads/master@{#379301}

TBR=fs@opera.com,pdr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=54306,592323

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

[commit-bot: I haz the power, 2016-03-07 12:41:11]

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