Closed
Bug 619182
Opened 14 years ago
Closed 14 years ago
Convert moz_indexedDB to mozIndexedDB
Categories
(Core :: Storage: IndexedDB, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | - |
People
(Reporter: dougt, Assigned: dougt)
Details
(Keywords: dev-doc-complete)
Attachments
(2 files, 1 obsolete file)
1.20 KB,
patch
|
sicking
:
review+
sicking
:
approval2.0+
|
Details | Diff | Splinter Review |
42.05 KB,
patch
|
sicking
:
review+
sicking
:
approval2.0+
|
Details | Diff | Splinter Review |
most (all?) mozilla specific objects in content are prefixed by moz not moz_. So,
moz_indexedDB really must be mozIndexedDB.
Comment 1•14 years ago
|
||
I agree, but Tantek seems to have other opinion about vendor prefixing
https://wiki.mozilla.org/Tantek-Mozilla-projects#DOM_API_vendor_prefixing
Assignee | ||
Comment 2•14 years ago
|
||
standardization of non-standard implementation object names is interesting, but I am not sure what sort of long term value there is.
I do value the consistency with all other non-standard object names we have used.
Assignee | ||
Comment 3•14 years ago
|
||
Attachment #497631 -
Flags: review?(jonas)
Assignee | ||
Comment 4•14 years ago
|
||
Attachment #497632 -
Flags: review?(jonas)
Assignee | ||
Comment 5•14 years ago
|
||
pushing to try.
Assignee | ||
Comment 6•14 years ago
|
||
ben, i can update your patch queue, or update m-c after you land.
Assignee | ||
Updated•14 years ago
|
Attachment #497632 -
Attachment is patch: true
Attachment #497632 -
Attachment mime type: application/octet-stream → text/plain
Assignee | ||
Comment 7•14 years ago
|
||
Attachment #497632 -
Attachment is obsolete: true
Attachment #497883 -
Flags: review?(jonas)
Attachment #497632 -
Flags: review?(jonas)
Attachment #497631 -
Flags: review?(jonas) → review+
Attachment #497883 -
Flags: review?(jonas) → review+
Assignee | ||
Comment 9•14 years ago
|
||
s/review/review+approve.
Attachment #497883 -
Flags: approval2.0+
Attachment #497631 -
Flags: approval2.0+
blocking2.0: ? → -
Assignee | ||
Comment 10•14 years ago
|
||
Comment 11•14 years ago
|
||
Docs refer to this as mozIndexedDB, so doc complete on this one.
Keywords: dev-doc-needed → dev-doc-complete
Component: DOM → DOM: IndexedDB
You need to log in
before you can comment on or make changes to this bug.
Description
•