Bug 48414

Summary: Warning when compiling the chromium port of DumpRenderShell with clang
Product: WebKit Reporter: Søren Gjesse <sgjesse>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Severity: Trivial CC: commit-queue, ctruta, hans
Priority: P5    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
Simple patch
Patch none

Description Søren Gjesse 2010-10-27 04:49:44 PDT
Created attachment 72010 [details]
Simple patch

Method javaScriptFlagsForLoad in TestShell.h compares a size_t argument >= 0 which causes a warning in clang
Comment 1 Hans Wennborg 2010-10-27 07:08:23 PDT
Looks good to me, but I am not a reviewer.
Comment 2 Hans Wennborg 2010-10-27 07:11:15 PDT
Looking closer, shouldn't the ChangeLog contain a link to this bug entry? I thought the tools put that in automatically...
Comment 3 Søren Gjesse 2010-10-29 02:38:08 PDT
Created attachment 72304 [details]
Comment 4 Andreas Kling 2010-10-29 06:35:00 PDT
Comment on attachment 72304 [details]

Comment 5 WebKit Commit Bot 2010-11-01 08:49:44 PDT
The commit-queue encountered the following flaky tests while processing attachment 72304 [details]:


Please file bugs against the tests.  These tests were authored by koivisto@iki.fi.  The commit-queue is continuing to process your patch.
Comment 6 WebKit Commit Bot 2010-11-01 08:50:29 PDT
Comment on attachment 72304 [details]

Clearing flags on attachment: 72304

Committed r71030: <http://trac.webkit.org/changeset/71030>
Comment 7 WebKit Commit Bot 2010-11-01 08:50:34 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Cosmin Truta 2010-11-02 14:38:57 PDT
*** Bug 48851 has been marked as a duplicate of this bug. ***