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

Unified Diff: blimp/client/core/contents/android/java/src/org/chromium/blimp/core/contents/BlimpContentsImpl.java

Issue 2315953002: Add theme color for Blimp on Android. (Closed)
Patch Set: Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: blimp/client/core/contents/android/java/src/org/chromium/blimp/core/contents/BlimpContentsImpl.java
diff --git a/blimp/client/core/contents/android/java/src/org/chromium/blimp/core/contents/BlimpContentsImpl.java b/blimp/client/core/contents/android/java/src/org/chromium/blimp/core/contents/BlimpContentsImpl.java
index a90a803bb8a700f688cbcf577f7804a9f586c3af..9342e026389f5d66476b651a4a3e7aac3b52ee5e 100644
--- a/blimp/client/core/contents/android/java/src/org/chromium/blimp/core/contents/BlimpContentsImpl.java
+++ b/blimp/client/core/contents/android/java/src/org/chromium/blimp/core/contents/BlimpContentsImpl.java
@@ -25,6 +25,8 @@ public class BlimpContentsImpl implements BlimpContents {
nativeBlimpContentsImplAndroid, navigationController, blimpView);
}
+ private static final int BLIMP_THEME_COLOR = 0xb3e5fc;
+
private long mNativeBlimpContentsImplAndroid;
// Given the importance of the navigation controller, this member is kept directly in Java to
@@ -73,6 +75,11 @@ public class BlimpContentsImpl implements BlimpContents {
}
@Override
+ public int getThemeColor() {
+ return BLIMP_THEME_COLOR;
David Trainor- moved to gerrit 2016/09/07 05:08:52 Add a TODO to get the right theme color when we do
nyquist 2016/09/07 15:11:04 I think this TODO also should come with a new crbu
xingliu 2016/09/07 17:23:36 Added a todo that links to a crbug.
+ }
+
+ @Override
public void addObserver(BlimpContentsObserver observer) {
assert mNativeBlimpContentsImplAndroid != 0;
if (mObserverProxy == null) mObserverProxy = new BlimpContentsObserverProxy(this);

Powered by Google App Engine
This is Rietveld 408576698