Closed
Bug 993018
Opened 11 years ago
Closed 11 years ago
Make the "place new call" feature independent of the callscreen->appwindow message channel
Categories
(Firefox OS Graveyard :: Gaia::Dialer, defect)
Tracking
(blocking-b2g:1.3T+, b2g-v1.3T fixed, b2g-v1.4 fixed)
RESOLVED
FIXED
blocking-b2g | 1.3T+ |
People
(Reporter: etienne, Assigned: etienne)
References
Details
Attachments
(2 files)
46 bytes,
text/x-github-pull-request
|
rik
:
review+
|
Details | Review |
15.22 KB,
patch
|
lmandel
:
approval-gaia-v1.4+
|
Details | Diff | Splinter Review |
Other blocking point for bug 990003.
Assignee | ||
Comment 1•11 years ago
|
||
1.3t? because blocking a 1.3t+
Assignee | ||
Comment 2•11 years ago
|
||
Same deal.
Sorry for the emergency, trying to sweeten the deal with more test coverage :)
Attachment #8402784 -
Flags: review?(anthony)
Comment 3•11 years ago
|
||
triage: 1.3T+ for resolving dialer incoming call performance
blocking-b2g: 1.3T? → 1.3T+
Updated•11 years ago
|
Whiteboard: [tarako_only]
Updated•11 years ago
|
status-b2g-v1.3T:
--- → affected
Comment 5•11 years ago
|
||
Comment on attachment 8402784 [details] [review]
Gaia PR
That actually makes more sense to me.
Attachment #8402784 -
Flags: review?(anthony) → review+
Assignee | ||
Comment 6•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
There are a lot of conflicts when cherry-pick this into v1.3t branch?
Can you please rebase your patch on v1.3t?
Flags: needinfo?(etienne)
Assignee | ||
Comment 8•11 years ago
|
||
Flags: needinfo?(etienne)
Comment 9•11 years ago
|
||
This is a clean cherry-pick from master, there were no conflicts while cherry-picking. This is a dependency of bug 990003 which should be uplifted to v1.4.
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Decoupling needed for bug 990003
[User impact] if declined: Cannot uplift bug 990003 to v1.4
[Testing completed]: The patch doesn't require any modifications and has already been used in master for a while
[Risk to taking this patch] (and alternatives if risky): Uplifting the dependencies without this patch is unrealistic
[String changes made]: None
Attachment #8446514 -
Flags: approval-gaia-v1.4?(release-mgmt)
Comment 10•11 years ago
|
||
Comment on attachment 8446514 [details] [diff] [review]
Cherry-picked patch for v1.4
Approved for 1.4.
Attachment #8446514 -
Flags: approval-gaia-v1.4?(release-mgmt) → approval-gaia-v1.4+
Comment 11•11 years ago
|
||
Uplifted to gaia/v1.4 171471a0b5f560977db1513017818479b92e1dc9
https://github.com/mozilla-b2g/gaia/commit/171471a0b5f560977db1513017818479b92e1dc9
status-b2g-v1.4:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•