http://trac.webkit.org/changeset/79053 was a fix for a long-standing WebKit bug. Hyatt proposed that we actually fix it in a different way though. I will soon post a patch that rolls out the original fix and implements Hyatt's suggested fix. <rdar://problem/9026946> Oh, and to reproduce: 1. Visit any page, say nytimes.com 2. Now visit another page -- one that you know will go into the back-forward cache. I visited a local test file I have that is just text inside a <body> with a fixed width. Make sure the window is narrow enough to have a horizontal scrollbar. 3. Click the back button. 4. Click the forward button. The bug was that the horizontal scrollbar would not be there even though scrolling horizontally was possible.
Created attachment 87242 [details] Patch
Comment on attachment 87242 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=87242&action=review > Source/WebCore/dom/Document.cpp:3947 > + v->resetScrollbars(FrameView::ClearContentsSize); Since the work to clear the contents size is completely separate, why not add a separate function call instead of having an enum to add more work onto this function?
(In reply to comment #2) > (From update of attachment 87242 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=87242&action=review > > > Source/WebCore/dom/Document.cpp:3947 > > + v->resetScrollbars(FrameView::ClearContentsSize); > > Since the work to clear the contents size is completely separate, why not add a separate function call instead of having an enum to add more work onto this function? I took Darin's advice and added a separate function. He gave me an r=him in person, so I am going mark the + flag.
Fixed with r 82185.
This bug was fixed, Currently EFL Port Page Cache was disabled. so I try to enable EFL Page Cache Function. but Horizontal Scroll Bar was not displayed. I want to know, why below code was deleted. if (!m_isRestoringFromBackForward) setScrollbarsSuppressed(true); else { setScrollbarsSuppressed(false); m_isRestoringFromBackForward = false; }