Commit f2780a08 authored by sadrul@chromium.org's avatar sadrul@chromium.org

aura: Do not attempt to remove the window multiple times on detach.

The window gets removed in RemovedFromWidget, so it is not necessary to do
this again in NativeViewDetaching. This fixes a couple of failing DOMViewTest
tests.

BUG=103487
TEST=DOMViewTest.TestRemoveAndDelete, TestReparent (browser_tests)

Review URL: http://codereview.chromium.org/8574016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110198 0039d316-1c4b-4281-b951-d872f2087c98
parent 330bc47b
......@@ -35,7 +35,6 @@ void NativeViewHostAura::NativeViewDetaching(bool destroyed) {
if (!destroyed) {
host_->native_view()->RemoveObserver(this);
host_->native_view()->Hide();
host_->GetWidget()->GetNativeView()->RemoveChild(host_->native_view());
}
}
......
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