Makes RenderFrameObserver/RenderViewObserver::OnDestruct pure virtual.
All subclasses must implement the method, even if it should be empty.
This would ensure the compiler makes a developer aware of the lifecycle
concerns around RenderFrame/RenderView, and avoid the unintended
transfering ownership when inheriting the observer interface.
BUG=616214
Review-Url: https://codereview.chromium.org/2034153002
Cr-Commit-Position: refs/heads/master@{#398395}
diff --git a/android_webview/renderer/aw_render_frame_ext.h b/android_webview/renderer/aw_render_frame_ext.h
index 4bf7ec1..6551c733 100644
--- a/android_webview/renderer/aw_render_frame_ext.h
+++ b/android_webview/renderer/aw_render_frame_ext.h
@@ -36,6 +36,7 @@
bool OnMessageReceived(const IPC::Message& message) override;
void FocusedNodeChanged(const blink::WebNode& node) override;
+ void OnDestruct() override;
void OnDocumentHasImagesRequest(uint32_t id);
void OnDoHitTest(const gfx::PointF& touch_center,
