Bug 30677

Summary: Missing UnusedParam.h include when INSPECTOR is not enabled
Product: WebKit Reporter: Steve Block <steveblock>
Component: WebCore Misc.Assignee: Ben Murdoch <benm>
Status: RESOLVED FIXED    
Severity: Normal CC: benm, steveblock
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch 1 for Bug 30677 eric: review+, benm: commit-queue-

Description Steve Block 2009-10-22 09:07:45 PDT
WebCore/page/Console.cpp and WebCore/workers/WorkerContext.cpp both use the UNUSED_PARAM macro but do not include UnusedParam.h. The macro is only used when INSPECTOR is not enabled, so the build succeeds when INSPECTOR is enabled.
Comment 1 Steve Block 2009-10-22 09:39:24 PDT
Created attachment 41663 [details]
Patch 1 for Bug 30677
Comment 2 Eric Seidel (no email) 2009-10-22 10:40:21 PDT
Comment on attachment 41663 [details]
Patch 1 for Bug 30677

OK, looks OK.
Comment 3 WebKit Commit Bot 2009-10-23 02:57:33 PDT
Comment on attachment 41663 [details]
Patch 1 for Bug 30677

Rejecting patch 41663 from commit-queue.

Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1
Running build-dumprendertree
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 11506 test cases.
fast/media/mq-transform-02.html -> failed

Exiting early after 1 failures. 7441 tests run.
135.60s total testing time

7440 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
3 test cases (<1%) had stderr output
Comment 4 Ben Murdoch 2009-10-23 03:06:20 PDT
Will land manually, looks like https://bugs.webkit.org/show_bug.cgi?id=30700 caused the cq to fail.
Comment 5 Ben Murdoch 2009-10-23 05:56:22 PDT
Landed as r49976.
Comment 6 Eric Seidel (no email) 2009-10-23 12:03:30 PDT
Comment on attachment 41663 [details]
Patch 1 for Bug 30677

Sorry about the commit-queue trouble.  Should work fine now.
Comment 7 Steve Block 2009-10-23 12:09:36 PDT
benm@google.com has already landed this by hand.