Use isHTMLPlugInElement to replace checks

Use isHTMLPlugInElement to replace checks 

All the checked Elements (<object>, <embed>, <applet>) derive from
HTMLPlugInElement and there is no other class deriving from it, so
use isHTMLPlugInElement here instead.

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

[rwlbuis, 2014-07-23 21:00:44]

PTAL.

[Inactive, 2014-07-23 21:05:10]

Can you take care of those in Source/core/accessibility/AXNodeObject.cpp as
well?

[rwlbuis, 2014-07-23 21:11:18]

On 2014/07/23 21:05:10, Chris Dumez wrote:
> Can you take care of those in Source/core/accessibility/AXNodeObject.cpp as
> well?

Nice find, will update.

[rwlbuis, 2014-07-23 21:15:51]

PTAL, I found even one more after a grep :)

[Inactive, 2014-07-23 21:21:14]

https://codereview.chromium.org/413783002/diff/20001/Source/core/html/WindowN...
File Source/core/html/WindowNameCollection.cpp (left):

https://codereview.chromium.org/413783002/diff/20001/Source/core/html/WindowN...
Source/core/html/WindowNameCollection.cpp:23: || isHTMLAppletElement(element)
Actually, I did not suggest this one because I prefer is we check explicitly for
applet, embed, object, as specified in the comment above and the spec
(http://www.whatwg.org/specs/web-apps/current-work/multipage/browsers.html#dom...).
I think it is clearer to leave as it was.

[rwlbuis, 2014-07-23 21:28:34]

Ok I backed out the WindowNameCollection change.

[Inactive, 2014-07-23 21:30:09]

On 2014/07/23 21:28:34, rwlbuis wrote:
> Ok I backed out the WindowNameCollection change.

Thanks, lgtm.

[rwlbuis, 2014-07-23 21:30:57]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-07-23 21:31:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/413783002/40001

[commit-bot: I haz the power, 2014-07-24 04:21:36]

Change committed as 178805