Closed Bug 486679 Opened 15 years ago Closed 15 years ago

misleading assertion about NS_APP_USER_PROFILE_50_DIR not being defined

Categories

(Core Graveyard :: Embedding: GRE Core, defect)

Product:
Core Graveyard
Component:
Embedding: GRE Core
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: tomeu, Assigned: tomeu)

References

Details

Attachments

(1 file)

proposed fix
584 bytes, patch
benjamin
: review+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.8) Gecko/2009032711 Ubuntu/8.10 (intrepid) Firefox/3.0.8
Build Identifier: 

The message of the assertion doesn't corresponds to what is actually asserted.

Reproducible: Always
Attached patch proposed fix — — Splinter Review 

    
    

    
  
Thanks for the patch. According to https://developer.mozilla.org/en/Getting_your_patch_in_the_tree you need a review in order to get the patch checked in.

(the original text added in http://hg.mozilla.org/mozilla-central/rev/80787c4cbf7a )
Assignee: nobody → tomeu
Blocks: 408925
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: unspecified → Trunk

    
  

        Attachment #370857 -
        Flags: review?(benjamin)

    
    

    
  
Comment on attachment 370857 [details] [diff] [review]
proposed fix

note, we should fix the spelling of "not" on checkin

        Attachment #370857 -
        Flags: review?(benjamin) → review+

    
  
Keywords: checkin-needed

    
    

    
  
http://hg.mozilla.org/mozilla-central/rev/260ad7c88d90
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Product: Core → Core Graveyard

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