The cl to enable page-zoom for OOPIFs (out-of-process, or site-isolation, iframes) includes a change to FrameView::setFrameRect() that, although necessary, doesn't seem like it *should* be necessary.

Without the change, scrollbars for subframes on *some* pages fail to update properly; they resize correctly, but fail to re-position themselves. This seems to happen reliably on complicated pages, like slashdot.org, but does not seem to occur on very simple pages.

The change forces updateScrollbarsIfNeeded() to always be called, when previously it was only called if needsLayout() was false. I suspect that this means that layout decides that everything is just scaling up, and thinks no changes are necessary, and possibly early outs before doing the scrollbars.

Calling updateScrollbarsIfNeeded() seems inexpensive and therefore is fine as a temporary fix, but it does point out some underlying layout issue may be present.