New issue
Advanced search Search tips

Issue 906302 link

Starred by 2 users

Issue metadata

Status: Duplicate
Owner:
Closed: Nov 19
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

DCHECK in virtual/android/rootscroller/position-fixed-in-unscrollable-document-iframe.html

Project Member Reported by atotic@chromium.org, Nov 17

Issue description

Discovered this while debugging scrolling of position:fixed for 
LayoutNG. Also happens when running in Legacy code.

To reproduce:

- launch content_shell in debugger
- load virtual/android/rootscroller/position-fixed-in-unscrollable-document-iframe.html
- use pageUp/pageDown to scroll past the end of scroll area
- hit DCHECK(scrollable_area) in scroll_manager.cc:276
- continue and crash dereferencing null scrollable_area

My guess is that pageUp/pageDown is trying to scroll past the end of the page when it should not, and scrollArea is just not there.
 
Status: Assigned (was: Untriaged)
Components: -Blink>Layout Blink>Scroll
Owner: bokan@chromium.org
atotic@, remember the stack trace? I'm guessing this is probably related to issue 904247 which looks something like this:

blink::ScrollableArea::ScrollStep(blink::WebScrollGranularity,blink::ScrollbarOrientation)
blink::ScrollManager::LogicalScroll(blink::WebScrollDirection,blink::WebScrollGranularity,blink::Node *,blink::Node *)
blink::ScrollManager::BubblingScroll(blink::WebScrollDirection,blink::WebScrollGranularity,blink::Node *,blink::Node *)
blink::EventHandler::BubblingScroll(blink::WebScrollDirection,blink::WebScrollGranularity,blink::Node *)
blink::LocalFrame::BubbleLogicalScrollFromChildFrame(blink::WebScrollDirection,blink::WebScrollGranularity,blink::Frame *)
blink::ScrollManager::BubblingScroll(blink::WebScrollDirection,blink::WebScrollGranularity,blink::Node *,blink::Node *)
blink::KeyboardEventManager::DefaultArrowEventHandler(blink::KeyboardEvent *,blink::Node *)

I've got a fix going through the trybots now.
Mergedinto: 904247
Status: Duplicate (was: Assigned)
The stack looks just like that.

Sign in to add a comment