Changeset 71030 in webkit

Timestamp:

Nov 1, 2010 8:50:19 AM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2010-11-01 Søren Gjesse <​sgjesse@chromium.org>

Reviewed by Andreas Kling.

Fix warning when compiling the chromium port of DumpRenderShell
with clang.
​https://bugs.webkit.org/show_bug.cgi?id=48414

• DumpRenderTree/chromium/TestShell.h: (TestShell::javaScriptFlagsForLoad):

Location:

trunk/WebKitTools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/chromium/TestShell.h (modified) (1 diff)

trunk/WebKitTools/ChangeLog

@@ +1 @@
+2010-11-01  Søren Gjesse  <sgjesse@chromium.org>
+
+        Reviewed by Andreas Kling.
+
+        Fix warning when compiling the chromium port of DumpRenderShell
+        with clang.
+        https://bugs.webkit.org/show_bug.cgi?id=48414
+
+        * DumpRenderTree/chromium/TestShell.h:
+        (TestShell::javaScriptFlagsForLoad):
+
 2010-11-01  Adam Roben  <aroben@apple.com>
 

trunk/WebKitTools/DumpRenderTree/chromium/TestShell.h

@@ -145 +145 @@
     // which can contain multiple flags (e.g. "--xxx --yyy"). With multiple load testing it is possible to specify
     // separate sets of flags to each load.
-    std::string javaScriptFlagsForLoad(size_t load) { return (load >= 0 && load < m_javaScriptFlags.size()) ? m_javaScriptFlags[load] : ""; }
+    std::string javaScriptFlagsForLoad(size_t load) { return (load < m_javaScriptFlags.size()) ? m_javaScriptFlags[load] : ""; }
     void setJavaScriptFlags(Vector<std::string> javaScriptFlags) { m_javaScriptFlags = javaScriptFlags; }