Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work

RESOLVED FIXED in Firefox 27

Status

defect
P2
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: emtwo, Assigned: kats)

Tracking

({testcase-wanted})

unspecified
Firefox 27
x86_64
Windows 8.1
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: feature=defect c=tbd u=tbd p=0)

Attachments

(3 attachments)

Reporter

Description

6 years ago
No description provided.
Summary: Touch scrolling for Facebook's side newsfeeds and chat boxes does not work → Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work
Whiteboard: [preview-triage] feature=defect c=tbd u=tbd p=0
Summary: Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work → [MP] Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work
Whiteboard: [preview-triage] feature=defect c=tbd u=tbd p=0 → [preview] feature=defect c=tbd u=tbd p=0

Updated

6 years ago
Depends on: 901607
Reporter

Comment 1

6 years ago
cc-ing kats for subframe scrolling issues. I noticed that the gtalk list on the right in gmail seems to be a similar issue.
No longer blocks: MetroPreviewRelease
Summary: [MP] Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work → Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work
Whiteboard: [preview] feature=defect c=tbd u=tbd p=0 → feature=defect c=tbd u=tbd p=0
Do you have a reduced testcase for this issue? Debugging it will require digging into layout code and it's much harder to do that on pages as complex as gmail/facebook.
Flags: needinfo?(msamuel)
Component: Input → Pan and Zoom
Keywords: testcase-wanted
OS: Windows 8 → Windows 8 Metro
Reporter

Comment 3

6 years ago
Not sure if this is what you're looking for but pasting https://pastebin.mozilla.org/3219265 or https://pastebin.mozilla.org/3219264 into an html file and opening it in metro doesn't let me touch scroll by dragging but I can scroll via the scrollbars.
Flags: needinfo?(msamuel)
Attachment #814562 - Attachment mime type: text/plain → text/html
Comment on attachment 814563 [details]
Test case 2 from comment 3

Yup, that's exactly what I'm looking for. Thanks! I'll updated my Metro build and see if I can get to the bottom of this.
Attachment #814563 - Attachment mime type: text/plain → text/html
Assignee: nobody → bugmail.mozilla
So this fails for the very simple reason that sub-frame APZC is still disabled on non-content processes :/

This code needs to updated so that it returns true for metro:
http://mxr.mozilla.org/mozilla-central/source/layout/generic/nsGfxScrollFrame.cpp?rev=45d9e6cd3473#2355
This enables scrolling in the test case on metro. Dragging the scrollbar results in badness but presumably the scrollbar will be removed.
Attachment #819134 - Flags: review?(tnikkel)
Attachment #819134 - Flags: review?(tnikkel) → review+
Blocks: metrov1it17
No longer blocks: metrov1backlog
Status: NEW → ASSIGNED
Priority: -- → P2
QA Contact: jbecerra
https://hg.mozilla.org/mozilla-central/rev/31121f69414d
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.