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