Changes between Initial Version and Version 1 of Ticket #17965, comment 11


Ignore:
Timestamp:
Jan 22, 2016, 11:35:55 AM (4 years ago)
Author:
gk
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #17965, comment 11

    initial v1  
    1 I did not look much on the patch yet but decided to try some test bundles with it. It breaks at least HTTPS-E and it seems in a way that sites like facebook.com are not working anymore. In the error console I get:
     1I did not look much at the patch yet but decided to try some test bundles with it. It breaks at least HTTPS-E and it seems in a way that sites like facebook.com are not working anymore. In the error console I get:
    22{{{
    33NS_ERROR_XPC_NOT_ENOUGH_ARGS: Not enough arguments [nsISiteSecurityService.isSecureURI] HTTPS.js:43:0
     
    1212Line: 621, column: 0
    1313}}}
    14 We might want to think about a different approach than "just" adding an additional parameter to nsISiteSecureService methods.
     14We might want to think about a different approach than "just" adding an additional parameter to nsISiteSecurityService methods.