RESOLVED FIXED 59859
V8Proxy.h shouldn't include SecurityOrigin.h 
https://bugs.webkit.org/show_bug.cgi?id=59859
Summary V8Proxy.h shouldn't include SecurityOrigin.h
Mihai Parparita
Reported 2011-04-29 18:49:42 PDT
V8Proxy.h shouldn't include SecurityOrigin.h
Attachments
Patch (8.21 KB, patch)
2011-04-29 18:52 PDT, Mihai Parparita 		no flags
DetailsFormatted DiffDiff
Patch for landing (8.64 KB, patch)
2011-04-30 00:21 PDT, Mihai Parparita 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mihai Parparita
Comment 1 2011-04-29 18:52:20 PDT
Created attachment 91790 [details] Patch
Eric Seidel (no email)
Comment 2 2011-04-29 18:55:37 PDT
Comment on attachment 91790 [details] Patch OK.
WebKit Review Bot
Comment 3 2011-04-29 19:12:28 PDT
Attachment 91790 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8526090
WebKit Review Bot
Comment 4 2011-04-29 21:06:22 PDT
Attachment 91790 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8526111
Mihai Parparita
Comment 5 2011-04-30 00:21:54 PDT
Created attachment 91799 [details] Patch for landing
Mihai Parparita
Comment 6 2011-04-30 00:22:31 PDT
(In reply to comment #4) > Attachment 91790 [details] did not build on chromium: > Build output: http://queues.webkit.org/results/8526111 Odd that this only happened with release builds. Should be OK now.
WebKit Commit Bot
Comment 7 2011-04-30 02:25:09 PDT
Comment on attachment 91799 [details] Patch for landing Clearing flags on attachment: 91799 Committed r85395: <http://trac.webkit.org/changeset/85395>
WebKit Commit Bot
Comment 8 2011-04-30 02:25:14 PDT
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.