Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[e10s]: newChannelFromURI - it sometimes throws the error #2252

Conversation

janekptacijarabaci
Copy link
Contributor

Ad #2202
The suggestion (for example).
It is necessary to test all options (e.g.: correct permissions / the correct principal?)!

error

nsIOService::NewChannelFromURI is deprecated:
https://hg.mozilla.org/mozilla-central/file/66a72727e1b3/netwerk/base/nsIOService.cpp#l603

Unfortunately, there are no references to the "NewChannelFromURI2" APIs in the documentation...

The short documentation:
https://hg.mozilla.org/mozilla-central/file/66a72727e1b3/netwerk/base/nsIIOService.idl#l68

@janekptacijarabaci janekptacijarabaci changed the title <tabbrowser>.addTab - it sometimes throws the error newChannelFromURI - it sometimes throws the error Aug 22, 2015
@janekptacijarabaci janekptacijarabaci changed the title newChannelFromURI - it sometimes throws the error [e10s]: newChannelFromURI - it sometimes throws the error Aug 22, 2015
@arantius arantius added this to the 3.4 milestone Aug 22, 2015
@the8472
Copy link
Contributor

the8472 commented Aug 22, 2015

have you considered using privileged XHRs instead of constructing channels manually? That might be more forward-portable.

Load flags on the channel can be set like this: xhr.channel.loadFlags |= Ci.nsIRequest.LOAD_ANONYMOUS | Ci.nsIRequest.LOAD_BYPASS_CACHE

arantius added a commit that referenced this pull request Aug 24, 2015
…rowsTheError

[e10s]: newChannelFromURI - it sometimes throws the error
@arantius arantius merged commit cc8848e into greasemonkey:master Aug 24, 2015
@janekptacijarabaci janekptacijarabaci deleted the newChannelFromURI_throwsTheError branch August 24, 2015 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants