Closed
Bug 880333
Opened 12 years ago
Closed 11 years ago
[STK] "STK_CMD_SET_UP_IDLE_MODE_TEXT" code refactoring
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(blocking-b2g:leo+, b2g18 fixed, b2g-v1.1hd fixed)
People
(Reporter: noemi, Assigned: frsela)
References
Details
(Whiteboard: QARegressExclude)
Attachments
(1 file)
STK code refactoring moving "STK_CMD_SET_UP_IDLE_MODE_TEXT" code from settings to system
Reporter | ||
Updated•12 years ago
|
blocking-b2g: --- → leo?
Updated•12 years ago
|
blocking-b2g: leo? → leo+
Assignee | ||
Comment 1•12 years ago
|
||
This patch is the first one of the STK refactoring process explained in 875679 so this patch also contains the skeleton for the rest of ICC commands to be moved to System
Attachment #763435 -
Flags: review?(kaze)
Comment 2•12 years ago
|
||
Comment on attachment 763435 [details]
Moved SETUP IDLE MODE TEXT to system
The Settings part looks OK to me (= code removal) but please address my nitpicks. My main objection would be to finish bug 881672 first so we could use a proper DUMP() method instead of defining yet another debug() one.
Clearing the review flag. Feel free to feedback? me when my comments are addressed, but I think Tim would be more qualified for a review since it’s mostly a System patch now.
Attachment #763435 -
Flags: review?(kaze)
Assignee | ||
Updated•12 years ago
|
Attachment #763435 -
Flags: feedback?(kaze)
Assignee | ||
Updated•12 years ago
|
Attachment #763435 -
Flags: review?(timdream)
Comment 3•12 years ago
|
||
Milestoning for QE3 (due by 6/24), since this blocks bug 875679 which blocks 865985 (QE3 milestoned).
Target Milestone: --- → 1.1 QE3 (24jun)
Reporter | ||
Comment 4•12 years ago
|
||
Notice that Bug 881672 needs to be uplifted to v1-train in order to enable STK/ICC testing
Depends on: 881672
Comment 5•12 years ago
|
||
Comment on attachment 763435 [details]
Moved SETUP IDLE MODE TEXT to system
I leave comments about that DUMP() without read bug 881672.
One small nit: please add function names to all functions so we could see them in the stack trace and profiling.
Attachment #763435 -
Flags: review?(timdream) → review+
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) from comment #5)
> Comment on attachment 763435 [details]
> Moved SETUP IDLE MODE TEXT to system
>
> I leave comments about that DUMP() without read bug 881672.
>
> One small nit: please add function names to all functions so we could see
> them in the stack trace and profiling.
Ok, thanks Tim. I'll add function names and merge it.
Assignee | ||
Comment 7•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 8•11 years ago
|
||
I was not able to uplift this bug to v1-train. If this bug has dependencies which are not marked in this bug, please comment on this bug. If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval. Otherwise, if this is just a merge conflict, you might be able to resolve it with:
git checkout v1-train
git cherry-pick -x -m1 e2a1b8437a8b100882e6c7b2f1f0780bfafbf8c1
<RESOLVE MERGE CONFLICTS>
git commit
Flags: needinfo?(frsela)
Assignee | ||
Comment 9•11 years ago
|
||
No dependencies. Conflict solved and uplifted:
https://github.com/mozilla-b2g/gaia/commit/d0a7727176137b7255d5f12731011c59048f4003
(In reply to John Ford [:jhford] -- please use 'needinfo?' instead of a CC from comment #8)
> I was not able to uplift this bug to v1-train. If this bug has dependencies
> which are not marked in this bug, please comment on this bug. If this bug
> depends on patches that aren't approved for v1-train, we need to re-evaluate
> the approval. Otherwise, if this is just a merge conflict, you might be
> able to resolve it with:
>
> git checkout v1-train
> git cherry-pick -x -m1 e2a1b8437a8b100882e6c7b2f1f0780bfafbf8c1
> <RESOLVE MERGE CONFLICTS>
> git commit
Flags: needinfo?(frsela)
Updated•11 years ago
|
status-b2g18:
--- → fixed
Comment 10•11 years ago
|
||
1.1hd: d0a7727176137b7255d5f12731011c59048f4003
status-b2g-v1.1hd:
--- → fixed
Updated•11 years ago
|
Whiteboard: QARegressExclude
Assignee | ||
Updated•11 years ago
|
Attachment #763435 -
Flags: feedback?(kaze)
You need to log in
before you can comment on or make changes to this bug.
Description
•