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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java

Issue 2878403002: Support setting mouse cursor icon in Android N. (Closed)
Patch Set: Support setting mouse cursor icon in Android N Created 3 years, 4 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/widget/OverviewListLayout.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java b/chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java
index 7a9177e75bd49a856249cef8716576e00c849cba..1ea62a1b7a1d2b3cbaa878b4d44233c42947b20a 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java
@@ -6,6 +6,7 @@ package org.chromium.chrome.browser.widget;
import android.content.Context;
import android.view.LayoutInflater;
+import android.view.View;
import android.view.ViewGroup;
import android.widget.FrameLayout;
import android.widget.ListView;
@@ -57,8 +58,10 @@ public class OverviewListLayout extends Layout implements AccessibilityTabModelA
if (container == null || mTabModelWrapper.getParent() != null) return;
- ((ViewGroup) container.findViewById(R.id.overview_list_layout_holder))
- .addView(mTabModelWrapper);
+ ViewGroup overviewList =
+ (ViewGroup) container.findViewById(R.id.overview_list_layout_holder);
+ overviewList.setVisibility(View.VISIBLE);
+ overviewList.addView(mTabModelWrapper);
}
@Override
@@ -152,7 +155,10 @@ public class OverviewListLayout extends Layout implements AccessibilityTabModelA
if (mTabModelSelector != null) mTabModelSelector.commitAllTabClosures();
if (mTabModelWrapper != null) {
ViewGroup parent = (ViewGroup) mTabModelWrapper.getParent();
- if (parent != null) parent.removeView(mTabModelWrapper);
+ if (parent != null) {
+ parent.setVisibility(View.GONE);
+ parent.removeView(mTabModelWrapper);
+ }
}
}

Powered by Google App Engine
This is Rietveld 408576698