RESOLVED FIXED 90812
IndexedDB: Remove obsolete accessor plumbing 
https://bugs.webkit.org/show_bug.cgi?id=90812
Summary IndexedDB: Remove obsolete accessor plumbing
Joshua Bell
Reported 2012-07-09 12:30:05 PDT
IndexedDB: Remove obsolete accessor plumbing
Attachments
Patch (17.24 KB, patch)
2012-07-09 12:34 PDT, Joshua Bell 		no flags
DetailsFormatted DiffDiff
Patch (17.28 KB, patch)
2012-07-09 13:19 PDT, Joshua Bell 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Joshua Bell
Comment 1 2012-07-09 12:34:51 PDT
Created attachment 151299 [details] Patch
Joshua Bell
Comment 2 2012-07-09 12:35:40 PDT
alecflett@, dgrogan@ - please take a look tony@ - r?
WebKit Review Bot
Comment 3 2012-07-09 12:38:07 PDT
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.
WebKit Review Bot
Comment 4 2012-07-09 12:50:43 PDT
Comment on attachment 151299 [details] Patch Attachment 151299 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13166328
Joshua Bell
Comment 5 2012-07-09 13:19:53 PDT
Created attachment 151306 [details] Patch
Joshua Bell
Comment 6 2012-07-09 13:21:13 PDT
Whoops, let's try that again after building/running tests locally. tony@ - looks like you're off the hook.
David Grogan
Comment 7 2012-07-09 14:29:56 PDT
Comment on attachment 151306 [details] Patch LGTM
Joshua Bell
Comment 8 2012-07-09 16:40:14 PDT
Comment on attachment 151306 [details] Patch Since it's just code deletions from the Chromium public API I'm going to blithely cq+ w/o approval from fishd@ & co.
WebKit Review Bot
Comment 9 2012-07-09 17:41:35 PDT
Comment on attachment 151306 [details] Patch Clearing flags on attachment: 151306 Committed r122172: <http://trac.webkit.org/changeset/122172>
WebKit Review Bot
Comment 10 2012-07-09 17:41:40 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.