Closed
Bug 900080
Opened 12 years ago
Closed 12 years ago
Marionette errors in onPacket lack exception information
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(firefox23 wontfix, firefox24 wontfix, firefox25 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix, b2g-v1.1hd fixed)
RESOLVED
FIXED
mozilla25
People
(Reporter: jgriffin, Assigned: jgriffin)
References
Details
Attachments
(1 file)
|
823 bytes,
patch
|
wlach
:
review+
|
Details | Diff | Splinter Review |
When Marionette catches an exception in onPacket, we don't include any information about the original exception, e.g.,
09:07:09 INFO - errors.MarionetteException: {u'message': {}, u'error': u"error occurred while processing 'deleteSession"}
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #783831 -
Flags: review?(wlachance)
|
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → jgriffin
|
||
Comment 2•12 years ago
|
||
Comment on attachment 783831 [details] [diff] [review]
Use e.message instead of e in onPacket errors,
lgtm
Attachment #783831 -
Flags: review?(wlachance) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
Target Milestone: --- → mozilla25
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•12 years ago
|
Keywords: checkin-needed
|
||
Comment 5•12 years ago
|
||
status-b2g18:
--- → fixed
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
status-b2g-v1.1hd:
--- → affected
status-firefox23:
--- → wontfix
status-firefox24:
--- → wontfix
status-firefox25:
--- → fixed
Keywords: checkin-needed
|
|
||
Comment 6•12 years ago
|
||
Apparently I forgot to mark this bug when it landed on the v1.1hd branch. My apologies.
https://hg.mozilla.org/releases/mozilla-b2g18_v1_1_0_hd/rev/ba71c64705e5
|
||
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•