From: Gyuyoung Kim <gyuyoung.kim@samsung.com>
Date: Wed, 3 Jul 2013 06:21:08 +0000 (+0900)
Subject: Fix horizontal scroll issue
X-Git-Tag: submit/tizen_2.2/20130714.131554~55
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dbb27f56bf77351d7cf2a51e5007c887689cd2e4;p=framework%2Fweb%2Fwebkit-efl.git

Fix horizontal scroll issue

[Issue#] N_SE-41768
[Problem] A page doesn't be scrolled on skype.com after logging in.
[Cause] A workaround patch of Viewport meta tag influences on this issue.
[Solution] Change existing condition for this issue

Change-Id: Id0d785c8b59bd3aee24d32a021fbf758fece4577
---

diff --git a/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp b/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp
index acb4939..5d47646 100755
--- a/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp
+++ b/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp
@@ -506,6 +506,7 @@ PlatformPageClient WebChromeClient::platformPageClient() const
 }
 
 #if ENABLE(TIZEN_VIEWPORT_META_TAG)
+static const int landscapeWidth = 640;
 bool WebChromeClient::canContentsSizeChange(Frame* frame, const IntSize& size) const
 {
     // FIXME1: This patch should be removed AS SOON AS unexpected layout change problem is fixed.
@@ -515,13 +516,9 @@ bool WebChromeClient::canContentsSizeChange(Frame* frame, const IntSize& size) c
     // ex) www.adobe.com - fixedLayoutSize.width: 360, changedContentsSize: 950
     // ex) www.mt.co.kr (desktop site) - fixedLayoutSize.width: 980, changedContentsSize: 1090
     FrameView* view = m_page->mainFrame()->view();
-    if (view->fixedLayoutSize().width() != size.width()
-        && size.width() < m_page->viewportSize().width()
-        // This condition is for regression by this work-around patch.
-        // http://en.wordpress.com/wp-login.php?action=lostpassword site doesn't work by this patch.
-        // This patch should be removed as soon as possible.
-        && m_page->corePage()->viewportArguments().initialScale > 0.9)
+    if (view->fixedLayoutSize().width() < size.width() && size.width() < landscapeWidth)
         return false;
+
     return true;
 }
 #endif