diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-13 19:49:47 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-13 19:49:47 +0000 |
| commit | f8d1a1f86c3d5fec1a7943eb417b971b09fb1900 (patch) | |
| tree | dc8fc267a03bef1e3436bb672cab21b1f8223448 /webkit/glue/webframeloaderclient_impl.cc | |
| parent | 84140f23d6732ba99dd147bc71ae0a044549c50c (diff) | |
| download | chromium_src-f8d1a1f86c3d5fec1a7943eb417b971b09fb1900.zip chromium_src-f8d1a1f86c3d5fec1a7943eb417b971b09fb1900.tar.gz chromium_src-f8d1a1f86c3d5fec1a7943eb417b971b09fb1900.tar.bz2 | |
Fix LayoutTests/http/tests/loading/gmail-assert-on-load.html test.
The trick here is to properly check for a Frame that has been removed by
checking for a null Page. This matches what happens in other WebKit ports.
R=dglazkov
Review URL: http://codereview.chromium.org/10882
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5359 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/webframeloaderclient_impl.cc')
| -rw-r--r-- | webkit/glue/webframeloaderclient_impl.cc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/webkit/glue/webframeloaderclient_impl.cc b/webkit/glue/webframeloaderclient_impl.cc index 946e784..7d7bcb2 100644 --- a/webkit/glue/webframeloaderclient_impl.cc +++ b/webkit/glue/webframeloaderclient_impl.cc @@ -396,14 +396,13 @@ bool WebFrameLoaderClient::dispatchDidLoadResourceFromMemoryCache( } void WebFrameLoaderClient::dispatchDidHandleOnloadEvents() { - WebViewImpl* webview = webframe_->webview_impl(); // During the onload event of a subframe, the subframe can be removed. In - // that case, we have no webview. This is covered by + // that case, it has no page. This is covered by // LayoutTests/fast/dom/replaceChild.html - if (!webview) + if (!webframe_->frame()->page()) return; + WebViewImpl* webview = webframe_->webview_impl(); WebViewDelegate* d = webview->delegate(); - if (d) d->DidHandleOnloadEventsForFrame(webview, webframe_); } |