RESOLVED FIXED 126616
Web Inspector: Remove InspectorClient::captureScreenshot() 
https://bugs.webkit.org/show_bug.cgi?id=126616
Summary Web Inspector: Remove InspectorClient::captureScreenshot()
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.