Bug 189811

Summary: Make "overflow: overlay" a synonym for "overflow: auto"
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, koivisto, rniwa, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar, WebExposed
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 189813    
Attachments:
Description Flags
Patch
none
Patch zalan: review+

Description Simon Fraser (smfr) 2018-09-20 17:51:53 PDT
Make "overflow: overlay" a synonym for "overlay: auto"
Comment 1 Simon Fraser (smfr) 2018-09-20 17:55:45 PDT
Created attachment 350293 [details]
Patch
Comment 2 Simon Fraser (smfr) 2018-09-20 18:02:44 PDT
Created attachment 350294 [details]
Patch
Comment 3 zalan 2018-09-20 21:36:15 PDT
Comment on attachment 350294 [details]
Patch

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

> LayoutTests/compositing/overflow/overflow-overlay-with-touch-expected.txt:5
>  (GraphicsLayer
> +  (anchor 0.00 0.00)
>    (bounds 800.00 600.00)
>    (children 1
>      (GraphicsLayer

How did this change?
Comment 4 Simon Fraser (smfr) 2018-09-21 11:28:37 PDT
(In reply to zalan from comment #3)
> Comment on attachment 350294 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=350294&action=review
> 
> > LayoutTests/compositing/overflow/overflow-overlay-with-touch-expected.txt:5
> >  (GraphicsLayer
> > +  (anchor 0.00 0.00)
> >    (bounds 800.00 600.00)
> >    (children 1
> >      (GraphicsLayer
> 
> How did this change?

This test was marked as an expected failure, and the results checked in are Chromium results with layers for scrollbars. Since it's failing anyway, I'll revert and clean up separately.
Comment 5 Simon Fraser (smfr) 2018-09-21 11:43:18 PDT
https://trac.webkit.org/r236341
Comment 6 Radar WebKit Bug Importer 2018-09-21 11:44:27 PDT
<rdar://problem/44687507>