Modern IDB: More Encoder/Decoder/Messaging scaffolding for WK2 IPC
Created attachment 271584 [details] Patch v1
Comment on attachment 271584 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=271584&action=review > Source/WebCore/Modules/indexeddb/shared/IDBDatabaseInfo.h:62 > + WEBCORE_EXPORT IDBDatabaseInfo(); I think this should go with the other constructor declaration. > Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBConnectionToServer.messages.in:41 > + intentional space?
(In reply to comment #2) > Comment on attachment 271584 [details] > Patch v1 > > View in context: > https://bugs.webkit.org/attachment.cgi?id=271584&action=review > > > Source/WebCore/Modules/indexeddb/shared/IDBDatabaseInfo.h:62 > > + WEBCORE_EXPORT IDBDatabaseInfo(); > > I think this should go with the other constructor declaration. I've been trying to make it clear - or at least suggest - that these default constructors are only for the sake of encoder/decoder (They didn't exist before today) > > > Source/WebKit2/WebProcess/Databases/IndexedDB/WebIDBConnectionToServer.messages.in:41 > > + > > intentional space? Yes.
Created attachment 271595 [details] Patch for linux EWS
GTK is broken for other reasons... I'm landing now.
http://trac.webkit.org/changeset/196725