diff options
| author | nick@chromium.org <nick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-06 18:14:28 +0000 |
|---|---|---|
| committer | nick@chromium.org <nick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-06 18:14:28 +0000 |
| commit | 2a3e3c054891c865540524beb4af96ec68c481ba (patch) | |
| tree | ec3887c1f2e6056a8efd89b4d50fa46a31c03374 /chrome/browser/views/dom_view.cc | |
| parent | 4d562dbd909cd5d3c5ee39a7c399a975f9f3156a (diff) | |
| download | chromium_src-2a3e3c054891c865540524beb4af96ec68c481ba.zip chromium_src-2a3e3c054891c865540524beb4af96ec68c481ba.tar.gz chromium_src-2a3e3c054891c865540524beb4af96ec68c481ba.tar.bz2 | |
DOMView: avoid double-detaching.
I'm seeing ~DOMView happen after NativeViewHost::Detach has already been
done as part of WM_DESTROY.
BUG=20339
TEST=cancel the sync login dialog and create app shortcuts dialogs.
Review URL: http://codereview.chromium.org/258028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28129 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/dom_view.cc')
| -rw-r--r-- | chrome/browser/views/dom_view.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/views/dom_view.cc b/chrome/browser/views/dom_view.cc index a1bf0dc..cd3dc8f 100644 --- a/chrome/browser/views/dom_view.cc +++ b/chrome/browser/views/dom_view.cc @@ -12,7 +12,7 @@ DOMView::DOMView() : initialized_(false), tab_contents_(NULL) { } DOMView::~DOMView() { - if (tab_contents_.get()) + if (native_view()) Detach(); } |