Bug 143360 – Scrollbars are left in the wrong position when resizing a fixed layout view

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Tim Horton]

Scrollbars are left in the wrong position when switching into fixed layout

[Tim Horton]

Created attachment 250040 [details]
Patch

[Tim Horton]

http://trac.webkit.org/changeset/182307

[Csaba Osztrogonác]

(In reply to comment #2)
> http://trac.webkit.org/changeset/182307

This new test is flakey on the Mac WK2 EWS. 
It reported a false failure in bug127428.

[Tim Horton]

(In reply to comment #3)
> (In reply to comment #2)
> > http://trac.webkit.org/changeset/182307
> 
> This new test is flakey on the Mac WK2 EWS. 
> It reported a false failure in bug127428.

Interesting. And it's the viewport-units part of the test that is broken, not anything actually important :( I'll try to figure it out in a bit.

[Brent Fulgham]

(In reply to comment #4)
> (In reply to comment #3)
> > (In reply to comment #2)
> > > http://trac.webkit.org/changeset/182307
> > 
> > This new test is flakey on the Mac WK2 EWS. 
> > It reported a false failure in bug127428.
> 
> Interesting. And it's the viewport-units part of the test that is broken,
> not anything actually important :( I'll try to figure it out in a bit.

It is acting as if the 'resizeTo' happens over some period of time (is it animated)?

Also, I'm confused by the expected test results. How does resizeTo(200, 200) result in a test expectation of 100x100? Is this a 2x resolution issue? But if so, why is the overall layer size correct (400x400)?

layer at (0,0) size 400x400
  RenderBlock (positioned) {DIV} at (0,0) size 400x400 [bgcolor=#008000]
layer at (0,0) size 100x100
  RenderBlock (positioned) {DIV} at (0,0) size 100x100 [bgcolor=#0000FF]

Very confusing!

[Tim Horton]

(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > (In reply to comment #2)
> > > > http://trac.webkit.org/changeset/182307
> > > 
> > > This new test is flakey on the Mac WK2 EWS. 
> > > It reported a false failure in bug127428.
> > 
> > Interesting. And it's the viewport-units part of the test that is broken,
> > not anything actually important :( I'll try to figure it out in a bit.
> 
> It is acting as if the 'resizeTo' happens over some period of time (is it
> animated)?
> 
> Also, I'm confused by the expected test results. How does resizeTo(200, 200)
> result in a test expectation of 100x100? Is this a 2x resolution issue? But
> if so, why is the overall layer size correct (400x400)?
> 
> layer at (0,0) size 400x400
>   RenderBlock (positioned) {DIV} at (0,0) size 400x400 [bgcolor=#008000]
> layer at (0,0) size 100x100
>   RenderBlock (positioned) {DIV} at (0,0) size 100x100 [bgcolor=#0000FF]
> 
> Very confusing!

Not confusing. Fixed layout size of 400 square, viewport size of 200 square. That div is specified as 50 viewport units square, so 100px.

[Brent Fulgham]

> > Very confusing!
> 
> Not confusing. Fixed layout size of 400 square, viewport size of 200 square.
> That div is specified as 50 viewport units square, so 100px.

Not confusing now! :-)