Closed Bug 406581 Opened 15 years ago Closed 15 years ago

Enhance cal::detectTimezone() to dump errors

Categories

(Calendar :: Internal Components, enhancement)

Product:
Calendar
Component:
Internal Components
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: dbo, Assigned: dbo)

Details

Attachments

(1 file)

patch - v1
3.33 KB, patch
Fallen
: review+
Details | Diff | Splinter Review 
- currently only NS_ASSERTION(tz, "no timezone found, falling back to floating!");
Better dump to js console resp. something more prominent.
Attached patch patch - v1 — — Splinter Review 
old patch lying around for some time

I am still not sure whether to log a warning or error... opinions?
Assignee: nobody → daniel.boelzle
Status: NEW → ASSIGNED
Attachment #334867 - Flags: review?(philipp)
Attachment #334867 - Attachment is patch: true
Attachment #334867 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 334867 [details] [diff] [review]
patch - v1

r=philipp
Attachment #334867 - Flags: review?(philipp) → review+
Changed to log an error and checked in on HEAD and MOZILLA_1_8_BRANCH => FIXED.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9
You need to log in before you can comment on or make changes to this bug.