Bug 106227 - [Chromium] Update some #includes in WebKit/chromium/public/ for new Platform directory
Summary: [Chromium] Update some #includes in WebKit/chromium/public/ for new Platform ...
Status: RESOLVED DUPLICATE of bug 107741
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Pilgrim (Google)
URL:
Keywords:
Depends on:
Blocks: 82948
  Show dependency tree / graph
 
Reported: 2013-01-07 09:34 PST by Mark Pilgrim (Google)
Modified: 2013-01-23 14:49 PST (History)
8 users (show)

See Also:

Attachments
WIP Patch (85.98 KB, patch)
2013-01-07 09:43 PST, Mark Pilgrim (Google) 		no flags Details | Formatted Diff | Diff
WIP Patch (89.68 KB, patch)
2013-01-07 10:13 PST, Mark Pilgrim (Google) 		no flags Details | Formatted Diff | Diff
Patch (80.04 KB, patch)
2013-01-09 11:20 PST, Mark Pilgrim (Google) 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Pilgrim (Google) 2013-01-07 09:34:23 PST 
[Chromium] Update some #includes in WebKit/chromium/public/ for new Platform directory
Comment 1 Mark Pilgrim (Google) 2013-01-07 09:43:19 PST 
Created attachment 181513 [details]
WIP Patch
Comment 2 Mark Pilgrim (Google) 2013-01-07 10:13:34 PST 
Created attachment 181519 [details]
WIP Patch
Comment 3 Mark Pilgrim (Google) 2013-01-09 11:20:00 PST 
Created attachment 181955 [details]
Patch
Comment 4 Mark Pilgrim (Google) 2013-01-09 11:21:09 PST 
Comment on attachment 181955 [details]
Patch

Shouldn't break Chromium compilation downstream, now that https://codereview.chromium.org/11748018/ has landed.
Comment 5 WebKit Review Bot 2013-01-09 11:21:37 PST 
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 6 WebKit Review Bot 2013-01-09 12:45:46 PST 
Comment on attachment 181955 [details]
Patch

Clearing flags on attachment: 181955

Committed r139227: <http://trac.webkit.org/changeset/139227>
Comment 7 WebKit Review Bot 2013-01-09 12:45:50 PST 
All reviewed patches have been landed.  Closing bug.
Comment 8 Dimitri Glazkov (Google) 2013-01-10 08:43:56 PST 
Did this break the Win builders? http://build.chromium.org/p/chromium.webkit/waterfall?builder=Win+Builder
Comment 9 Dimitri Glazkov (Google) 2013-01-10 08:52:37 PST 
Also http://build.chromium.org/p/chromium.webkit/waterfall?builder=Win7+(dbg)
Comment 10 Dimitri Glazkov (Google) 2013-01-10 09:28:47 PST 
Reverted r139227 for reason:

Broke Win component build.

Committed r139323: <http://trac.webkit.org/changeset/139323>
Comment 11 Mark Pilgrim (Google) 2013-01-23 14:49:19 PST 

*** This bug has been marked as a duplicate of bug 107741 ***