Title: [95154] trunk/Source/WebKit2
Revision
95154
Author
commit-qu...@webkit.org
Date
2011-09-14 19:53:13 -0700 (Wed, 14 Sep 2011)

Log Message

Eliminate WebKit2 compilation warnings.
https://bugs.webkit.org/show_bug.cgi?id=65025

Warnings found with gcc on linux.

Patch by Wajahat Siddiqui <mdwajahatali.siddi...@motorola.com> on 2011-09-14
Reviewed by Sam Weinig.

* Platform/CoreIPC/ArgumentEncoder.cpp:
(CoreIPC::ArgumentEncoder::~ArgumentEncoder):
* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::createPlugin):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (95153 => 95154)


--- trunk/Source/WebKit2/ChangeLog	2011-09-15 02:40:46 UTC (rev 95153)
+++ trunk/Source/WebKit2/ChangeLog	2011-09-15 02:53:13 UTC (rev 95154)
@@ -1,3 +1,17 @@
+2011-09-14  Wajahat Siddiqui  <mdwajahatali.siddi...@motorola.com>
+
+        Eliminate WebKit2 compilation warnings.
+        https://bugs.webkit.org/show_bug.cgi?id=65025
+
+        Warnings found with gcc on linux.
+
+        Reviewed by Sam Weinig.
+
+        * Platform/CoreIPC/ArgumentEncoder.cpp:
+        (CoreIPC::ArgumentEncoder::~ArgumentEncoder):
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebKit::WebFrameLoaderClient::createPlugin):
+
 2011-09-14  Ada Chan  <adac...@apple.com>
 
         Implement WKBundleFrameCopyWebArchive().

Modified: trunk/Source/WebKit2/Platform/CoreIPC/ArgumentEncoder.cpp (95153 => 95154)


--- trunk/Source/WebKit2/Platform/CoreIPC/ArgumentEncoder.cpp	2011-09-15 02:40:46 UTC (rev 95153)
+++ trunk/Source/WebKit2/Platform/CoreIPC/ArgumentEncoder.cpp	2011-09-15 02:53:13 UTC (rev 95154)
@@ -54,7 +54,7 @@
 #if !USE(UNIX_DOMAIN_SOCKETS)
     // FIXME: We need to dispose of the attachments in cases of failure.
 #else
-    for (int i = 0; i < m_attachments.size(); ++i)
+    for (size_t i = 0; i < m_attachments.size(); ++i)
         m_attachments[i].dispose();
 #endif
 }

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (95153 => 95154)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2011-09-15 02:40:46 UTC (rev 95153)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2011-09-15 02:53:13 UTC (rev 95154)
@@ -1266,7 +1266,7 @@
         // Currently we don't support transparency and windowed mode.
         // Inject wmode=opaque to make Flash work in these conditions.
         size_t wmodeIndex = parameters.names.find("wmode");
-        if (wmodeIndex == -1) {
+        if (wmodeIndex == notFound) {
             parameters.names.append("wmode");
             parameters.values.append("opaque");
         } else if (equalIgnoringCase(parameters.values[wmodeIndex], "window"))
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to