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

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/toolbar/Toolbar.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.graphics.Rect; 7 import android.graphics.Rect;
8 import android.view.View; 8 import android.view.View;
9 9
10 import org.chromium.chrome.browser.compositor.layouts.LayoutUpdateHost; 10 import org.chromium.chrome.browser.compositor.layouts.LayoutUpdateHost;
11 import org.chromium.chrome.browser.widget.BottomSheet;
11 12
12 /** 13 /**
13 * An interface for outside packages to interact with ToolbarLayout. Other than for testing purposes 14 * An interface for outside packages to interact with ToolbarLayout. Other than for testing purposes
14 * this interface should be used rather than {@link ToolbarLayout} and extending classes. 15 * this interface should be used rather than {@link ToolbarLayout} and extending classes.
15 */ 16 */
16 public interface Toolbar { 17 public interface Toolbar {
17 18
18 /** 19 /**
19 * Calculates the {@link Rect} that represents the content area of the locat ion bar. This 20 * Calculates the {@link Rect} that represents the content area of the locat ion bar. This
20 * rect will be relative to the toolbar. 21 * rect will be relative to the toolbar.
(...skipping 15 matching lines...) Expand all
36 37
37 /** 38 /**
38 * Sets whether or not the toolbar should draw as if it's being captured for a snapshot 39 * Sets whether or not the toolbar should draw as if it's being captured for a snapshot
39 * texture. In this mode it will only draw the toolbar in it's normal state (no TabSwitcher 40 * texture. In this mode it will only draw the toolbar in it's normal state (no TabSwitcher
40 * or animations). 41 * or animations).
41 * @param textureMode Whether or not to be in texture capture mode. 42 * @param textureMode Whether or not to be in texture capture mode.
42 */ 43 */
43 void setTextureCaptureMode(boolean textureMode); 44 void setTextureCaptureMode(boolean textureMode);
44 45
45 /** 46 /**
47 * Set the {@link BottomSheet} for triggering animations. This can be null i f Chrome Home is
48 * disabled.
49 * @param sheet The {@link BottomSheet}.
50 */
51 void setBottomSheet(BottomSheet sheet);
52
53 /**
46 * @return Whether a dirty check for invalidation makes sense at this time. 54 * @return Whether a dirty check for invalidation makes sense at this time.
47 */ 55 */
48 boolean isReadyForTextureCapture(); 56 boolean isReadyForTextureCapture();
49 57
50 /** 58 /**
51 * Sets whether or not the toolbar should force itself to draw for a texture capture regardless 59 * Sets whether or not the toolbar should force itself to draw for a texture capture regardless
52 * of other criteria used in isReadyForTextureCapture(). A texture capture w ill only be forced 60 * of other criteria used in isReadyForTextureCapture(). A texture capture w ill only be forced
53 * if the toolbar drawables tint is changing. 61 * if the toolbar drawables tint is changing.
54 * 62 *
55 * @param forceTextureCapture Whether the toolbar should force itself to dra w. 63 * @param forceTextureCapture Whether the toolbar should force itself to dra w.
(...skipping 23 matching lines...) Expand all
79 * Remove the update badge on the app menu button. Initially the badge is in visible so that it 87 * Remove the update badge on the app menu button. Initially the badge is in visible so that it
80 * gets measured and the tab switcher animation looks correct when the badge is first shown. If 88 * gets measured and the tab switcher animation looks correct when the badge is first shown. If
81 * the badge will never be shown or should no longer be shown, this method s hould be called to 89 * the badge will never be shown or should no longer be shown, this method s hould be called to
82 * change the visibility to gone to avoid unnecessary layout work. The disap pearance of the 90 * change the visibility to gone to avoid unnecessary layout work. The disap pearance of the
83 * badge is optionally animated if it was previously visible. 91 * badge is optionally animated if it was previously visible.
84 * 92 *
85 * @param animate Whether the removal of the badge should be animated. 93 * @param animate Whether the removal of the badge should be animated.
86 */ 94 */
87 void removeAppMenuUpdateBadge(boolean animate); 95 void removeAppMenuUpdateBadge(boolean animate);
88 } 96 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698