Title: [143684] trunk/Source/WebCore
Revision
143684
Author
d...@apple.com
Date
2013-02-21 19:19:46 -0800 (Thu, 21 Feb 2013)

Log 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.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (143683 => 143684)


--- trunk/Source/WebCore/ChangeLog	2013-02-22 03:12:04 UTC (rev 143683)
+++ trunk/Source/WebCore/ChangeLog	2013-02-22 03:19:46 UTC (rev 143684)
@@ -1,3 +1,12 @@
+2013-02-21  Dean Jackson  <d...@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  <o...@chromium.org>
 
         Clean up computePreferredLogicalWidths functions in TableLayout subclasses

Modified: trunk/Source/WebCore/html/HTMLPlugInImageElement.cpp (143683 => 143684)


--- trunk/Source/WebCore/html/HTMLPlugInImageElement.cpp	2013-02-22 03:12:04 UTC (rev 143683)
+++ trunk/Source/WebCore/html/HTMLPlugInImageElement.cpp	2013-02-22 03:19:46 UTC (rev 143684)
@@ -423,7 +423,7 @@
     LayoutRect rect = toRenderEmbeddedObject(renderer())->contentBoxRect();
     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;
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to