Remove work-around patch for naver
authorGyuyoung Kim <gyuyoung.kim@samsung.com>
Mon, 15 Jul 2013 09:52:44 +0000 (18:52 +0900)
committerGerrit Code Review <gerrit@gerrit.vlan144.tizendev.org>
Mon, 15 Jul 2013 10:01:54 +0000 (10:01 +0000)
[Title] Remove work-around patch for naver
[Issue#] N_SE-46200
[Problem] Browser comes to crash when url is null in a work-around patch.
[Cause] There is no null check
[Solution] Remove it because the patch is not needed anymore

Change-Id: I4a946fa3c23b10188f601f5e1fe5df5f90e3a9e3

Source/WebCore/loader/EmptyClients.h
Source/WebCore/page/Chrome.cpp
Source/WebCore/page/Chrome.h
Source/WebCore/page/ChromeClient.h
Source/WebCore/page/FrameView.cpp

index 5fdf20e..eb85774 100755 (executable)
@@ -169,11 +169,6 @@ public:
     virtual PlatformPageClient platformPageClient() const { return 0; }
     virtual void contentsSizeChanged(Frame*, const IntSize&) const { }
 
-#if ENABLE(TIZEN_VIEWPORT_META_TAG)
-    virtual bool canContentsSizeChange(Frame*, const IntSize&) const { return true; }
-#endif
-
-
     virtual void scrollbarsModeDidChange() const { }
     virtual void mouseDidMoveOverElement(const HitTestResult&, unsigned) { }
 
index 3abff29..b18f505 100644 (file)
@@ -131,13 +131,6 @@ PlatformPageClient Chrome::platformPageClient() const
     return m_client->platformPageClient();
 }
 
-#if ENABLE(TIZEN_VIEWPORT_META_TAG)
-bool Chrome::canContentsSizeChange(Frame* frame, const IntSize& size) const
-{
-    return m_client->canContentsSizeChange(frame, size);
-}
-#endif
-
 void Chrome::contentsSizeChanged(Frame* frame, const IntSize& size) const
 {
     m_client->contentsSizeChanged(frame, size);
index 4a0285b..970191e 100644 (file)
@@ -95,10 +95,6 @@ namespace WebCore {
 
         void scrollRectIntoView(const IntRect&) const;
 
-#if ENABLE(TIZEN_VIEWPORT_META_TAG)
-        bool canContentsSizeChange(Frame*, const IntSize&) const;
-#endif
-
         void contentsSizeChanged(Frame*, const IntSize&) const;
         void layoutUpdated(Frame*) const;
 
index 7775f0b..9b01a21 100755 (executable)
@@ -188,10 +188,6 @@ namespace WebCore {
 
         virtual void dispatchViewportPropertiesDidChange(const ViewportArguments&) const { }
 
-#if ENABLE(TIZEN_VIEWPORT_META_TAG)
-        virtual bool canContentsSizeChange(Frame*, const IntSize&) const = 0;
-#endif
-
         virtual void contentsSizeChanged(Frame*, const IntSize&) const = 0;
         virtual void layoutUpdated(Frame*) const { }
         virtual void scrollRectIntoView(const IntRect&) const { }; // Currently only Mac has a non empty implementation.
index fac4c62..8c42246 100755 (executable)
@@ -533,32 +533,14 @@ void FrameView::setContentsSize(const IntSize& size)
     if (size == contentsSize())
         return;
 
-#if ENABLE(TIZEN_VIEWPORT_META_TAG)
-    Page* page = frame() ? frame()->page() : 0;
-    if (!page)
-        return;
-
-    // According to decision of port, content size can be changed.
-    if (!page->chrome()->canContentsSizeChange(frame(), size)) {
-        // FIXME: This patch should be removed when unexpected layout change problem is fixed.
-        m_deferSetNeedsLayouts++;
-        ScrollView::setContentsSize(IntSize(fixedLayoutSize().width(), size.height()));
-    } else {
-        m_deferSetNeedsLayouts++;
-        ScrollView::setContentsSize(size);
-    }
-#else
     m_deferSetNeedsLayouts++;
     ScrollView::setContentsSize(size);
-#endif
 
     ScrollView::contentsResized();
 
-#if !ENABLE(TIZEN_VIEWPORT_META_TAG)
     Page* page = frame() ? frame()->page() : 0;
     if (!page)
         return;
-#endif
 
     updateScrollableAreaSet();