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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/StaticStripStacker.java

Issue 1836453003: Use a scrollable tab strip on tablets at small widths (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add tests Created 4 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/compositor/overlays/strip/StaticStripStacker.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/StaticStripStacker.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/StaticStripStacker.java
deleted file mode 100644
index 992b5bde0615544f1f49e1b28ce2757d46780ef9..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/StaticStripStacker.java
+++ /dev/null
@@ -1,66 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.chrome.browser.compositor.overlays.strip;
-
-import org.chromium.chrome.browser.util.MathUtils;
-
-/**
- * A stacker that tells the {@link StripHelper} how to layer the tabs for the
- * {@link StaticLayout}. This will basically be focused tab in front with tabs cascading
- * back to each side.
- */
-public class StaticStripStacker implements StripStacker {
- @Override
- public boolean canShowCloseButton() {
- return true;
- }
-
- @Override
- public boolean canSlideTitleText() {
- return true;
- }
-
- @Override
- public void createVisualOrdering(int selectedIndex, StripLayoutTab[] indexOrderedTabs,
- StripLayoutTab[] outVisualOrderedTabs) {
- assert indexOrderedTabs.length == outVisualOrderedTabs.length;
-
- selectedIndex = MathUtils.clamp(selectedIndex, 0, indexOrderedTabs.length);
-
- int outIndex = 0;
- for (int i = 0; i < selectedIndex; i++) {
- outVisualOrderedTabs[outIndex++] = indexOrderedTabs[i];
- }
-
- for (int i = indexOrderedTabs.length - 1; i >= selectedIndex; --i) {
- outVisualOrderedTabs[outIndex++] = indexOrderedTabs[i];
- }
- }
-
- @Override
- public void performOcclusionPass(int selectedIndex, StripLayoutTab[] indexOrderedTabs) {
- for (int i = 1; i < indexOrderedTabs.length; i++) {
- StripLayoutTab prevTab = indexOrderedTabs[i - 1];
- StripLayoutTab currTab = indexOrderedTabs[i];
-
- if ((int) prevTab.getDrawY() == (int) currTab.getDrawY()
- && (int) prevTab.getDrawX() == (int) currTab.getDrawX()) {
- if (i <= selectedIndex) {
- prevTab.setVisible(false);
- } else if (i > selectedIndex) {
- currTab.setVisible(false);
- }
- } else if ((int) prevTab.getDrawX() != (int) currTab.getDrawX()) {
- if (i <= selectedIndex) {
- prevTab.setVisible(true);
- } else if (i > selectedIndex) {
- currTab.setVisible(true);
- }
- }
-
- if (i == selectedIndex) currTab.setVisible(true);
- }
- }
-}

Powered by Google App Engine
This is Rietveld 408576698