summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-06-04 06:56:49 +0000
committerdimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-06-04 06:56:49 +0000
commit64367c0e4aedaaf41f168f3e46aabf034d16d95c (patch)
treedeb00dae0ccc78492cfd13f343cd76f55e0204ea
parent16882fc997ef4fb4d945beb0f60497f0873fc99b (diff)
downloadchromium_src-64367c0e4aedaaf41f168f3e46aabf034d16d95c.zip
chromium_src-64367c0e4aedaaf41f168f3e46aabf034d16d95c.tar.gz
chromium_src-64367c0e4aedaaf41f168f3e46aabf034d16d95c.tar.bz2
WebKit 44372:44405, resubmit.
Fixed Mac build (needed to remove a file that was removed upstream). Added expected Linux layout tests failures from the first landing attempt. Removing the chromium-specific snapshot for list-wrapping-image-crash-expected.html test because the test was incorrectly added upstream. Will fix it upstream and then re-enable in Chromium. Original code review: http://codereview.chromium.org/118215 TBR=ukai BUG=13305, 13313, 13314 (layout tests - reenable, rebaseline) TEST=none Review URL: http://codereview.chromium.org/119156 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17616 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--DEPS2
-rw-r--r--webkit/data/layout_tests/platform/chromium-mac/LayoutTests/editing/execCommand/list-wrapping-image-crash-expected.txt2
-rw-r--r--webkit/glue/webframe_impl.cc14
-rw-r--r--webkit/glue/webwidget_impl.h2
-rw-r--r--webkit/tools/layout_tests/test_expectations.txt28
-rw-r--r--webkit/webkit.gyp2
6 files changed, 35 insertions, 15 deletions
diff --git a/DEPS b/DEPS
index 75d988d..574ad6f 100644
--- a/DEPS
+++ b/DEPS
@@ -1,7 +1,7 @@
vars = {
"webkit_trunk":
"http://svn.webkit.org/repository/webkit/trunk",
- "webkit_revision": "44372",
+ "webkit_revision": "44405",
}
diff --git a/webkit/data/layout_tests/platform/chromium-mac/LayoutTests/editing/execCommand/list-wrapping-image-crash-expected.txt b/webkit/data/layout_tests/platform/chromium-mac/LayoutTests/editing/execCommand/list-wrapping-image-crash-expected.txt
deleted file mode 100644
index 90a0a2e..0000000
--- a/webkit/data/layout_tests/platform/chromium-mac/LayoutTests/editing/execCommand/list-wrapping-image-crash-expected.txt
+++ /dev/null
@@ -1,2 +0,0 @@
-CONSOLE MESSAGE: line 9: Uncaught Error: HIERARCHY_REQUEST_ERR: DOM Exception 3
-
diff --git a/webkit/glue/webframe_impl.cc b/webkit/glue/webframe_impl.cc
index 6d71e5a..0d78729 100644
--- a/webkit/glue/webframe_impl.cc
+++ b/webkit/glue/webframe_impl.cc
@@ -94,6 +94,7 @@ MSVC_PUSH_WARNING_LEVEL(0);
#include "GraphicsContext.h"
#include "HTMLCollection.h"
#include "HTMLHeadElement.h"
+#include "HTMLFrameOwnerElement.h"
#include "HTMLLinkElement.h"
#include "HTMLNames.h"
#include "HistoryItem.h"
@@ -184,7 +185,6 @@ using WebCore::FrameLoadType;
using WebCore::FrameTree;
using WebCore::FrameView;
using WebCore::HistoryItem;
-using WebCore::HTMLFrameElementBase;
using WebCore::IntRect;
using WebCore::KURL;
using WebCore::Node;
@@ -1455,12 +1455,12 @@ void WebFrameImpl::CreateFrameView() {
WebViewImpl* web_view = GetWebViewImpl();
- WebCore::FrameView* view;
+ RefPtr<WebCore::FrameView> view;
if (is_main_frame) {
IntSize size = webkit_glue::WebSizeToIntSize(web_view->size());
- view = new FrameView(frame_, size);
+ view = FrameView::create(frame_, size);
} else {
- view = new FrameView(frame_);
+ view = FrameView::create(frame_);
}
frame_->setView(view);
@@ -1471,7 +1471,7 @@ void WebFrameImpl::CreateFrameView() {
// TODO(darin): The Mac code has a comment about this possibly being
// unnecessary. See installInFrame in WebCoreFrameBridge.mm
if (frame_->ownerRenderer())
- frame_->ownerRenderer()->setWidget(view);
+ frame_->ownerRenderer()->setWidget(view.get());
if (HTMLFrameOwnerElement* owner = frame_->ownerElement()) {
view->setCanHaveScrollbars(
@@ -1480,10 +1480,6 @@ void WebFrameImpl::CreateFrameView() {
if (is_main_frame)
view->setParentVisible(true);
-
- // FrameViews are created with a refcount of 1 so it needs releasing after we
- // assign it to a RefPtr.
- view->deref();
}
// static
diff --git a/webkit/glue/webwidget_impl.h b/webkit/glue/webwidget_impl.h
index 6d889b0..a0078db 100644
--- a/webkit/glue/webwidget_impl.h
+++ b/webkit/glue/webwidget_impl.h
@@ -27,7 +27,7 @@ namespace WebKit {
class WebKeyboardEvent;
class WebMouseEvent;
class WebMouseWheelEvent;
-class WebRect;
+struct WebRect;
}
struct MenuItem;
diff --git a/webkit/tools/layout_tests/test_expectations.txt b/webkit/tools/layout_tests/test_expectations.txt
index 12ac343..472a77a 100644
--- a/webkit/tools/layout_tests/test_expectations.txt
+++ b/webkit/tools/layout_tests/test_expectations.txt
@@ -2710,3 +2710,31 @@ BUG13267 LINUX : LayoutTests/fast/overflow/overflow-rtl-inline-scrollbar.html =
BUG13267 LINUX : LayoutTests/svg/custom/mask-inside-defs.svg = PASS FAIL
BUG13267 LINUX : LayoutTests/svg/custom/stroked-pattern.svg = PASS FAIL
BUG13267 LINUX : LayoutTests/editing/selection/5131716-3.html = PASS FAIL
+
+// WebKit change http://trac.webkit.org/changeset/44375
+// added this test with wrong expected file. Will fix upstream and enable.
+BUG13305 : LayoutTests/editing/execCommand/list-wrapping-image-crash.html = FAIL
+
+// WebKit 44372:44405
+// Due to change http://trac.webkit.org/changeset/44401 Linux renders scrollbars differently.
+// The following tests failed and need rebaseline.
+BUG13313 LINUX REBASELINE : LayoutTests/fast/css/line-height-overflow.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/flexbox/009.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/frames/invalid.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/frames/valid.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/overflow/unreachable-overflow-rtl-bug.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/repaint/overflow-scroll-delete.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/svg/hixie/links/003-broken.xml = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/tables/mozilla/bugs/bug113235-1.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/tables/mozilla/core/cell_heights.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/css/line-height-overflow.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/flexbox/009.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/frames/invalid.html = FAIL
+BUG13313 LINUX REBASELINE : LayoutTests/fast/frames/valid.html = FAIL
+
+// WebKit 44372:44405
+// New Linux crashes.
+BUG13314 LINUX : LayoutTests/css2.1/atrule_longest_match.html = CRASH
+BUG13314 LINUX : LayoutTests/editing/deleting/25322-1.html = CRASH
+BUG13314 LINUX : LayoutTests/platform/mac/accessibility/accesskey.html = CRASH
+BUG13314 LINUX : LayoutTests/svg/W3C-SVG-1.1/animate-elem-02-t.svg = CRASH
diff --git a/webkit/webkit.gyp b/webkit/webkit.gyp
index f1ac224..41acb49 100644
--- a/webkit/webkit.gyp
+++ b/webkit/webkit.gyp
@@ -2624,8 +2624,6 @@
'../third_party/WebKit/WebCore/platform/mac/WebCoreObjCExtras.mm',
'../third_party/WebKit/WebCore/platform/mac/WebCoreSystemInterface.h',
'../third_party/WebKit/WebCore/platform/mac/WebCoreSystemInterface.mm',
- '../third_party/WebKit/WebCore/platform/mac/WebCoreTextRenderer.h',
- '../third_party/WebKit/WebCore/platform/mac/WebCoreTextRenderer.mm',
'../third_party/WebKit/WebCore/platform/mac/WebCoreView.h',
'../third_party/WebKit/WebCore/platform/mac/WebCoreView.m',
'../third_party/WebKit/WebCore/platform/mac/WebFontCache.h',