Commit f224f15f authored by anicolao@google.com's avatar anicolao@google.com

Implement RenderWidgetHostViewViews::Destroy. See RWHVGtk for reference.

TEST=manually, trybots
BUG=none

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71968 0039d316-1c4b-4281-b951-d872f2087c98
parent 105f9327
...@@ -130,8 +130,6 @@ RenderWidgetHostViewViews::RenderWidgetHostViewViews(RenderWidgetHost* host) ...@@ -130,8 +130,6 @@ RenderWidgetHostViewViews::RenderWidgetHostViewViews(RenderWidgetHost* host)
} }
RenderWidgetHostViewViews::~RenderWidgetHostViewViews() { RenderWidgetHostViewViews::~RenderWidgetHostViewViews() {
RenderViewGone(base::TERMINATION_STATUS_NORMAL_TERMINATION,
ResultCodes::NORMAL_EXIT);
} }
void RenderWidgetHostViewViews::InitAsChild() { void RenderWidgetHostViewViews::InitAsChild() {
...@@ -293,8 +291,12 @@ void RenderWidgetHostViewViews::RenderViewGone(base::TerminationStatus status, ...@@ -293,8 +291,12 @@ void RenderWidgetHostViewViews::RenderViewGone(base::TerminationStatus status,
} }
void RenderWidgetHostViewViews::Destroy() { void RenderWidgetHostViewViews::Destroy() {
// TODO(anicolao): deal with any special popup cleanup // host_'s destruction brought us here, null it out so we don't use it
NOTIMPLEMENTED(); host_ = NULL;
if (GetParent())
GetParent()->RemoveChildView(this);
MessageLoop::current()->DeleteSoon(FROM_HERE, this);
} }
void RenderWidgetHostViewViews::SetTooltipText(const std::wstring& tip) { void RenderWidgetHostViewViews::SetTooltipText(const std::wstring& tip) {
......
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