Bug 75373 - [Chromium] Remove WebWorkerClient.h
Summary: [Chromium] Remove WebWorkerClient.h
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dmitry Lomov
URL:
Keywords:
Depends on: 75377
Blocks:
  Show dependency tree / graph
 
Reported: 2011-12-29 16:43 PST by Dmitry Lomov
Modified: 2011-12-30 00:44 PST (History)
3 users (show)

See Also:

Attachments
Fix (5.17 KB, patch)
2011-12-29 16:46 PST, Dmitry Lomov 		no flags Details | Formatted Diff | Diff
Relanding fix (5.14 KB, patch)
2011-12-29 23:18 PST, Dmitry Lomov 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Dmitry Lomov 2011-12-29 16:43:26 PST 
After coordinated patch in chromium, WebWorkerClient alias for WebSharedWorkerClient is no longer needed.
Comment 1 Dmitry Lomov 2011-12-29 16:46:12 PST 
Created attachment 120773 [details]
Fix
Comment 2 WebKit Review Bot 2011-12-29 16:48:40 PST 
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
Comment 3 Adam Barth 2011-12-29 16:50:54 PST 
Comment on attachment 120773 [details]
Fix

fishd is out of the office, but this looks pretty safe to r+.  :)
Comment 4 WebKit Review Bot 2011-12-29 18:13:34 PST 
Comment on attachment 120773 [details]
Fix

Clearing flags on attachment: 120773

Committed r103831: <http://trac.webkit.org/changeset/103831>
Comment 5 WebKit Review Bot 2011-12-29 18:13:44 PST 
All reviewed patches have been landed.  Closing bug.
Comment 6 Dmitry Lomov 2011-12-29 23:05:42 PST 
Broke chromium build.
Comment 7 Dmitry Lomov 2011-12-29 23:08:57 PST 
Comment on attachment 120773 [details]
Fix

Chromium side fix landed in http://src.chromium.org/viewvc/chrome?view=rev&revision=116031
Comment 8 Dmitry Lomov 2011-12-29 23:18:26 PST 
Created attachment 120783 [details]
Relanding fix

http://src.chromium.org/viewvc/chrome?view=rev&revision=116031 is a fix for breakage on chromium side.
Comment 9 WebKit Review Bot 2011-12-30 00:44:03 PST 
Comment on attachment 120783 [details]
Relanding fix

Clearing flags on attachment: 120783

Committed r103837: <http://trac.webkit.org/changeset/103837>
Comment 10 WebKit Review Bot 2011-12-30 00:44:09 PST 
All reviewed patches have been landed.  Closing bug.