Convert moz_indexedDB to mozIndexedDB

RESOLVED FIXED

Status

()

RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

({dev-doc-complete})

unspecified
x86
Mac OS X
dev-doc-complete
Points:
---

Firefox Tracking Flags

(blocking2.0 -)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

8 years ago
most (all?) mozilla specific objects in content are prefixed by moz not moz_.  So,

moz_indexedDB really must be mozIndexedDB.
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

8 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

8 years ago
Created attachment 497631 [details] [diff] [review]
patch v.1 (core)
Attachment #497631 - Flags: review?(jonas)
(Assignee)

Comment 4

8 years ago
Created attachment 497632 [details] [diff] [review]
patch v.1 (tests)
Attachment #497632 - Flags: review?(jonas)
(Assignee)

Comment 5

8 years ago
pushing to try.
(Assignee)

Comment 6

8 years ago
ben, i can update your patch queue, or update m-c after you land.
(Assignee)

Updated

8 years ago
Attachment #497632 - Attachment is patch: true
Attachment #497632 - Attachment mime type: application/octet-stream → text/plain
(Assignee)

Comment 7

8 years ago
Created attachment 497883 [details] [diff] [review]
patch v.2 (tests)
Attachment #497632 - Attachment is obsolete: true
Attachment #497883 - Flags: review?(jonas)
Attachment #497632 - Flags: review?(jonas)
(Assignee)

Comment 8

8 years ago
jst, can you review?
blocking2.0: --- → ?
(Assignee)

Comment 9

8 years ago
s/review/review+approve.
(Assignee)

Comment 10

8 years ago
http://hg.mozilla.org/mozilla-central/rev/ca0f807691f9
http://hg.mozilla.org/mozilla-central/rev/0aff32dee6a7
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Keywords: dev-doc-needed
Resolution: --- → FIXED
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.