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

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

Issue 2876733002: IPH UI : Fix text bubble x-alignment (Closed)
Patch Set: Replaced menu anchor with menu button 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.AnimatorSet; 9 import android.animation.AnimatorSet;
10 import android.content.Context; 10 import android.content.Context;
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 } 228 }
229 229
230 /** 230 /**
231 * @return The view containing the menu button and menu button badge. 231 * @return The view containing the menu button and menu button badge.
232 */ 232 */
233 protected View getMenuButtonWrapper() { 233 protected View getMenuButtonWrapper() {
234 return mMenuButtonWrapper; 234 return mMenuButtonWrapper;
235 } 235 }
236 236
237 /** 237 /**
238 * @return The view containing the menu button.
239 */
240 protected View getMenuButton() {
241 return mMenuButton;
242 }
243
244 /**
238 * @return The {@link ProgressBar} this layout uses. 245 * @return The {@link ProgressBar} this layout uses.
239 */ 246 */
240 @VisibleForTesting 247 @VisibleForTesting
241 public ToolbarProgressBar getProgressBar() { 248 public ToolbarProgressBar getProgressBar() {
242 return mProgressBar; 249 return mProgressBar;
243 } 250 }
244 251
245 @Override 252 @Override
246 public void getPositionRelativeToContainer(View containerView, int[] positio n) { 253 public void getPositionRelativeToContainer(View containerView, int[] positio n) {
247 ViewUtils.getRelativeDrawPosition(containerView, this, position); 254 ViewUtils.getRelativeDrawPosition(containerView, this, position);
(...skipping 592 matching lines...) Expand 10 before | Expand all | Expand 10 after
840 R.string.accessibility_toolbar_btn_menu_update)); 847 R.string.accessibility_toolbar_btn_menu_update));
841 } else { 848 } else {
842 mMenuButton.setContentDescription(getResources().getString( 849 mMenuButton.setContentDescription(getResources().getString(
843 R.string.accessibility_toolbar_btn_menu)); 850 R.string.accessibility_toolbar_btn_menu));
844 } 851 }
845 } 852 }
846 853
847 @Override 854 @Override
848 public void setBottomSheet(BottomSheet sheet) {} 855 public void setBottomSheet(BottomSheet sheet) {}
849 } 856 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698