Make sure FrameScheduler is cleared when LocalFrame::detach() finishes

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 90b315142fcc9d574e24892217ac080802a8d9ef..893741919788fa153a89c79598e759c468379fcb 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -1188,7 +1188,7 @@ void RenderFrameImpl::Initialize() {
 
 void RenderFrameImpl::InitializeBlameContext(RenderFrameImpl* parent_frame) {
   DCHECK(!blame_context_);
-  blame_context_ = new FrameBlameContext(this, parent_frame);
+  blame_context_ = base::WrapUnique(new FrameBlameContext(this, parent_frame));
   blame_context_->Initialize();
 }
 
@@ -2638,7 +2638,7 @@ blink::WebCookieJar* RenderFrameImpl::cookieJar() {
 
 blink::BlameContext* RenderFrameImpl::frameBlameContext() {
   DCHECK(blame_context_);
-  return blame_context_;
+  return blame_context_.get();

[Primiano Tucci (use gerrit), 2016/06/10 07:50:35]

how do you know that this is the right thread (The one where blamecontext lives)
and that weakptr.get() is safe?

[Xiaocheng, 2016/06/10 08:07:08]

blame_context_ is an std::unique_ptr so we are safe here.

 }
 
 blink::WebServiceWorkerProvider*