Unified Diff: lib/contentPolicy.js
Issue 6263536791584768:
Issue 2232 - Got rid of non-standard conditional catch clauses (Closed)
Patch Set:
Created March 30, 2015, 4:57 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/contentPolicy.js |
| =================================================================== |
| --- a/lib/contentPolicy.js |
| +++ b/lib/contentPolicy.js |
| @@ -344,12 +344,17 @@ |
| { |
| registrar.registerFactory(this.classID, this.classDescription, this.contractID, this); |
| } |
| - catch (e if e.result == Cr.NS_ERROR_FACTORY_EXISTS) |
| + catch (e) |
| { |
| // See bug 924340 - it might be too early to init now, the old version |
| // we are replacing didn't finish removing itself yet. |
| - Utils.runAsync(this.init.bind(this)); |
| - return; |
| + if (e.result == Cr.NS_ERROR_FACTORY_EXISTS) |
| + { |
| + Utils.runAsync(this.init.bind(this)); |
| + return; |
| + } |
| + |
| + Cu.reportError(e); |
| } |
| let catMan = Utils.categoryManager; |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
