Bug 126616

Summary: Web Inspector: Remove InspectorClient::captureScreenshot()
Product: WebKit Reporter: Seokju Kwon <seokju>
Component: Web InspectorAssignee: Seokju Kwon <seokju>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Seokju Kwon
Reported 2014-01-07 21:15:25 PST
Remove leftover mothod as Page.captureScreenshot API was removed in r160202.
Attachments
Patch (1.51 KB, patch)
2014-01-07 21:17 PST, Seokju Kwon 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2014-01-07 21:16:05 PST
<rdar://problem/15769676>
Seokju Kwon
Comment 2 2014-01-07 21:17:01 PST
Created attachment 220591 [details] Patch
Timothy Hatcher
Comment 3 2014-01-07 21:27:26 PST
Comment on attachment 220591 [details] Patch I'm surprised we removed captureScreenshot. Brian Burg was implementing it.
Joseph Pecoraro
Comment 4 2014-01-08 10:09:36 PST
Comment on attachment 220591 [details] Patch r=me This was the port hook to make the port capture a screenshot. Now it is all done in WebCore. This is just a stale line of code from the old, removed, command. The new "snapshot*" commands still exist.
WebKit Commit Bot
Comment 5 2014-01-08 18:08:42 PST
Comment on attachment 220591 [details] Patch Clearing flags on attachment: 220591 Committed r161534: <http://trac.webkit.org/changeset/161534>
WebKit Commit Bot
Comment 6 2014-01-08 18:08:45 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.