Chromium Code Reviews

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java

Issue 1203223003: Replace return button with close button on custom tab toolbar (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: get resource from intent data provider Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
index 38654cecb01939fe8f46ddbf647917cd6b4ab5c8..f8e617e6ebd52f29f1496c8ea8bf71c9bee47505 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
@@ -506,7 +506,7 @@ public class ToolbarManager implements ToolbarTabController, UrlFocusChangeListe
mToolbar.setOnTabSwitcherClickHandler(tabSwitcherClickHandler);
mToolbar.setOnNewTabClickHandler(newTabClickHandler);
mToolbar.setBookmarkClickHandler(bookmarkClickHandler);
- mToolbar.setCustomTabReturnClickHandler(customTabsBackClickHandler);
+ mToolbar.setCustomTabCloseClickHandler(customTabsBackClickHandler);
mToolbarModel.initializeWithNative();
@@ -797,6 +797,13 @@ public class ToolbarManager implements ToolbarTabController, UrlFocusChangeListe
}
/**
+ * Sets the icon resource that the close button shows.
+ */
+ public void setCloseButtonIcon(int iconRes) {
+ mToolbar.setCloseButtonIcon(iconRes);
+ }
+
+ /**
* Sets whether a title should be shown within the Toolbar.
* @param showTitle Whether a title should be shown.
*/

Powered by Google App Engine