2011-02-01 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Feb 2011 05:13:39 +0000 (05:13 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Feb 2011 05:13:39 +0000 (05:13 +0000)
        Reviewed by Andreas Kling.

        Remove useless comment
        https://bugs.webkit.org/show_bug.cgi?id=53549

        The reason for this parameter is captured in
        plugins/netscape-plugin-setwindow-size.html, which is a better place to
        capture it than in this comment (which otherwise just re-iterates the
        name of the parameter).

        * html/HTMLPlugInImageElement.cpp:
        (WebCore::HTMLPlugInImageElement::updateWidgetIfNecessary):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@77361 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLPlugInImageElement.cpp

index bcd22c4..a0bc296 100644 (file)
@@ -1,3 +1,18 @@
+2011-02-01  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Andreas Kling.
+
+        Remove useless comment
+        https://bugs.webkit.org/show_bug.cgi?id=53549
+
+        The reason for this parameter is captured in
+        plugins/netscape-plugin-setwindow-size.html, which is a better place to
+        capture it than in this comment (which otherwise just re-iterates the
+        name of the parameter).
+
+        * html/HTMLPlugInImageElement.cpp:
+        (WebCore::HTMLPlugInImageElement::updateWidgetIfNecessary):
+
 2011-02-01  James Simonsen  <simonjam@chromium.org>
 
         Reviewed by Tony Gentilcore.
index e34cd91..1c91c05 100644 (file)
@@ -166,7 +166,6 @@ void HTMLPlugInImageElement::updateWidgetIfNecessary()
     if (!renderEmbeddedObject() || renderEmbeddedObject()->pluginCrashedOrWasMissing())
         return;
 
-    // True indicates that this code path should only create non-netscape plugins (no clue why).
     updateWidget(true);
 }