Bug 1577252 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
STEPS TO REPRODUCE:

1) Load about:config
2) Click through the warning
3) Type "foo.bar" in the about:config search field
4) Click the "Add" button to try to add the pref

EXPECTED RESULTS: No exceptions

ACTUAL RESULTS:
```
JavaScript error: chrome://browser/content/content.js, line 46: TypeError: nodePrincipal.URI is null
```

ADDITIONAL INFORMATION: Look at that URL locally, line 46 is:
```
  return nodePrincipal.isNullPrincipal || nodePrincipal.URI.schemeIs("about");
```
in the `shouldIgnoreLoginManagerEvent` function.  This is the code that was added in bug 1550171, as far as I can tell.  In this case `nodePrincipal` is a system principal, so there is no URI and the function throws an exception.
Revision 1 by
on 
STEPS TO REPRODUCE:

1) Load about:config
2) Click through the warning
3) Type "foo.bar" in the about:config search field
4) Click the "Add" button to try to add the pref

EXPECTED RESULTS: No exceptions

ACTUAL RESULTS:
```
JavaScript error: chrome://browser/content/content.js, line 46: TypeError: nodePrincipal.URI is null
```

ADDITIONAL INFORMATION: Looking at that URL locally, line 46 is:
```
  return nodePrincipal.isNullPrincipal || nodePrincipal.URI.schemeIs("about");
```
in the `shouldIgnoreLoginManagerEvent` function.  This is the code that was added in bug 1550171, as far as I can tell.  In this case `nodePrincipal` is a system principal, so there is no URI and the function throws an exception.

Back to Bug 1577252 Comment 0