Bug 91546 - Implement putIndexKeys in WebIDBObjectStoreImpl
Summary: Implement putIndexKeys in WebIDBObjectStoreImpl
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: Alec Flett
URL:
Keywords:
Depends on:
Blocks: 91123
  Show dependency tree / graph
 
Reported: 2012-07-17 15:12 PDT by Alec Flett
Modified: 2012-07-18 12:27 PDT (History)
3 users (show)

See Also:

Attachments
Patch (3.91 KB, patch)
2012-07-17 15:13 PDT, Alec Flett 		no flags Details | Formatted Diff | Diff
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 Alec Flett 2012-07-17 15:12:21 PDT 
Implement putIndexKeys in WebIDBObjectStoreImpl
Comment 1 Alec Flett 2012-07-17 15:13:11 PDT 
Created attachment 152842 [details]
Patch
Comment 2 Alec Flett 2012-07-17 15:14:11 PDT 
Ugh, I managed to break up my patches incorrectly. This has to land before the chrome side.
Comment 3 Darin Fisher (:fishd, Google) 2012-07-18 11:04:56 PDT 
Comment on attachment 152842 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=152842&action=review

> Source/WebKit/chromium/src/WebIDBObjectStoreImpl.cpp:72
> +    for (size_t i = 0; i < webIndexNames.size(); ++i) {

Perhaps WebVector<T> should have a helper method for converting to a WTF::Vector.
Comment 4 WebKit Review Bot 2012-07-18 12:27:08 PDT 
Comment on attachment 152842 [details]
Patch

Clearing flags on attachment: 152842

Committed r122999: <http://trac.webkit.org/changeset/122999>
Comment 5 WebKit Review Bot 2012-07-18 12:27:13 PDT 
All reviewed patches have been landed.  Closing bug.