MainFrameObserver is inline owned, so it shouldn't be freed by OnDestruct

content/renderer/web_ui_mojo.h

Index: content/renderer/web_ui_mojo.h
diff --git a/content/renderer/web_ui_mojo.h b/content/renderer/web_ui_mojo.h
index f7179ef5d816447e1f314b56cdb14966d7e5fe53..bfc4eb50575e03ddb437b86c8409b26a14ebcc64 100644
--- a/content/renderer/web_ui_mojo.h
+++ b/content/renderer/web_ui_mojo.h
@@ -40,6 +40,7 @@ class WebUIMojo
     void WillReleaseScriptContext(v8::Local<v8::Context> context,
                                   int world_id) override;
     void DidFinishDocumentLoad() override;
+    void OnDestruct() override {}

[sky, 2015/05/08 16:28:38]

Style guide says we shouldn't inline this, also, it would be good to add the
comment you have in the description as to why this does nothing.

[nasko, 2015/05/08 17:28:50]

Done.

 
    private:
     WebUIMojo* web_ui_mojo_;