RESOLVED FIXED 73065
[Chromium] Remove unused WebSecurityOrigin::isEmpty from WebKit API 
https://bugs.webkit.org/show_bug.cgi?id=73065
Summary [Chromium] Remove unused WebSecurityOrigin::isEmpty from WebKit API
Adam Barth
Reported 2011-11-23 18:34:09 PST
[Chromium] Remove unused WebSecurityOrigin::isEmpty from WebKit API
Attachments
Patch (2.04 KB, patch)
2011-11-23 18:34 PST, Adam Barth 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2011-11-23 18:34:43 PST
Created attachment 116465 [details] Patch
Eric Seidel (no email)
Comment 2 2011-11-23 21:56:16 PST
Comment on attachment 116465 [details] Patch LGTM. You may want fishd's approval as well.
WebKit Review Bot
Comment 3 2011-11-23 21:57:50 PST
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
Adam Barth
Comment 4 2011-11-23 22:09:32 PST
Comment on attachment 116465 [details] Patch He reviewed the original patch with the FIXME. I'm not that worried about it.
Eric Seidel (no email)
Comment 5 2011-11-23 22:19:49 PST
Oh reviewbot, you're so smart.
Darin Fisher (:fishd, Google)
Comment 6 2011-11-23 22:57:50 PST
Yes, R+
WebKit Review Bot
Comment 7 2011-11-23 23:03:17 PST
Comment on attachment 116465 [details] Patch Clearing flags on attachment: 116465 Committed r101121: <http://trac.webkit.org/changeset/101121>
WebKit Review Bot
Comment 8 2011-11-23 23:03:22 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.