Bug 106654

Summary: [chromium] move inspector related methods to TestRunner library
Product: WebKit Reporter: jochen
Component: New BugsAssignee: jochen
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, fishd, jamesr, tkent+wkapi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 91308    
Attachments:
Description Flags
Patch
none
Patch for landing none

Description jochen 2013-01-11 04:20:14 PST
[chromium] move inspector related methods to TestRunner library
Comment 1 jochen 2013-01-11 04:20:35 PST
Created attachment 182318 [details]
Patch
Comment 2 WebKit Review Bot 2013-01-11 04:23:21 PST
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 3 Kentaro Hara 2013-01-11 04:23:41 PST
Comment on attachment 182318 [details]
Patch

ok
Comment 4 jochen 2013-01-11 04:26:02 PST
Comment on attachment 182318 [details]
Patch

setting to back to r? to wait for an API review
Comment 5 Adam Barth 2013-01-11 10:17:54 PST
Comment on attachment 182318 [details]
Patch

LGTM2
Comment 6 WebKit Review Bot 2013-01-11 11:08:17 PST
Comment on attachment 182318 [details]
Patch

Rejecting attachment 182318 [details] from commit-queue.

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', u'--status-host=queues.webkit.org', ..." exit_code: 2 cwd: /mnt/git/webkit-commit-queue

Last 500 characters of output:
ls/DumpRenderTree/chromium/TestRunner/src/TestRunner.h
Hunk #1 succeeded at 289 (offset 8 lines).
patching file Tools/DumpRenderTree/chromium/WebViewHost.cpp
patching file Tools/DumpRenderTree/chromium/WebViewHost.h
Hunk #1 FAILED at 150.
1 out of 1 hunk FAILED -- saving rejects to file Tools/DumpRenderTree/chromium/WebViewHost.h.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Adam Barth']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Full output: http://queues.webkit.org/results/15811385
Comment 7 jochen 2013-01-11 12:12:49 PST
Created attachment 182389 [details]
Patch for landing
Comment 8 WebKit Review Bot 2013-01-11 13:22:47 PST
Comment on attachment 182389 [details]
Patch for landing

Rejecting attachment 182389 [details] from commit-queue.

New failing tests:
inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html
Full output: http://queues.webkit.org/results/15815286
Comment 9 WebKit Review Bot 2013-01-12 05:58:52 PST
Comment on attachment 182389 [details]
Patch for landing

Clearing flags on attachment: 182389

Committed r139549: <http://trac.webkit.org/changeset/139549>
Comment 10 WebKit Review Bot 2013-01-12 05:58:56 PST
All reviewed patches have been landed.  Closing bug.