Bug 81749

Summary: [chromium] ScrollbarLayerChromiumTest.resolveScrollLayerPointer failing on Lion
Product: WebKit Reporter: Keishi Hattori <keishi>
Component: Tools / TestsAssignee: Keishi Hattori <keishi>
Status: RESOLVED WONTFIX    
Severity: Normal CC: enne, eric, jamesr, mitz, rniwa, schenney, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Keishi Hattori 2012-03-21 02:31:34 PDT 
ScrollbarLayerChromiumTest.resolveScrollLayerPointer failing on Lion
Comment 1 Keishi Hattori 2012-03-21 02:37:47 PDT 
Created attachment 132998 [details]
Patch
Comment 2 Ryosuke Niwa 2012-03-21 22:12:46 PDT 
Comment on attachment 132998 [details]
Patch

There's no way to test this?
Comment 3 James Robinson 2012-03-22 14:57:24 PDT 
Comment on attachment 132998 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=132998&action=review

Oh! That is a somewhat embarrassing bug.

Ryosuke - we do have a test for this (although it's not very direct) and it's failing currently.

> Source/WebCore/ChangeLog:6
> +        Reviewed by NOBODY (OOPS!).

would you mind documenting the test that this fixes (I believe it only impacts Lion since we force non-overlay scrollbars in tests on Snow Leopard, and I suppose that isn't true on Lion).
Comment 4 Ryosuke Niwa 2012-03-22 15:06:21 PDT 
(In reply to comment #3)
> (From update of attachment 132998 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=132998&action=review
> 
> Oh! That is a somewhat embarrassing bug.
> 
> Ryosuke - we do have a test for this (although it's not very direct) and it's failing currently.

That's good to know. Does the test pass after this change?