Created attachment 632380 [details] [diff] [review] patch v1: corrects return values While reviewing the fix for bug 760747, I noticed that in the Windows webapp runtime stub AttemptGRELoadAndLaunch() sometimes returns an nsresult (i.e. an int), although its return type is boolean. And main() assigns the return value of GetFirefoxDirFromRegistry() to `rv`, which is an nsresult, although the return value of that function is a boolean. Here are the obvious fixes.
Comment on attachment 632380 [details] [diff] [review] patch v1: corrects return values https://hg.mozilla.org/integration/mozilla-inbound/rev/deaaaefbc089
Sounds like this is internal code changes - no verification needed (correct me if I'm incorrect though Myk if you think there is end-user testing to this).