[Chromium] Move WebKitPlatformSupport declaration to Platform.h
Created attachment 189160 [details] Patch
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 on attachment 189160 [details] Patch whoa.
Comment on attachment 189160 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=189160&action=review > Source/Platform/chromium/public/Platform.h:491 > +class WebKitPlatformSupport : public Platform { > +protected: > + ~WebKitPlatformSupport() { } > +}; We should change all consumers to use the name WebKit::Platform instead. :)
(In reply to comment #4) > We should change all consumers to use the name WebKit::Platform instead. :) It's on my list. I want this change first, then update all the #includes downstream ( https://codereview.chromium.org/11824036/ ), then we can remove the entire WK/chromium/public/platform directory.
Comment on attachment 189160 [details] Patch Clearing flags on attachment: 189160 Committed r143401: <http://trac.webkit.org/changeset/143401>
All reviewed patches have been landed. Closing bug.
This was reverted along with r143382 for reason: Breaks Chromium win/mac canary compilation Committed r143403: <http://trac.webkit.org/changeset/143403>
Build breakage was fixed, rolled DEPS, let's give this another shot.
Comment on attachment 189160 [details] Patch Clearing flags on attachment: 189160 Committed r143496: <http://trac.webkit.org/changeset/143496>