1371893 - Stop relying on nsIStringInputStream inheriting from nsIInputStream

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Boris Zbarsky [:bzbarsky]]

I'm going to change that in bug 1371699.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code

Comment 2

[Boris Zbarsky [:bzbarsky]]

Note: I have NOT compiled this patch locally or done a try run with it.  Assuming it compiles, I believe, based on code inspection for comm-central, that it should do the right thing.

Comment 3

[Jorg K (CEST = GMT+2)]

Attachment: Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code (v2).

Made it compile. Boris, happy with those changes?

Try run is here, I'll comment later on the result:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=09b17530481d24ca49aff452258d052134c073f2

Comment 4

[Jorg K (CEST = GMT+2)]

I had to lose the const due to this error:

c:/mozilla-source/comm-central/calendar/base/backend/libical/calICSService.cpp(1012): error C2664: 'nsresult CallQueryInterface<nsIStringInputStream,nsIInputStream>(nsCOMPtr<nsIStringInputStream> &,DestinationType **)': cannot convert argument 1 from 'const nsCOMPtr<nsIStringInputStream>' to 'nsCOMPtr<nsIStringInputStream> &'
        with
        [
            DestinationType=nsIInputStream
        ]
c:/mozilla-source/comm-central/calendar/base/backend/libical/calICSService.cpp(1012): note: Conversion loses qualifiers

Comment 5

[Jorg K (CEST = GMT+2)]

https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=09b17530481d24ca49aff452258d052134c073f2
is "green". Intermittent failure on Mac xpcshell-test, Mozmill is red due to bug 1371898.

Comment 6

[Jorg K (CEST = GMT+2)]

Comment on attachment 8876373 [details] [diff] [review]
Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code (v2).

The changes look OK to me. Thanks for taking care of Thunderbird :-)
Feel free to request review from Kent.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8876373 [details] [diff] [review]
Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code (v2).

Thanks for fixing up the compile issues.  Looks good to me!

It's basically up to you whether you think review from Kent is needed.  If you feel competent to review the changes, I'm good with it, I think.

Comment 8

[Jorg K (CEST = GMT+2)]

Comment on attachment 8876373 [details] [diff] [review]
Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code (v2).

When are you going to land bug 1371699? Do you think we should land this patch here now so we don't have to track the other bug?

I put r?rkent just in case. Of course if Kent's review hasn't arrived before the patch is needed, I'll land it with my review only.

Comment 9

[Boris Zbarsky [:bzbarsky]]

> When are you going to land bug 1371699? 

Well, seeing as I don't have patches up yet and need reviews... Certainly not until Monday.  More likely a few days after that.

Comment 10

[Kent James (:rkent)]

Comment on attachment 8876373 [details] [diff] [review]
Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code (v2).

Review of attachment 8876373 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.

Comment 11

[Jorg K (CEST = GMT+2)]

This now ports bug 1401171.

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/4ff3727aec0f
Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code. r=jorgk,rkent

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e570d6d99079
Backed out changeset 4ff3727aec0f as experiment to see whether it's related to crash in bug 1408175. a=jorgk

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/8cd587465eab
Stop relying on nsIStringInputStream inheriting from nsIInputStream in comm-central code. r=jorgk,rkent