Use new is*Element() helper functions more in web/ code

Source/web/ChromeClientImpl.cpp

Index: Source/web/ChromeClientImpl.cpp
diff --git a/Source/web/ChromeClientImpl.cpp b/Source/web/ChromeClientImpl.cpp
index 89c16b709325a15d0161415fa32fbe17fbb7a2f2..0d93d8c5a15ac83732716ca738ae6bd33439d635 100644
--- a/Source/web/ChromeClientImpl.cpp
+++ b/Source/web/ChromeClientImpl.cpp
@@ -543,11 +543,11 @@ void ChromeClientImpl::mouseDidMoveOverElement(
 
     WebURL url;
     // Find out if the mouse is over a link, and if so, let our UI know...
-    if (result.isLiveLink() && !result.absoluteLinkURL().string().isEmpty())
+    if (result.isLiveLink() && !result.absoluteLinkURL().string().isEmpty()) {
         url = result.absoluteLinkURL();
-    else if (result.innerNonSharedNode()
-             && (result.innerNonSharedNode()->hasTagName(HTMLNames::objectTag)
-                 || result.innerNonSharedNode()->hasTagName(HTMLNames::embedTag))) {
+    } else if (result.innerNonSharedNode()
+        && (isHTMLObjectElement(*result.innerNonSharedNode())

[adamk, 2014/03/14 20:49:12]

I liked the original indentation better, but I'm guessing lint didn't?

[Inactive, 2014/03/14 21:07:17]

Yes, the style script complained about indentation.

+            || isHTMLEmbedElement(*result.innerNonSharedNode()))) {
         RenderObject* object = result.innerNonSharedNode()->renderer();
         if (object && object->isWidget()) {
             Widget* widget = toRenderWidget(object)->widget();