Closed
Bug 1230466
Opened 8 years ago
Closed 8 years ago
unexpected scrollbar display
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: alice0775, Assigned: bzbarsky)
References
()
Details
(Whiteboard: [parity-IE] [parity-Chrome][parity-Edge])
Attachments
(3 files, 1 obsolete file)
Steps to Reproduce: 1. Open http://www.sony.jp/ or index.html of reduced.zip Actual Results: horizontal scrollbar appears Expected Results: no horizontal scrollbar
Reporter | ||
Updated•8 years ago
|
Whiteboard: [parity-Chrome][parity-IE] → [parity-IE] [parity-Chrome][parity-Edge]
Reporter | ||
Comment 1•8 years ago
|
||
Attachment #8695774 -
Attachment is obsolete: true
Assignee | ||
Comment 2•8 years ago
|
||
Looks like the page sets "overflow:scroll" on the <body>, then flushes layout to compute some things, then removes it from the DOM. During the flush we propagate the overflow style to the viewport, then the removal still leaves it there.
Assignee | ||
Comment 3•8 years ago
|
||
Assignee | ||
Comment 4•8 years ago
|
||
Oh, in the attached zip the relevant code is in libs.js, search for "viewportSize".
Assignee | ||
Comment 5•8 years ago
|
||
Looks like we call UpdateViewportScrollbarStylesOverride only on insertions (if I ignore the fullscreen stuff), Should we add a call on frame removal too?
Flags: needinfo?(roc)
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8695956 -
Flags: review?(roc)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(roc)
Attachment #8695956 -
Flags: review?(roc) → review+
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f4eaab30e7e7
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•