1589337 - Simplify nsIClassInfo impl in nsIURI objects

Status: RESOLVED FIXED

(Core :: Networking, task, P2)

Description

[Valentin Gosu [:valentin] (he/him)]

It seems that serializable nsIURI objects all implement nsIClassInfo in order to work with nsBinaryOutputStream

However nsIClassInfoImpl.h provides a much simpler way to implement this.

Comment 1

[anjali.jha1903]

Hi @valentin. I would ike to work on this issue. Can you please guide me a bit?

Comment 2

[Valentin Gosu [:valentin] (he/him)]

(In reply to anjali.jha1903 from comment #1)

Hi @valentin. I would ike to work on this issue. Can you please guide me a bit?

This issue is a bit tricky and I already have a patch for it.

Comment 3

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1589337 - Use nsIClassInfoImpl in DefaultURI r=#necko

Comment 4

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1589337 - Use nsIClassInfoImpl in nsIconURI r=#necko

Depends on D90254

Comment 5

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1589337 - Use nsIClassInfoImpl in nsJARURI r=#necko

Depends on D90255

Comment 6

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1589337 - Use nsIClassInfoImpl in nsSimpleURI and children r=#necko

Depends on D90256

Comment 7

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1589337 - Use nsIClassInfoImpl in nsStandardURL and children r=#necko

Depends on D90257

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b130f13ed5a9
Use nsIClassInfoImpl in DefaultURI r=necko-reviewers,xpcom-reviewers,kershaw,sg
https://hg.mozilla.org/integration/autoland/rev/0424bd36a7bf
Use nsIClassInfoImpl in nsIconURI r=necko-reviewers,kershaw
https://hg.mozilla.org/integration/autoland/rev/20f842b9132f
Use nsIClassInfoImpl in nsJARURI r=necko-reviewers,kershaw
https://hg.mozilla.org/integration/autoland/rev/604653c8b7d2
Use nsIClassInfoImpl in nsSimpleURI and children r=necko-reviewers,kershaw,xpcom-reviewers,nika
https://hg.mozilla.org/integration/autoland/rev/89ce38817d63
Use nsIClassInfoImpl in nsStandardURL and children r=necko-reviewers,dragana

Comment 9

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/b130f13ed5a9
https://hg.mozilla.org/mozilla-central/rev/0424bd36a7bf
https://hg.mozilla.org/mozilla-central/rev/20f842b9132f
https://hg.mozilla.org/mozilla-central/rev/604653c8b7d2
https://hg.mozilla.org/mozilla-central/rev/89ce38817d63