742447 - Scrolling with the keyboard does not work

Status: RESOLVED WORKSFORME

(Core :: Widget: Win32, defect)

Description

[Brian R. Bondy [:bbondy]]

Page up, Page down, up arrow and down arrow should scroll the page. 
I verified and the keyboard shortcuts actually do work.

I did some preliminary debugging and found that the reason is because of:

> nsRect
> nsGfxScrollFrameInner::GetScrollRange() const
> {
>  nsRect range = GetScrolledRect();
>  range.width -= mScrollPort.width;
>  range.height -= mScrollPort.height;

range == mScrollPort, but that shouldn't be the case.

Something isn't setup right but this has nothing to do with keyboard handling.

Comment 1

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

I'll take a look

Comment 2

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

This issue doesn't reproduce in my local builds running on this machine:
  Win8 RTM
  Built with VS2012 RTM
  Samsung Series 7
  Attached USB keyboard

I spoke with bbondy in IRC; he mentioned that a lot of keyboard shortcut fixes and widget keyboard changes have gone in since this bug was filed so it's probably been fixed.  Let's reopen if this reappears.