Bug 90812 - IndexedDB: Remove obsolete accessor plumbing
Summary: IndexedDB: Remove obsolete accessor plumbing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Joshua Bell
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-09 12:30 PDT by Joshua Bell
Modified: 2012-07-09 17:41 PDT (History)
9 users (show)

See Also:

Attachments
Patch (17.24 KB, patch)
2012-07-09 12:34 PDT, Joshua Bell 		no flags Details | Formatted Diff | Diff
Patch (17.28 KB, patch)
2012-07-09 13:19 PDT, Joshua Bell 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joshua Bell 2012-07-09 12:30:05 PDT 
IndexedDB: Remove obsolete accessor plumbing
Comment 1 Joshua Bell 2012-07-09 12:34:51 PDT 
Created attachment 151299 [details]
Patch
Comment 2 Joshua Bell 2012-07-09 12:35:40 PDT 
alecflett@, dgrogan@ - please take a look

tony@ - r?
Comment 3 WebKit Review Bot 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.
Comment 4 WebKit Review Bot 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
Comment 5 Joshua Bell 2012-07-09 13:19:53 PDT 
Created attachment 151306 [details]
Patch
Comment 6 Joshua Bell 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.
Comment 7 David Grogan 2012-07-09 14:29:56 PDT 
Comment on attachment 151306 [details]
Patch

LGTM
Comment 8 Joshua Bell 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.
Comment 9 WebKit Review Bot 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>
Comment 10 WebKit Review Bot 2012-07-09 17:41:40 PDT 
All reviewed patches have been landed.  Closing bug.