Bug 59859 - V8Proxy.h shouldn't include SecurityOrigin.h
Summary: V8Proxy.h shouldn't include SecurityOrigin.h
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Mihai Parparita
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-29 18:49 PDT by Mihai Parparita
Modified: 2011-04-30 02:25 PDT (History)
6 users (show)

See Also:


Attachments
Patch (8.21 KB, patch)
2011-04-29 18:52 PDT, Mihai Parparita
no flags Details | Formatted Diff | Diff
Patch for landing (8.64 KB, patch)
2011-04-30 00:21 PDT, Mihai Parparita
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mihai Parparita 2011-04-29 18:49:42 PDT
V8Proxy.h shouldn't include SecurityOrigin.h
Comment 1 Mihai Parparita 2011-04-29 18:52:20 PDT
Created attachment 91790 [details]
Patch
Comment 2 Eric Seidel (no email) 2011-04-29 18:55:37 PDT
Comment on attachment 91790 [details]
Patch

OK.
Comment 3 WebKit Review Bot 2011-04-29 19:12:28 PDT
Attachment 91790 [details] did not build on chromium:
Build output: http://queues.webkit.org/results/8526090
Comment 4 WebKit Review Bot 2011-04-29 21:06:22 PDT
Attachment 91790 [details] did not build on chromium:
Build output: http://queues.webkit.org/results/8526111
Comment 5 Mihai Parparita 2011-04-30 00:21:54 PDT
Created attachment 91799 [details]
Patch for landing
Comment 6 Mihai Parparita 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.
Comment 7 WebKit Commit Bot 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>
Comment 8 WebKit Commit Bot 2011-04-30 02:25:14 PDT
All reviewed patches have been landed.  Closing bug.