Closed Bug 582442 Opened 10 years ago Closed 10 years ago

Make unserializable URIs less fatal

Categories

(Core :: IPC, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fennec 2.0a1+ ---

People

(Reporter: jdm, Assigned: jdm)

References

Details

Attachments

(1 file, 1 obsolete file)

As suggested by bug 516728 comment 66, let's just make this a warning for now.
Attached patch Patch (obsolete) — Splinter Review
Assignee: nobody → josh
Attachment #460717 - Flags: review?(doug.turner)
Comment on attachment 460717 [details] [diff] [review]
Patch

hmm. what about the abort in the serialization?
Attached patch PatchSplinter Review
Actual patch.
Attachment #460717 - Attachment is obsolete: true
Attachment #460732 - Flags: review?(doug.turner)
Attachment #460717 - Flags: review?(doug.turner)
Attachment #460732 - Flags: review?(doug.turner) → review+
tracking-fennec: ? → 2.0a1+
It seems we need this patch to be able to land other fennec-blocking patches
hmm... I've just changed 
-      NS_ABORT_IF_FALSE(scheme.EqualsASCII("about:") ||
+      NS_WARN_IF_FALSE(scheme.EqualsASCII("about:") ||

in my local builds
http://hg.mozilla.org/mozilla-central/rev/f9214514d547
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.