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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java

Issue 2434893003: Hide the page info dialog if the underlying webcontents is hidden. (Closed)
Patch Set: Created 4 years, 2 months 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/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java b/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
index f3e2c6a9f860d88c78103ceb205fe86ae57e6c5d..365811066e556c685eed2f61288dc6209780f153 100644
--- a/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
+++ b/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
@@ -140,6 +140,22 @@ class WebContentsObserverProxy extends WebContentsObserver {
@Override
@CalledByNative
+ public void wasShown() {
+ for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
+ mObserversIterator.next().wasShown();
+ }
+ }
+
+ @Override
+ @CalledByNative
+ public void wasHidden() {
+ for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
+ mObserversIterator.next().wasHidden();
+ }
+ }
+
+ @Override
+ @CalledByNative
public void didNavigateAnyFrame(String url, String baseUrl, boolean isReload) {
for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
mObserversIterator.next().didNavigateAnyFrame(url, baseUrl, isReload);

Powered by Google App Engine
This is Rietveld 408576698