Modern IDB: Pipe through cursor functions from client to server The result of all requests will always be an error for now.
(In reply to comment #0) > Modern IDB: Pipe through cursor functions from client to server > > The result of all requests will always be an error for now. Until https://bugs.webkit.org/show_bug.cgi?id=151196 is resolved
Created attachment 265437 [details] Patch v1
Comment on attachment 265437 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=265437&action=review > Source/WebCore/Modules/indexeddb/client/IDBCursorImpl.cpp:49 > + , m_source(adoptRef(*IDBAny::create(objectStore).leakRef())) can you use wtf::move here, or just *IDBAny::create(objectStore), or make IDBAny::create return a Ref? > Source/WebCore/Modules/indexeddb/client/IDBCursorImpl.cpp:56 > + , m_source(adoptRef(*IDBAny::create(index).leakRef())) same.
(In reply to comment #3) > Comment on attachment 265437 [details] > Patch v1 > > View in context: > https://bugs.webkit.org/attachment.cgi?id=265437&action=review > > > Source/WebCore/Modules/indexeddb/client/IDBCursorImpl.cpp:49 > > + , m_source(adoptRef(*IDBAny::create(objectStore).leakRef())) > > can you use wtf::move here, or just *IDBAny::create(objectStore), or make > IDBAny::create return a Ref? The first two, no. The third, yes. > > > Source/WebCore/Modules/indexeddb/client/IDBCursorImpl.cpp:56 > > + , m_source(adoptRef(*IDBAny::create(index).leakRef())) > > same. Same.
Created attachment 265441 [details] Patch for landing
Comment on attachment 265441 [details] Patch for landing Clearing flags on attachment: 265441 Committed r192396: <http://trac.webkit.org/changeset/192396>
All reviewed patches have been landed. Closing bug.