Commit 4b118995 authored by tony@chromium.org's avatar tony@chromium.org
Browse files

2011-01-10 Tony Chang <tony@chromium.org>

        Unreviewed, rolling out r75398, r75407,
        http://trac.webkit.org/changeset/75398
        https://bugs.webkit.org/show_bug.cgi?id=52008

        Some layout tests are crashing on Chromium Win.

        * platform/chromium-win/fast/backgrounds/animated-gif-as-background-expected.checksum: Added.
        * platform/chromium-win/fast/backgrounds/animated-gif-as-background-expected.png: Added.
        * platform/chromium/test_expectations.txt:
2011-01-10  Tony Chang  <tony@chromium.org>

        Unreviewed, rolling out r75398.
        http://trac.webkit.org/changeset/75398
        https://bugs.webkit.org/show_bug.cgi?id=52008

        Some layout tests are crashing on Chromium Win.

        * DumpRenderTree/chromium/WebViewHost.cpp:
        (WebViewHost::updatePaintRect):
        * DumpRenderTree/chromium/WebViewHost.h:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@75433 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent a3be9acb
2011-01-10 Tony Chang <tony@chromium.org>
Unreviewed, rolling out r75398, r75407,
http://trac.webkit.org/changeset/75398
https://bugs.webkit.org/show_bug.cgi?id=52008
Some layout tests are crashing on Chromium Win.
* platform/chromium-win/fast/backgrounds/animated-gif-as-background-expected.checksum: Added.
* platform/chromium-win/fast/backgrounds/animated-gif-as-background-expected.png: Added.
* platform/chromium/test_expectations.txt:
2011-01-10 Mihai Parparita <mihaip@chromium.org>
 
Unreviewed Chromium test expectations update.
......
......@@ -892,6 +892,9 @@ BUGCR11153 WIN LINUX : fast/canvas/canvas-incremental-repaint.html = FAIL
BUGCR20507 : http/tests/uri/resolve-encoding-relative.html = FAIL
// test_shell, DRT do not correctly schedule paints when invalidating.
BUGCR62433 WIN : fast/images/gif-loop-count.html = IMAGE
// -----------------------------------------------------------------
// SVG TESTS
// -----------------------------------------------------------------
......@@ -2848,9 +2851,9 @@ BUGCR65279 LINUX DEBUG : fast/css/visited-link-hang.html = TEXT PASS
// Regressions caused by switching to DRT.
///////////////////////////////////////////////////////////////////////////
// These need to be rebaselined on Windows.
BUGCR62433 WIN : fast/backgrounds/animated-gif-as-background.html = IMAGE
BUGCR62433 WIN : fast/images/gif-loop-count.html = IMAGE
// DRT needs to schedule paints on invalidations like Chrome does.
BUGCR62433 MAC LINUX : fast/backgrounds/animated-gif-as-background.html = IMAGE
BUGCR62433 MAC LINUX : fast/images/gif-loop-count.html = IMAGE
// input-speech related failures
BUG_DRT WIN MAC : fast/speech/input-appearance-numberandspeech.html = IMAGE
......
2011-01-10 Tony Chang <tony@chromium.org>
Unreviewed, rolling out r75398.
http://trac.webkit.org/changeset/75398
https://bugs.webkit.org/show_bug.cgi?id=52008
Some layout tests are crashing on Chromium Win.
* DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::updatePaintRect):
* DumpRenderTree/chromium/WebViewHost.h:
2011-01-10 Adam Roben <aroben@apple.com>
 
Roll out r75392
......@@ -656,6 +656,8 @@ void WebViewHost::show(WebNavigationPolicy)
updatePaintRect(WebRect(0, 0, size.width, size.height));
}
void WebViewHost::closeWidget()
{
m_hasWindow = false;
......@@ -1405,13 +1407,6 @@ void WebViewHost::setAddressBarURL(const WebURL&)
// Painting functions ---------------------------------------------------------
class WebViewHostPaintTask : public MethodTask<WebViewHost> {
public:
WebViewHostPaintTask(WebViewHost* object)
: MethodTask<WebViewHost>(object) {}
virtual void runIfValid() { m_object->paintInvalidatedRegion(); }
};
void WebViewHost::updatePaintRect(const WebRect& rect)
{
// m_paintRect = m_paintRect U rect
......@@ -1426,8 +1421,6 @@ void WebViewHost::updatePaintRect(const WebRect& rect)
int right = max(m_paintRect.x + m_paintRect.width, rect.x + rect.width);
int bottom = max(m_paintRect.y + m_paintRect.height, rect.y + rect.height);
m_paintRect = WebRect(left, top, right - left, bottom - top);
postDelayedTask(new WebViewHostPaintTask(this), 0);
}
void WebViewHost::paintRect(const WebRect& rect)
......
......@@ -32,7 +32,6 @@
#define WebViewHost_h
#include "MockSpellCheck.h"
#include "Task.h"
#include "TestNavigationController.h"
#include "WebAccessibilityNotification.h"
#include "WebCursorInfo.h"
......@@ -101,9 +100,6 @@ class WebViewHost : public WebKit::WebViewClient, public WebKit::WebFrameClient,
WebKit::WebSpeechInputControllerMock* speechInputControllerMock() { return m_speechInputControllerMock.get(); }
// Needed by the MethodTask template.
TaskList* taskList() { return &m_taskList; }
// NavigationHost
virtual bool navigate(const TestNavigationEntry&, bool reload);
......@@ -312,8 +308,6 @@ private:
WebKit::WebRect m_paintRect;
bool m_isPainting;
TaskList m_taskList;
OwnPtr<WebKit::WebContextMenuData> m_lastContextMenuData;
// Geolocation
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment