Remove always-false null checks for getElementsByTagName(NS)

Remove always-false null checks for getElementsByTagName(NS)

getElementsByTagNameNS is called from WebNode::getElementsByHTMLTagName,
which in turn is only called with the static kLabel ("label").
getElementsByTagName is also called internally in
getElementsByTagNameNS. Other than this, both are called from bindings.
In no case is it possible for localName to be a null AtomicString, so
these checks are not needed.

Committed: https://crrev.com/43c8760bc6f96d7a8d4c4375d9186124e1c8dc0a
Cr-Commit-Position: refs/heads/master@{#370616}

[philipj_slow, 2016-01-18 15:27:56]

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

[commit-bot: I haz the power, 2016-01-18 15:28:06]

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

[commit-bot: I haz the power, 2016-01-18 17:05:12]

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

[commit-bot: I haz the power, 2016-01-18 17:05:13]

Dry run: This issue passed the CQ dry run.

[philipj_slow, 2016-01-20 15:32:49]

Description was changed from

==========
Remove always-false null checks for getElementsByTagName(NS)

BUG=
==========

to

==========
Remove always-false null checks for getElementsByTagName(NS)

getElementsByTagNameNS is called from WebNode::getElementsByHTMLTagName,
which in turn is only called with the static kLabel ("label").
getElementsByTagName is also called internally in
getElementsByTagNameNS. Other than this, both are called from bindings.
In no case is it possible for localName to be a null AtomicString, so
these checks are not needed.
==========

[philipj_slow, 2016-01-20 15:34:14]

philipj@opera.com changed reviewers:
+ tkent@chromium.org

[philipj_slow, 2016-01-20 15:34:14]

PTAL. If you prefer I could add ASSERTs.

[tkent, 2016-01-20 23:14:07]

The CQ bit was checked by tkent@chromium.org

[tkent, 2016-01-20 23:14:07]

lgtm

[commit-bot: I haz the power, 2016-01-20 23:15:04]

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

[commit-bot: I haz the power, 2016-01-21 04:12:47]

Description was changed from

==========
Remove always-false null checks for getElementsByTagName(NS)

getElementsByTagNameNS is called from WebNode::getElementsByHTMLTagName,
which in turn is only called with the static kLabel ("label").
getElementsByTagName is also called internally in
getElementsByTagNameNS. Other than this, both are called from bindings.
In no case is it possible for localName to be a null AtomicString, so
these checks are not needed.
==========

to

==========
Remove always-false null checks for getElementsByTagName(NS)

getElementsByTagNameNS is called from WebNode::getElementsByHTMLTagName,
which in turn is only called with the static kLabel ("label").
getElementsByTagName is also called internally in
getElementsByTagNameNS. Other than this, both are called from bindings.
In no case is it possible for localName to be a null AtomicString, so
these checks are not needed.
==========

[commit-bot: I haz the power, 2016-01-21 04:12:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-21 04:14:04]

Description was changed from

==========
Remove always-false null checks for getElementsByTagName(NS)

getElementsByTagNameNS is called from WebNode::getElementsByHTMLTagName,
which in turn is only called with the static kLabel ("label").
getElementsByTagName is also called internally in
getElementsByTagNameNS. Other than this, both are called from bindings.
In no case is it possible for localName to be a null AtomicString, so
these checks are not needed.
==========

to

==========
Remove always-false null checks for getElementsByTagName(NS)

getElementsByTagNameNS is called from WebNode::getElementsByHTMLTagName,
which in turn is only called with the static kLabel ("label").
getElementsByTagName is also called internally in
getElementsByTagNameNS. Other than this, both are called from bindings.
In no case is it possible for localName to be a null AtomicString, so
these checks are not needed.

Committed: https://crrev.com/43c8760bc6f96d7a8d4c4375d9186124e1c8dc0a
Cr-Commit-Position: refs/heads/master@{#370616}
==========

[commit-bot: I haz the power, 2016-01-21 04:14:05]

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