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

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java

Issue 2866213003: Revert of Add tests to make sure closed CCT activities are not leaked. (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.toolbar; 5 package org.chromium.chrome.browser.toolbar;
6 6
7 import android.animation.Animator; 7 import android.animation.Animator;
8 import android.animation.AnimatorListenerAdapter; 8 import android.animation.AnimatorListenerAdapter;
9 import android.animation.ValueAnimator; 9 import android.animation.ValueAnimator;
10 import android.animation.ValueAnimator.AnimatorUpdateListener; 10 import android.animation.ValueAnimator.AnimatorUpdateListener;
(...skipping 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 } 226 }
227 227
228 /** 228 /**
229 * @return The custom action button. For test purpose only. 229 * @return The custom action button. For test purpose only.
230 */ 230 */
231 @VisibleForTesting 231 @VisibleForTesting
232 public ImageButton getCustomActionButtonForTest() { 232 public ImageButton getCustomActionButtonForTest() {
233 return mCustomActionButton; 233 return mCustomActionButton;
234 } 234 }
235 235
236 /**
237 * @return The close button. For test purpose only.
238 */
239 @VisibleForTesting
240 public ImageButton getCloseButtonForTest() {
241 return mCloseButton;
242 }
243
244 @Override 236 @Override
245 public int getTabStripHeight() { 237 public int getTabStripHeight() {
246 return 0; 238 return 0;
247 } 239 }
248 240
249 @Override 241 @Override
250 public Tab getCurrentTab() { 242 public Tab getCurrentTab() {
251 return getToolbarDataProvider().getTab(); 243 return getToolbarDataProvider().getTab();
252 } 244 }
253 245
(...skipping 495 matching lines...) Expand 10 before | Expand all | Expand 10 after
749 public View getMenuButtonWrapper() { 741 public View getMenuButtonWrapper() {
750 // This class has no menu button wrapper, so return the menu button inst ead. 742 // This class has no menu button wrapper, so return the menu button inst ead.
751 return mMenuButton; 743 return mMenuButton;
752 } 744 }
753 745
754 @Override 746 @Override
755 public boolean mustQueryUrlBarLocationForSuggestions() { 747 public boolean mustQueryUrlBarLocationForSuggestions() {
756 return false; 748 return false;
757 } 749 }
758 } 750 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698