Closed
Bug 887093
Opened 11 years ago
Closed 11 years ago
[MMS] view attach image in text field will crash
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox25 | --- | unaffected |
| b2g18 | --- | fixed |
| b2g18-v1.0.0 | --- | wontfix |
| b2g18-v1.0.1 | --- | wontfix |
| b2g-v1.1hd | --- | fixed |
People
(Reporter: bent.mozilla, Assigned: bent.mozilla)
References
Details
(Keywords: crash, Whiteboard: [b2g-crash] [TD-42216] leorun3)
Attachments
(2 files)
|
7.54 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
|
1.69 KB,
patch
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #877482 +++
Platform fix. Not sure if we want to block leo on this or not given that we have a gaia workaround. We need this in the platform regardless though.
Attachment #767545 -
Flags: review?(khuey)
|
||
Comment 1•11 years ago
|
||
|
|
||
Comment 2•11 years ago
|
||
I've tested your patch on v1-train and it fixes all related bugs: bug 880825, bug 886772, bug 886773 and bug 877482 once the gaia workaround is removed - see attachment 767587 [details] [diff] [review]. If you think this can land this week I'll push for it to be leo+'d; I very much prefer this than landing three distinct workarounds in Gaia.
|
||
Updated•11 years ago
|
blocking-b2g: leo? → leo+
|
Assignee | |
Comment 3•11 years ago
|
||
khuey, what's your timeline for review here? Please see comment 2.
Flags: needinfo?(khuey)
Dude you requested review on this 4 hours ago ...
Flags: needinfo?(khuey)
|
|
Assignee | |
Comment 5•11 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #4)
> Dude you requested review on this 4 hours ago ...
Not trying to rush, just wanted to know if you think you can do it this week or if we should go with the other patch, like Gabriele is asking in comment 2.
We can review this in the office tomorrow.
Attachment #767545 -
Flags: review?(khuey) → review+
|
|
Assignee | |
Updated•11 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 8•11 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM UTC-4][PTO 6/27-6/30] from comment #7)
> This doesn't apply to birch/m-c at all.
Oops! Sorry, this is a b2g18-only patch.
Keywords: checkin-needed
|
||
Comment 9•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
Resolution: --- → FIXED
|
|
||
Comment 10•11 years ago
|
||
and backout:
https://hg.mozilla.org/releases/mozilla-b2g18/rev/a80ebb0a0d51
because of
https://tbpl.mozilla.org/php/getParsedLog.php?id=24698178&tree=Mozilla-B2g18&full=1
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
||
Updated•11 years ago
|
Keywords: checkin-needed
|
|
||
Updated•11 years ago
|
|
|
Assignee | |
Comment 11•11 years ago
|
||
|
|
Assignee | |
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•11 years ago
|
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
|
||
Updated•11 years ago
|
status-b2g-v1.1hd:
--- → affected
|
|
||
Comment 12•11 years ago
|
||
status-firefox25:
--- → unaffected
Target Milestone: --- → 1.1 QE4 (15jul)
|
|
||
Updated•11 years ago
|
Blocks: gaia-euro-sprint-6
|
|
||
Updated•11 years ago
|
No longer blocks: gaia-euro-sprint-6
|
||
Comment 14•11 years ago
|
||
Verified by pvt build 20130715230201
Issue is solved successfully without crash.
Status: RESOLVED → VERIFIED
Blocks: 912861
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•