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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java

Issue 1960273002: Reland "Update NTP to new material design spec." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2016_05_09_revert
Patch Set: disable Created 4 years, 7 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: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java
index 89cbf0d4caf24c791288b77a5c9d2f1310922b6b..bdcb696c55fd681fe873607e7c1e093017125996 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java
@@ -11,7 +11,6 @@ import android.app.Dialog;
import android.content.Context;
import android.content.Intent;
import android.graphics.Canvas;
-import android.graphics.Color;
import android.graphics.Rect;
import android.net.Uri;
import android.os.Build;
@@ -590,9 +589,7 @@ public class NewTabPage
mProfile = tab.getProfile();
mTitle = activity.getResources().getString(R.string.button_new_tab);
- mBackgroundColor = ChromeFeatureList.isEnabled(ChromeFeatureList.NTP_SNIPPETS)
- ? ApiCompatibilityUtils.getColor(activity.getResources(), R.color.ntp_with_cards_bg)
- : ApiCompatibilityUtils.getColor(activity.getResources(), R.color.ntp_bg);
+ mBackgroundColor = NtpColorUtils.getBackgroundColorResource(activity.getResources());
mThemeColor = ApiCompatibilityUtils.getColor(
activity.getResources(), R.color.default_primary_color);
TemplateUrlService.getInstance().addObserver(this);
@@ -823,7 +820,9 @@ public class NewTabPage
@Override
public int getThemeColor() {
- return isLocationBarShownInNTP() ? Color.WHITE : mThemeColor;
+ return isLocationBarShownInNTP()
+ ? NtpColorUtils.getBackgroundColorResource(mActivity.getResources())
+ : mThemeColor;
}
@Override

Powered by Google App Engine
This is Rietveld 408576698