Last Comment Bug 783738 - Use Components.results.NS_BINDING_ABORTED instead of hardcoding the value
: Use Components.results.NS_BINDING_ABORTED instead of hardcoding the value
Product: Core
Classification: Components
Component: General (show other bugs)
: Trunk
: All All
-- minor (vote)
: mozilla17
Assigned To: Phil Ringnalda (:philor)
Depends on:
  Show dependency treegraph
Reported: 2012-08-17 21:48 PDT by Phil Ringnalda (:philor)
Modified: 2012-08-18 04:25 PDT (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

fix (5.42 KB, patch)
2012-08-17 21:48 PDT, Phil Ringnalda (:philor) review+
Details | Diff | Splinter Review

Description User image Phil Ringnalda (:philor) 2012-08-17 21:48:53 PDT
Created attachment 653019 [details] [diff] [review]

+++ This bug was initially created as a clone of Bug #454360 +++

Since bug 418406 was nice enough to expose network error constants in Components.results back in 2008, we made use of them rather than hardcoding 0x804b0002 all over the place in JS...

but that's the sort of thing that creeps back in, from people remembering all the years before 2008 instead of the years after it.
Comment 1 User image :Gavin Sharp [email:] 2012-08-17 21:59:46 PDT
Comment on attachment 653019 [details] [diff] [review]

Cr is defined in all of the test files, you could use it
Comment 2 User image Phil Ringnalda (:philor) 2012-08-17 22:15:38 PDT
I did consider using Cr, but it's freshly defined from a tree-wide change, nothing else in the files uses any of the C*s, so using just one down in the middle is a bit risky, and changing them all seemed both intrusive and likely to bring down the wrath of someone with a superstition about not using them.
Comment 3 User image Ryan VanderMeulen [:RyanVM] 2012-08-18 04:25:36 PDT

Note You need to log in before you can comment on or make changes to this bug.