Closed
Bug 1152425
Opened 11 years ago
Closed 11 years ago
String type check in error lookup should consider unicode strings
Categories
(Testing :: Marionette Client and Harness, defect)
Testing
Marionette Client and Harness
Tracking
(firefox40 fixed)
RESOLVED
FIXED
mozilla40
| Tracking | Status | |
|---|---|---|
| firefox40 | --- | fixed |
People
(Reporter: ato, Assigned: ato)
References
Details
(Keywords: pi-marionette-client)
Attachments
(1 file, 1 obsolete file)
The error lookup by string doesn't consider unicode strings, which is unfortunate since the json module unmarshals to unicode strings.
|
Assignee | |
Comment 1•11 years ago
|
||
/r/6725 - Bug 1152425: Fix unicode string lookup in Marionette client
Pull down this commit:
hg pull -r 2981a890a0365f7ff519362400c893dd2c0ba51d https://reviewboard-hg.mozilla.org/gecko/
Attachment #8589758 -
Flags: review?(james)
|
|
Assignee | |
Updated•11 years ago
|
|
||
Comment 2•11 years ago
|
||
Comment on attachment 8589758 [details]
MozReview Request: bz://1152425/ato
https://reviewboard.mozilla.org/r/6723/#review5633
Ship It!
Attachment #8589758 -
Flags: review?(james) → review+
|
|
Assignee | |
Updated•11 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 3•11 years ago
|
||
Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/b9fcd818c945
|
|
Assignee | |
Updated•11 years ago
|
Keywords: checkin-needed
|
||
Comment 4•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-firefox40:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
|
|
Assignee | |
Comment 5•11 years ago
|
||
Attachment #8589758 -
Attachment is obsolete: true
Attachment #8619990 -
Flags: review+
|
|
Assignee | |
Comment 6•11 years ago
|
||
|
||
Updated•3 years ago
|
Product: Testing → Remote Protocol
|
||
Comment 7•3 years ago
|
||
Moving bugs for Marionette client due to component changes.
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•