Closed
Bug 902821
Opened 10 years ago
Closed 10 years ago
Screen rotation will be lagged for a while.
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: vlin, Assigned: vlin)
References
Details
Attachments
(1 file, 1 obsolete file)
2.02 KB,
patch
|
mwu
:
review+
|
Details | Diff | Splinter Review |
Bug 796722 will not fix screen rotation issue completely. Open a new bug to fix it. The modification is referenced to the *PaintWindow calls flow in Bug 826817.
Attachment #787354 -
Flags: review?(mwu)
Attachment #787354 -
Flags: review?(mchen)
Comment 1•10 years ago
|
||
Comment on attachment 787354 [details] [diff] [review] fix.patch Review of attachment 787354 [details] [diff] [review]: ----------------------------------------------------------------- Michael is the correct one to do the review and is the module owner. ::: widget/gonk/nsWindow.cpp @@ +255,4 @@ > AutoLayerManagerSetup setupLayerManager( > gWindowToRedraw, ctx, mozilla::layers::BUFFER_NONE, > ScreenRotation(EffectiveScreenRotation())); > + Remove spaces. @@ +255,5 @@ > AutoLayerManagerSetup setupLayerManager( > gWindowToRedraw, ctx, mozilla::layers::BUFFER_NONE, > ScreenRotation(EffectiveScreenRotation())); > + > + listener = gWindowToRedraw->GetWidgetListener(); Does it need to get listener again and again or just get it once is enough?
Attachment #787354 -
Flags: review?(mchen)
Assignee | ||
Comment 2•10 years ago
|
||
(In reply to Marco Chen [:mchen] from comment #1) > Comment on attachment 787354 [details] [diff] [review] > fix.patch > > Review of attachment 787354 [details] [diff] [review]: > ----------------------------------------------------------------- > > Michael is the correct one to do the review and is the module owner. > > ::: widget/gonk/nsWindow.cpp > @@ +255,4 @@ > > AutoLayerManagerSetup setupLayerManager( > > gWindowToRedraw, ctx, mozilla::layers::BUFFER_NONE, > > ScreenRotation(EffectiveScreenRotation())); > > + > > Remove spaces. > > @@ +255,5 @@ > > AutoLayerManagerSetup setupLayerManager( > > gWindowToRedraw, ctx, mozilla::layers::BUFFER_NONE, > > ScreenRotation(EffectiveScreenRotation())); > > + > > + listener = gWindowToRedraw->GetWidgetListener(); > > Does it need to get listener again and again or just get it once is enough? Yes, it does. Comment from here. https://bugzilla.mozilla.org/show_bug.cgi?id=796722#c20
Assignee | ||
Updated•10 years ago
|
blocking-b2g: --- → hd?
Assignee | ||
Updated•10 years ago
|
blocking-b2g: hd? → leo?
Updated•10 years ago
|
Assignee: nobody → vlin
Comment 3•10 years ago
|
||
Spoken to vlin, bug 796722 will work on its own for master, but this bug is required for 796722 to have effect on v1.1. leo+ing this as bug 796722 is leo+ and has been included on v1.1
Comment 4•10 years ago
|
||
Comment on attachment 787354 [details] [diff] [review] fix.patch Review of attachment 787354 [details] [diff] [review]: ----------------------------------------------------------------- Is this a 1.1 only patch? There's a bunch of tabs in this patch. Please replace them with spaces and make sure the style is consistent.
Comment 5•10 years ago
|
||
Comment on attachment 787354 [details] [diff] [review] fix.patch Clearing review while waiting for a new patch and an answer about whether this is 1.1 only.
Attachment #787354 -
Flags: review?(mwu)
Comment 6•10 years ago
|
||
(In reply to Wayne Chang [:wchang] from comment #3) > leo+ing this as bug 796722 is leo+ and has been included on v1.1 This isn't automatically leo+, it all depends on current user experience. Are we sure it's not good enough already?
blocking-b2g: leo+ → leo?
Comment 7•10 years ago
|
||
Flagging Patrick to comment on this rotation issue since this is HD. Please let me know if this should be a blocker for 1.1.
Flags: needinfo?(padamczyk)
Comment 8•10 years ago
|
||
Can I get more info? I am running 2013-08-22's build and the rotation on apps like the browser is OKAY, but could be faster. Its about 2 second delay before rotation, I'd expect it to be under 1 second. Is this what you mean?
Assignee | ||
Comment 9•10 years ago
|
||
(In reply to Patryk Adamczyk [:patryk] UX from comment #8) > Can I get more info? I am running 2013-08-22's build and the rotation on > apps like the browser is OKAY, but could be faster. Its about 2 second delay > before rotation, I'd expect it to be under 1 second. Is this what you mean? Yap~ You can flip screen between 0 and 180 orientation. This issue will be more visible and reproducible. And after orientation changes, sometimes user even have to touch the screen to trigger screen update.
Assignee | ||
Comment 10•10 years ago
|
||
This patch is for 1.1 only.
Attachment #795234 -
Flags: review?(mwu)
Comment 11•10 years ago
|
||
Leo has stopped accepting patches for their 1.1 build, so not going to block on this unless they explicitly ask for this to block the release.
blocking-b2g: leo? → -
Updated•10 years ago
|
Attachment #787354 -
Attachment is obsolete: true
Comment 12•10 years ago
|
||
Comment on attachment 795234 [details] [diff] [review] Remove tabs. r=me for a straightforward backport of a fix. This is probably worth landing on 1.1 if we can. If not for leo, then for other 1.1 updates.
Attachment #795234 -
Flags: review?(mwu) → review+
Comment 13•10 years ago
|
||
Dietrich, as part of our mega triage of 1.1 leo+ blockers, this bug had a vote to retain in 1.1 I'll wait for Patryk's testing and if worth while will move to a +
Comment 14•10 years ago
|
||
I am going to need more information. + how do I test this? + where is the repo? + how much faster is it (before / after) From my experience on the leo device, I don't think the rotation performance is a blocker, but if it affects all 1.1 devices, we should push this into 1.1 HD and 1.2, 100%
blocking-b2g: - → leo?
Flags: needinfo?(padamczyk)
Comment 15•10 years ago
|
||
(In reply to Patryk Adamczyk [:patryk] UX from comment #14) > I am going to need more information. > + how do I test this? > + where is the repo? > + how much faster is it (before / after) needinfo'ing vin to help with this info here. > > From my experience on the leo device, I don't think the rotation performance > is a blocker, Agreed ! > but if it affects all 1.1 devices, we should push this into > 1.1 HD and 1.2, 100% This is close to resolution on 1.2 anyway, nevertheless flipping it back to koi?
blocking-b2g: leo? → koi?
Comment 16•10 years ago
|
||
This is a backport of a fix that's already on 1.2, so no need to block on koi.
blocking-b2g: koi? → ---
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•10 years ago
|
blocking-b2g: --- → hd?
Comment 17•10 years ago
|
||
Can't land without appropriate blocking status. Please re-request checkin once it does.
Keywords: checkin-needed
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(mwu)
Assignee | ||
Comment 19•10 years ago
|
||
Please make sure the blocking status(Project Flags).
Status: NEW → ASSIGNED
Flags: needinfo?(wchang)
Comment 20•10 years ago
|
||
Let's hold this for now as we're all moving towards v1.2, unless a partner has a strong request on leo for this. Setting state according to comment 16.
blocking-b2g: hd? → ---
Flags: needinfo?(wchang)
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•