diff options
| author | dimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-04 04:37:41 +0000 |
|---|---|---|
| committer | dimich@google.com <dimich@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-04 04:37:41 +0000 |
| commit | ebc4dfee97e29da7b03c9a9dda10e5fb4b8786d5 (patch) | |
| tree | 71b9eaae95a7909a5ef6e37f045cd41cfe524221 /webkit/glue/webframe_impl.cc | |
| parent | c199130efc467a32bcae3fdc1699434303bcac1c (diff) | |
| download | chromium_src-ebc4dfee97e29da7b03c9a9dda10e5fb4b8786d5.zip chromium_src-ebc4dfee97e29da7b03c9a9dda10e5fb4b8786d5.tar.gz chromium_src-ebc4dfee97e29da7b03c9a9dda10e5fb4b8786d5.tar.bz2 | |
WebKit DEP roll 44372:44405
BUG=none
TEST=no compile or layout test failures.
There is a change (http://trac.webkit.org/changeset/44401) here which will likely break quite a few Linux layout tests. The plan is to run the build, then add all the tests with REBASELINE keyword into test_expectations.txt for automated rebaselining.
Review URL: http://codereview.chromium.org/118215
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17601 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/webframe_impl.cc')
| -rw-r--r-- | webkit/glue/webframe_impl.cc | 14 |
1 files changed, 5 insertions, 9 deletions
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 |