Closed
Bug 1028271
Opened 10 years ago
Closed 10 years ago
[B2G][Dialer]Call logs are not displaying correctly with the patch from bug 1021248
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | unaffected |
b2g-v2.1 | --- | verified |
People
(Reporter: astole, Assigned: kats)
References
()
Details
(Keywords: regression, smoketest)
Attachments
(1 file)
70.75 KB,
text/plain
|
Details |
When first opening the call log when there is call history, nothing is shown until selecting the edit button or selecting a different menu then returning to the call log.
Repro Steps:
1) Update a Flame to BuildID: 20140620040204
2) Make or receive a call to DUT
3) Open dialer app
4) Select call log
Actual:
There is no information displayed in the call log
Expected:
The call log displays correct information
2.1 Environmental Variables:
Device: Flame 2.1
BuildID: 20140620040204
Gaia: ccd70903544486bea04e85d8a4aacf63f1de2a72
Gecko: bdac18bd6c74
Version: 33.0a1
Firmware Version: v121-2
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0
Repro frequency: 2/2, 100%
See attached: logcat, video
Reporter | ||
Comment 1•10 years ago
|
||
This issue does not repro on the latest 2.0 on Flame. The call log displays correct information.
2.0 Environmental Variables:
Device: Flame 2.0
BuildID: 20140620000202
Gaia: c5dd47e3f9e18872961946735fdbc071a1656ac9
Gecko: 71b1b4b0850c
Version: 32.0a2
Firmware Version: v121-2
Reporter | ||
Comment 2•10 years ago
|
||
Note:
When attempting to edit the call log, sometimes there are no check boxes and if there are check boxes, the numbers can be selected but no check appears in the box notifying that the number has been selected.
Reporter | ||
Comment 3•10 years ago
|
||
Note:
Also, after closing the edit menu, the date ('Today' in video) is cut off above number in call log.
Reporter | ||
Updated•10 years ago
|
Flags: needinfo?(jmitchell)
Updated•10 years ago
|
blocking-b2g: --- → 2.1?
Updated•10 years ago
|
Flags: needinfo?(jmitchell)
QA Contact: pcheng
Comment 5•10 years ago
|
||
Mozilla inbound regression window:
Last Working Environmental Variables:
Device: Flame
Build ID: 20140619055933
Gaia: 82e957160ca017087bd374cd051339e55b641e68
Gecko: 5f1041f40876
Version: 33.0a1 (2.1 - Master)
Firmware Version: v121-2
First Broken Environmental Variables:
Device: Flame
Build ID: 20140619062833
Gaia: 82e957160ca017087bd374cd051339e55b641e68
Gecko: f17c906538c9
Version: 33.0a1 (2.1 - Master)
Firmware Version: v121-2
Gaia is the same on both builds, which means this is a Gecko issue.
Gecko pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=5f1041f40876&tochange=f17c906538c9
Flags: needinfo?(jmitchell)
Keywords: qaurgent,
regressionwindow-wanted
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell)
Comment 6•10 years ago
|
||
Broken by either bug 1021248 or bug 1026756.
kats - can you figure out which patch caused this & either backout the patch or fix this quickly?
Component: Gaia::Dialer → Graphics: Layers
Flags: needinfo?(bugmail.mozilla)
Product: Firefox OS → Core
Version: unspecified → Trunk
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Assignee | ||
Comment 7•10 years ago
|
||
The patches on bug 1021085 seem to fix this, based on my local testing. They're pending review and should land soonish but if you don't want to wait I can back out and reland everything together. Let me know.
Assignee | ||
Comment 8•10 years ago
|
||
I landed bug 1021085 on inbound just now which I think should fix this. After some more local testing I was only able to intermittently reproduce this issue though so I may be wrong. Would like somebody who can consistently reproduce this to retest with an inbound build that includes bug 1021085 to see if it is fixed.
Flags: needinfo?(jsmith)
Comment 9•10 years ago
|
||
Ok - marking qawanted to get someone to test this on the latest mozilla-inbound build.
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [lead-review+]
Updated•10 years ago
|
QA Whiteboard: [lead-review+]
Comment 10•10 years ago
|
||
This issue DOES reproduce on the latest Mozilla Inbound build.
Environmental Variables:
Device: Flame 2.1
Build ID: 20140623055944
Gaia: bd5065ced020014df5fd45259fba1ac32d65673b
Gecko: 6aade5365ee1
Version: 33.0a1
Firmware Version: v121-2
Comment 11•10 years ago
|
||
So we've confirmed above we can still reproduce with the patch mentioned.
kats - can we backout the regressing patch here & see what's causing this?
Flags: needinfo?(bugmail.mozilla)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell)
Assignee | ||
Comment 12•10 years ago
|
||
Ok, but I'm going to need better STR if I have a chance of fixing this at all, because the last few times I tried I couldn't reproduce it at all. How many items are in the call log when this happens? Is it less than a screenful or more?
Comment 13•10 years ago
|
||
Andrew or Harvey- can you address the question in comment 12?
Flags: needinfo?(astole)
Assignee | ||
Comment 14•10 years ago
|
||
I backed out bug 1021248 to fix this (in my local testing that was the one of the two patches that seemed to be the culprit).
https://hg.mozilla.org/integration/mozilla-inbound/rev/0f07aa386616
Flags: needinfo?(bugmail.mozilla)
Comment 15•10 years ago
|
||
Ok - will mark this fixed via backout once the changeset from comment 14 merges to central.
Assignee | ||
Comment 16•10 years ago
|
||
I managed to grab a log from one instance of this problem occurring. The log indicates that AboutToCheckerboard is returning true when it shouldn't be, and so the paint gets aborted and it just blanks.
The metrics that result in AboutToCheckerboard returning true are:
content metrics: critdp=(0, 0, 320x500.333344) viewportScroll=(0, 0)
compositor metrics: viewportScroll=(-0, 0) cb=(0, 0, 480x751)
When the composition bounds is converted to CSS pixels (1.5 scale) it ends up 320x500.666667 which is 0.3333 taller than the critical displayport, and so the painted area doesn't "contain" the visible area. This is basically a rounding error. I can fix it as part of bug 1024126.
Assignee | ||
Comment 17•10 years ago
|
||
Oh, also in comment 16, the scrollableRect is 320x500.333344 so clamping the "showing" area to the scrollable rect would also have fixed it in this instance.
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Comment 18•10 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #14)
> I backed out bug 1021248 to fix this (in my local testing that was the one
> of the two patches that seemed to be the culprit).
>
> https://hg.mozilla.org/integration/mozilla-inbound/rev/0f07aa386616
Has merged to m-c now.
Comment 19•10 years ago
|
||
Clearing flags now that the regressing patch has been backed out.
blocking-b2g: 2.1? → ---
Keywords: regression,
smoketest
Assignee | ||
Comment 20•10 years ago
|
||
Resolving since the regressing patch was backed out. The underlying issue behind this bug (as described in comment 16) was also fixed by bug 1024126. I verified that locally on a build that had the patch from bug 1024126 without the backout.
Not sure if this should be resolved FIXED or WORKSFORME (or even duped to 1024126?) but since there's no actual patch on here I'll close this as WFM for now.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
Comment 21•10 years ago
|
||
Usually I mark fixed by backout as FIXED mainly because the regressing patch being backed out counts as the patch to fix the bug.
Updated•10 years ago
|
Target Milestone: --- → mozilla33
Comment 22•10 years ago
|
||
The edit window in call log displays and functions as expected. Unable to reproduce issue.
Device: Flame Master M/C
BuildID: 20140625040202
Gaia: 57da30f405ba37a5d4844f32bb292271b81faee2
Gecko: a19e0434ea52
Version: 33.0a1
Firmware Version: v122
resolving as verified fixed
Status: RESOLVED → VERIFIED
Comment 23•10 years ago
|
||
Verified fixed in latest Flame 2.1 build.
Call log displays properly each time it is opened.
Environmental Variables:
Device: Flame 2.1
BuildID: 20141021001201
Gaia: e458f5804c0851eb4e93c9eb143fe044988cecda
Gecko: ee86921a986f
Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d
Version: 34.0 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [QAnalyst-Triage?][lead-review+]
Flags: needinfo?(pbylenga)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?][lead-review+] → [QAnalyst-Triage+][lead-review+]
Flags: needinfo?(pbylenga)
You need to log in
before you can comment on or make changes to this bug.
Description
•