Closed Bug 441979 Opened 17 years ago Closed 16 years ago

gAppData null check not required in nsXREDirProvider::GetUserDataDirectoryHome

Categories

(Toolkit :: Startup and Profile System, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.1a2

People

(Reporter: romaxa, Assigned: romaxa)

References

Details

Attachments

(1 file)

Attached patch Possible fixSplinter Review
./run-mozilla.sh ./TestGtkEmbed about: ............... *** registering nsLoginManager.js: [ LoginManager ] *** registering nsBlocklistService.js: [ Blocklist Service ] *** Registering done. WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file mozilla/toolkit/xre/nsXREDirProvider.cpp, line 1079 ............... Warning appears because gAppData not defined and GetUserDataDirectoryHome return always NS_ERROR: http://mxr.mozilla.org/seamonkey/source/toolkit/xre/nsXREDirProvider.cpp#985
Attachment #326859 - Flags: review?(benjamin)
I'm not sure, maybe Embedding: GRE Core is more better component for this bug?
Attachment #326859 - Flags: review?(benjamin) → review+
Assignee: nobody → romaxa
Keywords: checkin-needed
Component: XRE Startup → Startup and Profile System
QA Contact: xre.startup → startup
<http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1217484420.1217485284.24505.gz> {{ Linux nye Depend bloat on 2008/07/30 23:07:00 WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /home/andrew/tbox/SeaMonkey-Debug/Linux_2.6.22.14-72.fc6_Depend/src/mozilla/toolkit/xre/nsXREDirProvider.cpp, line 1134 }} *** checkin-needed !
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
(In reply to comment #2) > .../toolkit/xre/nsXREDirProvider.cpp, line 1134 Different bug ... moved to bug 450898.
Keywords: assertion
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: