[Chromium] Move idbFactory to Platform.h

[Mark Pilgrim (Google)]

[Chromium] Move idbFactory to Platform.h

[Mark Pilgrim (Google)]

Created attachment 140448 [details]
Patch

[Kentaro Hara]

Comment on attachment 140448 [details]
Patch

Looks OK.

[WebKit Review Bot]

Please wait for approval from [e-mail], [e-mail], [e-mail], [e-mail] or [e-mail] before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.

[Kentaro Hara]

Comment on attachment 140448 [details]
Patch

Removing r+ since it requires approval from API reviewers.

[Build Bot]

Comment on attachment 140448 [details]
Patch

Attachment 140448 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/12631904

[Mark Pilgrim (Google)]

EWS bot result appears to be flake. Not sure why this patch would cause a segmentation fault.

[Adam Barth]

Comment on attachment 140448 [details]
Patch

WebIDBFactory is a bit trickier.  Specifically, line 58 of WebKitPlatformSupport.h says:

class WebIDBFactory; // FIXME: Does this belong in platform?

The trouble is that WebIDBFactory::getDatabaseNames depends on WebFrame.  Source/Platform isn't allowed to know about Frames or WebFrames, so there's a bit of untangling to do.

[Mark Pilgrim (Google)]

Created attachment 179999 [details]
WIP Patch

[Mark Pilgrim (Google)]

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