2008-11-03 Alp Toker <alp@nuanti.com>

        Reviewed by Holger Freyther.

        Redundant scrollbars appear in frames where they shouldn't be visible
        eg. embedded Google adverts.

        Call setCanHaveScrollbars() when necessary. The Mac and Win ports
        already have this but it was missing in FrameLoaderClientGtk.

        * WebCoreSupport/FrameLoaderClientGtk.cpp:
        (WebKit::FrameLoaderClient::transitionToCommittedForNewPage):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@38074 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/gtk/ChangeLog

+2008-11-03  Alp Toker  <alp@nuanti.com>
+
+        Reviewed by Holger Freyther.
+
+        Redundant scrollbars appear in frames where they shouldn't be visible
+        eg. embedded Google adverts.
+
+        Call setCanHaveScrollbars() when necessary. The Mac and Win ports
+        already have this but it was missing in FrameLoaderClientGtk.
+
+        * WebCoreSupport/FrameLoaderClientGtk.cpp:
+        (WebKit::FrameLoaderClient::transitionToCommittedForNewPage):
+
 2008-11-03  Alp Toker  <alp@nuanti.com>
 
         Rubber-stamped by Holger Freyther.

WebKit/gtk/WebCoreSupport/FrameLoaderClientGtk.cpp

@@ -787,8 +787,8 @@ void FrameLoaderClient::transitionToCommittedForNewPage()
     if (frame->ownerRenderer())
         frame->ownerRenderer()->setWidget(frameView);
 
-    if (!frame->ownerElement())
-        return;
+    if (HTMLFrameOwnerElement* owner = frame->ownerElement())
+        frame->view()->setCanHaveScrollbars(owner->scrollingMode() != ScrollbarAlwaysOff);
 }
 
 }