Closed Bug 787988 Opened 12 years ago Closed 12 years ago

"Error: 0" reported when opening preference dialog (leftover debug statement?)

Categories

(Thunderbird :: Preferences, defect)

Product:
Thunderbird
Component:
Preferences
Version:
16 Branch
Platform:
All
Windows 7
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(thunderbird17 fixed)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 18.0
Tracking Flags:
Tracking Status
thunderbird17 --- fixed

People

(Reporter: ssitter, Assigned: aceman)

References

Details

Attachments

(1 file)

patch
1.20 KB, patch
standard8
: review+
standard8
: approval-comm-beta+
Details | Diff | Splinter Review 
Thunderbird 16.0b1, Earlybird 17.0a2, Daily 18.0a1

Opening the preference dialog, advanced panel will throw the following error in Error Console:

> Error: 0
> Source file: chrome://messenger/content/preferences/advanced.js Line: 237

This looks like a leftover debug statement
> http://mxr.mozilla.org/comm-beta/source/mail/components/preferences/advanced.js#240

The error message is confusing and should be removed. Or it should be downgraded to an informational message with useful content if a debug message is required.
Version: unspecified → 16
Nice, I can remove it.
Assignee: nobody → acelists
Blocks: 758326
Hardware: x86_64 → All
Attached patch patch — — Splinter Review 

        Attachment #658173 -
        Flags: review?(mbanner)

    
  
Severity: minor → trivial
Status: NEW → ASSIGNED

        Attachment #658173 -
        Flags: review?(mbanner) → review+

    
  
Keywords: checkin-needed

    
    

    
  
https://hg.mozilla.org/comm-central/rev/49a9b1d1e8be
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0

    
    

    
  
Comment on attachment 658173 [details] [diff] [review]
patch

[Approval Request Comment]

User impact if declined: Meaningless error message appears
Testing completed (on c-c, etc.): Yes
Risk to taking this patch (and alternatives if risky): None

        Attachment #658173 -
        Flags: approval-comm-beta?

        Attachment #658173 -
        Flags: approval-comm-beta? → approval-comm-beta+

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: