Fix TagWatcher when a text node is added.

Fix TagWatcher when a text node is added.

The previous implementation was causing Javascript errors, when a text node got inserted to the document. This patch fixes this issue by checking if the inserted node is an Element node.

TEST=(1) Add a <webview...></webview> node in inspector, (2) add <div><webview...></webview></div> node, (3) add text node in inspector.
BUG=222663
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194138

[mtomasz, 2013-04-12 07:19:29]

@benwells, @jeremya: PTAL at this tiny patch. Thanks.

[jeremya, 2013-04-12 16:41:06]

lgtm with change mentioned below :)

https://codereview.chromium.org/14222005/diff/1/chrome/renderer/resources/ext...
File chrome/renderer/resources/extensions/tag_watcher.js (right):

https://codereview.chromium.org/14222005/diff/1/chrome/renderer/resources/ext...
chrome/renderer/resources/extensions/tag_watcher.js:24: if (addedNode instanceof
Element) {
Rather use addedNode.nodeType === Node.ELEMENT_NODE here, I think. instanceof is
scary. See https://developer.mozilla.org/en-US/docs/DOM/Node.nodeType for more
info about nodeType.

[mtomasz, 2013-04-15 02:27:30]

https://codereview.chromium.org/14222005/diff/1/chrome/renderer/resources/ext...
File chrome/renderer/resources/extensions/tag_watcher.js (right):

https://codereview.chromium.org/14222005/diff/1/chrome/renderer/resources/ext...
chrome/renderer/resources/extensions/tag_watcher.js:24: if (addedNode instanceof
Element) {
On 2013/04/12 16:41:06, jeremya wrote:
> Rather use addedNode.nodeType === Node.ELEMENT_NODE here, I think. instanceof
is
> scary. See https://developer.mozilla.org/en-US/docs/DOM/Node.nodeType for more
> info about nodeType.

Done.

[commit-bot: I haz the power, 2013-04-15 02:27:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/14222005/7001

[commit-bot: I haz the power, 2013-04-15 04:53:37]

Change committed as 194138