Bug 127717

Summary: IDB: Rename "IDBTransactionIdentifier" to be more general purpose
Product: WebKit Reporter: Brady Eidson <beidson>
Component: WebKit2Assignee: Brady Eidson <beidson>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 124521    
Attachments:
Description Flags
Patch v1
thorton: review+
Patch take 2 (v1) thorton: review+

Description Brady Eidson 2014-01-27 15:38:08 PST
IDB: Rename "IDBTransactionIdentifier" to be more general purpose

The purpose of IDBTransactionIdentifier is to pair a DatabaseProcessIDBConnection with an integer ID for unique hashing.

Previously we only cared about transaction integer IDs but now we'll need to use for cursor integer IDs also.

My proposed rename is to "IDBIdentifier"
Comment 1 Brady Eidson 2014-01-27 15:40:43 PST
Created attachment 222371 [details]
Patch v1
Comment 2 Brady Eidson 2014-01-27 15:44:28 PST
http://trac.webkit.org/changeset/162877
Comment 3 Brady Eidson 2014-01-27 15:54:40 PST
This was incomplete.  Since the type name no longer carries all the information, variable names will need to be updated as well.
Comment 4 Brady Eidson 2014-01-27 16:00:44 PST
Created attachment 222376 [details]
Patch take 2 (v1)
Comment 5 Brady Eidson 2014-01-27 17:13:40 PST
Followup landed in http://trac.webkit.org/changeset/162891