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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/TabViewAndroidDelegate.java

Issue 2804953003: Refactor ContentViewClient (6/6) (Closed)
Patch Set: Created 3 years, 8 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: chrome/android/java/src/org/chromium/chrome/browser/tab/TabViewAndroidDelegate.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tab/TabViewAndroidDelegate.java b/chrome/android/java/src/org/chromium/chrome/browser/tab/TabViewAndroidDelegate.java
index e4689e0e512166228ee0419bf8d85ef1cc9a4e82..c3203654e7910f0210c966e3bc7fcfe6a6fc7c16 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tab/TabViewAndroidDelegate.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tab/TabViewAndroidDelegate.java
@@ -9,20 +9,23 @@ import android.content.Intent;
import android.view.ViewGroup;
import org.chromium.base.Log;
+import org.chromium.base.VisibleForTesting;
import org.chromium.base.metrics.RecordUserAction;
+import org.chromium.chrome.browser.ChromeActivity;
import org.chromium.ui.base.ViewAndroidDelegate;
/**
* Implementation of the abstract class {@link ViewAndroidDelegate} for Chrome.
*/
-class TabViewAndroidDelegate extends ViewAndroidDelegate {
+@VisibleForTesting
+public class TabViewAndroidDelegate extends ViewAndroidDelegate {
/** Used for logging. */
private static final String TAG = "TabVAD";
private final Tab mTab;
private final ViewGroup mContainerView;
- TabViewAndroidDelegate(Tab tab, ViewGroup containerView) {
+ public TabViewAndroidDelegate(Tab tab, ViewGroup containerView) {
mTab = tab;
mContainerView = containerView;
}
@@ -53,6 +56,15 @@ class TabViewAndroidDelegate extends ViewAndroidDelegate {
}
@Override
+ public int getSystemWindowInsetBottom() {
+ ChromeActivity activity = mTab.getActivity();
+ if (activity != null && activity.getInsetObserverView() != null) {
+ return activity.getInsetObserverView().getSystemWindowInsetsBottom();
+ }
+ return 0;
+ }
+
+ @Override
public ViewGroup getContainerView() {
return mContainerView;
}

Powered by Google App Engine
This is Rietveld 408576698