Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(286)

Unified Diff: content/browser/android/java/gin_java_bridge_message_filter.h

Issue 1547043002: [Android] Fix GinJavaBridgeMessageFilter registration issue in Java Bridge. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addresses review inputs. Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/android/java/gin_java_bridge_message_filter.h
diff --git a/content/browser/android/java/gin_java_bridge_message_filter.h b/content/browser/android/java/gin_java_bridge_message_filter.h
index 40165a81d1d6c1c70b8e31f11b7da61ddc4277db..ee5051af76be88a8ec16321f81c6039e104c0784 100644
--- a/content/browser/android/java/gin_java_bridge_message_filter.h
+++ b/content/browser/android/java/gin_java_bridge_message_filter.h
@@ -45,6 +45,11 @@ class GinJavaBridgeMessageFilter : public BrowserMessageFilter {
static scoped_refptr<GinJavaBridgeMessageFilter> FromHost(
GinJavaBridgeDispatcherHost* host, bool create_if_not_exists);
+ // Removes the filter, which triggers its deletion. Needs to be called when
+ // the corresponding RenderProcessHost cleans itself up, e.g. on renderer
+ // process exit.
+ static void RemoveFilter(GinJavaBridgeDispatcherHost* host);
+
private:
friend class BrowserThread;
friend class base::DeleteHelper<GinJavaBridgeMessageFilter>;

Powered by Google App Engine
This is Rietveld 408576698