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

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: rebase 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 bottom sheet for triggering animations. This can be null if Chrom e Home is disabled.
48 * @param sheet The bottom sheet.
gone 2017/01/20 19:04:08 Should probably just use {@link BottomSheet} rathe
mdjones 2017/01/20 20:11:47 Done.
49 */
50 void setBottomSheet(BottomSheet sheet);
51
52 /**
46 * @return Whether a dirty check for invalidation makes sense at this time. 53 * @return Whether a dirty check for invalidation makes sense at this time.
47 */ 54 */
48 boolean isReadyForTextureCapture(); 55 boolean isReadyForTextureCapture();
49 56
50 /** 57 /**
51 * Sets whether or not the toolbar should force itself to draw for a texture capture regardless 58 * 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 59 * of other criteria used in isReadyForTextureCapture(). A texture capture w ill only be forced
53 * if the toolbar drawables tint is changing. 60 * if the toolbar drawables tint is changing.
54 * 61 *
55 * @param forceTextureCapture Whether the toolbar should force itself to dra w. 62 * @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 86 * 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 87 * 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 88 * 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 89 * 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. 90 * badge is optionally animated if it was previously visible.
84 * 91 *
85 * @param animate Whether the removal of the badge should be animated. 92 * @param animate Whether the removal of the badge should be animated.
86 */ 93 */
87 void removeAppMenuUpdateBadge(boolean animate); 94 void removeAppMenuUpdateBadge(boolean animate);
88 } 95 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698