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

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

Issue 2630513003: Expand bottom sheet when URL bar is focused (Closed)
Patch Set: address comments Created 3 years, 11 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.annotation.SuppressLint; 10 import android.annotation.SuppressLint;
(...skipping 17 matching lines...) Expand all
28 import org.chromium.chrome.R; 28 import org.chromium.chrome.R;
29 import org.chromium.chrome.browser.appmenu.AppMenuButtonHelper; 29 import org.chromium.chrome.browser.appmenu.AppMenuButtonHelper;
30 import org.chromium.chrome.browser.compositor.Invalidator; 30 import org.chromium.chrome.browser.compositor.Invalidator;
31 import org.chromium.chrome.browser.compositor.layouts.LayoutUpdateHost; 31 import org.chromium.chrome.browser.compositor.layouts.LayoutUpdateHost;
32 import org.chromium.chrome.browser.fullscreen.BrowserStateBrowserControlsVisibil ityDelegate; 32 import org.chromium.chrome.browser.fullscreen.BrowserStateBrowserControlsVisibil ityDelegate;
33 import org.chromium.chrome.browser.ntp.NewTabPage; 33 import org.chromium.chrome.browser.ntp.NewTabPage;
34 import org.chromium.chrome.browser.omaha.UpdateMenuItemHelper; 34 import org.chromium.chrome.browser.omaha.UpdateMenuItemHelper;
35 import org.chromium.chrome.browser.omnibox.LocationBar; 35 import org.chromium.chrome.browser.omnibox.LocationBar;
36 import org.chromium.chrome.browser.tab.Tab; 36 import org.chromium.chrome.browser.tab.Tab;
37 import org.chromium.chrome.browser.util.ViewUtils; 37 import org.chromium.chrome.browser.util.ViewUtils;
38 import org.chromium.chrome.browser.widget.BottomSheet;
38 import org.chromium.chrome.browser.widget.TintedImageButton; 39 import org.chromium.chrome.browser.widget.TintedImageButton;
39 import org.chromium.chrome.browser.widget.ToolbarProgressBar; 40 import org.chromium.chrome.browser.widget.ToolbarProgressBar;
40 import org.chromium.ui.UiUtils; 41 import org.chromium.ui.UiUtils;
41 import org.chromium.ui.widget.Toast; 42 import org.chromium.ui.widget.Toast;
42 43
43 import javax.annotation.Nullable; 44 import javax.annotation.Nullable;
44 45
45 /** 46 /**
46 * Layout class that contains the base shared logic for manipulating the toolbar component. For 47 * Layout class that contains the base shared logic for manipulating the toolbar component. For
47 * interaction that are not from Views inside Toolbar hierarchy all interactions should be done 48 * interaction that are not from Views inside Toolbar hierarchy all interactions should be done
(...skipping 739 matching lines...) Expand 10 before | Expand all | Expand 10 after
787 */ 788 */
788 protected void setMenuButtonContentDescription(boolean isUpdateBadgeVisible) { 789 protected void setMenuButtonContentDescription(boolean isUpdateBadgeVisible) {
789 if (isUpdateBadgeVisible) { 790 if (isUpdateBadgeVisible) {
790 mMenuButton.setContentDescription(getResources().getString( 791 mMenuButton.setContentDescription(getResources().getString(
791 R.string.accessibility_toolbar_btn_menu_update)); 792 R.string.accessibility_toolbar_btn_menu_update));
792 } else { 793 } else {
793 mMenuButton.setContentDescription(getResources().getString( 794 mMenuButton.setContentDescription(getResources().getString(
794 R.string.accessibility_toolbar_btn_menu)); 795 R.string.accessibility_toolbar_btn_menu));
795 } 796 }
796 } 797 }
798
799 @Override
800 public void setBottomSheet(BottomSheet sheet) {}
797 } 801 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698