Closed
Bug 902611
Opened 11 years ago
Closed 11 years ago
test_bug475156.html failing on b2g mochitest with (NS_NOINTERFACE) [nsISupports.QueryInterface]
Categories
(Core :: Networking, defect)
Tracking
()
RESOLVED
FIXED
mozilla26
People
(Reporter: martijn.martijn, Assigned: martijn.martijn)
References
()
Details
Attachments
(1 file, 2 obsolete files)
1.99 KB,
patch
|
Details | Diff | Splinter Review |
Follow-up from bug 901343, comment 7.
This is the code that's causing this error:
20 var ch = SpecialPowers.wrap(xhr).channel.QueryInterface(SpecialPowers.Ci.nsICachingChannel);
from bug 901343, comment 8:
"
Well, necko in a child process acts a bit differently... e.g. HttpChannelChild does not in fact QI to nsICachingChannel. So at least that test will need fixing to work in an e10s environment...
"
Assignee | ||
Comment 1•11 years ago
|
||
This is a poc patch that seems to work, not sure if this is the best way to fix this.
Attachment #787112 -
Flags: feedback?(bzbarsky)
Comment 2•11 years ago
|
||
Comment on attachment 787112 [details] [diff] [review]
475156.diff (poc)
Honza?
Attachment #787112 -
Flags: feedback?(bzbarsky) → feedback?(honzab.moz)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → martijn.martijn
Assignee | ||
Comment 3•11 years ago
|
||
Fromm bug 901343, comment 10:
Yes, we deliberately split out a subset of the the nsICachingChannel features into nsICacheInfoChannel, which is all that you get on the child side at the moment.
Assignee | ||
Comment 4•11 years ago
|
||
Ok, QI to nsICacheInfoChannel works very well, also works on desktop, so I can just replace the existing nsICachingChannel, I think.
Attachment #787112 -
Attachment is obsolete: true
Attachment #787112 -
Flags: feedback?(honzab.moz)
Attachment #787371 -
Flags: review?(jduell.mcbugs)
Comment 5•11 years ago
|
||
Comment on attachment 787371 [details] [diff] [review]
475156.diff
Review of attachment 787371 [details] [diff] [review]:
-----------------------------------------------------------------
glad that worked.
Attachment #787371 -
Flags: review?(jduell.mcbugs) → review+
Assignee | ||
Comment 6•11 years ago
|
||
Just to make sure it's not causing failures somehow on some platform, I pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=16f49767dff7
Assignee | ||
Comment 7•11 years ago
|
||
Everything green, this is for check-in.
Attachment #787371 -
Attachment is obsolete: true
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 8•11 years ago
|
||
Flags: in-testsuite+
Keywords: checkin-needed
Comment 9•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in
before you can comment on or make changes to this bug.
Description
•