The default bug view has changed. See this FAQ.

Add various console warnings

RESOLVED FIXED in mozilla9

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

({dev-doc-complete})

Trunk
mozilla9
dev-doc-complete
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 561498 [details] [diff] [review]
Part a: Allow using nsIDocument::WarnOnceAbout for more cases
Attachment #561498 - Flags: review?(jonas)
(Assignee)

Comment 2

6 years ago
Created attachment 561500 [details] [diff] [review]
Part b: Add various warnings
Attachment #561500 - Flags: review?(jonas)
(Assignee)

Comment 3

6 years ago
Jonas, if you can't review this tomorrow, please reassign the r?.
(Assignee)

Updated

6 years ago
Attachment #561498 - Flags: review?(jonas) → review?(bzbarsky)
(Assignee)

Updated

6 years ago
Attachment #561500 - Flags: review?(jonas) → review?(bzbarsky)
Comment on attachment 561498 [details] [diff] [review]
Part a: Allow using nsIDocument::WarnOnceAbout for more cases

r=me
Attachment #561498 - Flags: review?(bzbarsky) → review+
Comment on attachment 561500 [details] [diff] [review]
Part b: Add various warnings

r=me assuming these are in fact all deprecated..
Attachment #561500 - Flags: review?(bzbarsky) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/da0d9bc38cad
https://hg.mozilla.org/integration/mozilla-inbound/rev/58d2b33f0e0d
Keywords: checkin-needed
backed out since it didn't compile
(Assignee)

Comment 8

6 years ago
Created attachment 562692 [details] [diff] [review]
Part b: Add various warnings (v2)

Sorry about that, I was sure I fixed it.
Attachment #561500 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
I did fix that locally too and ready to be pushed.
Keywords: checkin-needed
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/19f8b8150ee4
https://hg.mozilla.org/mozilla-central/rev/5a6768dfca35
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 11

6 years ago
Thanks a lot, guys!
(Assignee)

Updated

6 years ago
Whiteboard: [inbound]

Comment 12

5 years ago
I added a "deprecated" warning for document.inputEncoding to the MDC doc.
Is there actually a plan/bug to drop support for it?
Keywords: dev-doc-complete
Depends on: 966385
You need to log in before you can comment on or make changes to this bug.