Bug 124068

Summary:

Merge IDBCursorBackendInterface and IDBCursorBackendImpl

Product:

WebKit

Reporter:

Brady Eidson <beidson>

Component:

WebCore Misc.

Assignee:

Brady Eidson <beidson>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

alecflett, commit-queue, gyuyoung.kim, jsbell, rakuco

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch v1	andersca: review+
Updated patch for EWS	commit-queue: commit-queue-

Description Brady Eidson 2013-11-08 13:41:57 PST

Merge IDBCursorBackendInterface and IDBCursorBackendImpl

No need for the abstraction anymore.

Comment 1 Brady Eidson 2013-11-08 14:10:26 PST

Created attachment 216437 [details]
Patch v1

Comment 2 Brady Eidson 2013-11-08 14:27:15 PST

Thanks for the review - I'll rebase and run the updated patch by EWS before landing

Comment 3 Brady Eidson 2013-11-08 14:28:23 PST

Created attachment 216440 [details]
Updated patch for EWS

Comment 4 WebKit Commit Bot 2013-11-08 14:36:12 PST

Comment on attachment 216440 [details]
Updated patch for EWS

Rejecting attachment 216440 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 216440, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/WebCore/ChangeLog contains OOPS!.

Full output: http://webkit-queues.appspot.com/results/22508620

Comment 5 Brady Eidson 2013-11-08 14:47:05 PST

http://trac.webkit.org/changeset/158959

Comment 6 Brady Eidson 2013-11-08 15:15:10 PST

Windows build fix followup in http://trac.webkit.org/changeset/158961