Revert "Simplify SVGSVGElement::getElementById method"

Revert "Simplify SVGSVGElement::getElementById method"

This reverts commit 54c88454a2f94147690450bc9b499361044c8ba3.

Reverting this fix because it assumed when SVG.getElementById is called the element is already part of the document and did not search svg subtree when the element is not found using above api.

Add test when svg element is not part of any document, it should still
be able to getElementById for it's child elements.

BUG=351831
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169157

[maheshkk, 2014-03-13 17:09:10]

PTAL

[Inactive, 2014-03-13 17:12:33]

On 2014/03/13 17:09:10, maheshkk wrote:
> PTAL

Please explain in the changelog the reason why you are reverting (the reason the
patch was wrong).

[maheshkk, 2014-03-13 18:02:20]

On 2014/03/13 17:12:33, Chris Dumez wrote:
> On 2014/03/13 17:09:10, maheshkk wrote:
> > PTAL
> 
> Please explain in the changelog the reason why you are reverting (the reason
the
> patch was wrong).

Updated description.

[Inactive, 2014-03-13 18:11:35]

lgtm

https://codereview.chromium.org/196193003/diff/1/Source/core/svg/SVGSVGElemen...
File Source/core/svg/SVGSVGElement.cpp (right):

https://codereview.chromium.org/196193003/diff/1/Source/core/svg/SVGSVGElemen...
Source/core/svg/SVGSVGElement.cpp:828: for (Node* node = firstChild(); node;
node = NodeTraversal::next(*node, this)) {
We may want to use ElementTraversal API in a follow up to make the code a bit
simpler.

[commit-bot: I haz the power, 2014-03-13 18:11:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mahesh.kk@samsung.com/196193003/1

[commit-bot: I haz the power, 2014-03-13 19:03:29]

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

[commit-bot: I haz the power, 2014-03-13 19:03:33]

Try jobs failed on following builders:
  tryserver.chromium on win_blink_rel

[maheshkk, 2014-03-13 19:09:58]

The CQ bit was checked by mahesh.kk@samsung.com

[commit-bot: I haz the power, 2014-03-13 19:10:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mahesh.kk@samsung.com/196193003/1

[commit-bot: I haz the power, 2014-03-13 22:02:57]

Change committed as 169157