Changeset 143684 in webkit

Timestamp:

Feb 21, 2013, 7:19:46 PM (12 years ago)

Author:

dino@apple.com

Message:

Followup commit for ​https://bugs.webkit.org/show_bug.cgi?id=110541
from a comment by Jon Lee.

• html/HTMLPlugInImageElement.cpp:

(WebCore::HTMLPlugInImageElement::subframeLoaderWillCreatePlugIn): Logic
in conditional was unnecessarily complex.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTMLPlugInImageElement.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-02-21  Dean Jackson  <dino@apple.com>
+
+        Followup commit for https://bugs.webkit.org/show_bug.cgi?id=110541
+        from a comment by Jon Lee.
+
+        * html/HTMLPlugInImageElement.cpp:
+        (WebCore::HTMLPlugInImageElement::subframeLoaderWillCreatePlugIn): Logic
+        in conditional was unnecessarily complex.
+
 2013-02-21  Ojan Vafai  <ojan@chromium.org>
 

trunk/Source/WebCore/html/HTMLPlugInImageElement.cpp

@@ -424 +424 @@
     int width = rect.width();
     int height = rect.height();
-    if (!width || !height || (width <= sizingTinyDimensionThreshold || height <= sizingTinyDimensionThreshold)) {
+    if (width <= sizingTinyDimensionThreshold || height <= sizingTinyDimensionThreshold) {
         LOG(Plugins, "%p Plug-in is %dx%d, set to play", this, width, height);
         return;