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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/banners/AppData.java

Issue 1237303003: Move more tests from ChromeShellTest to ChromePublicTest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 5 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/banners/AppData.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/banners/AppData.java b/chrome/android/java/src/org/chromium/chrome/browser/banners/AppData.java
index d0f1447b1d77a5417b93b1883f833bb566324aaa..dabe5015f39dc6109dd5274be8831336f7ed5938 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/banners/AppData.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/banners/AppData.java
@@ -7,6 +7,8 @@ package org.chromium.chrome.browser.banners;
import android.app.PendingIntent;
import android.content.Intent;
+import org.chromium.base.VisibleForTesting;
+
/**
* Stores information about a particular app.
*/
@@ -28,6 +30,7 @@ public class AppData {
* @param siteUrl URL for the site requesting the banner.
* @param packageName Name of the package associated with the app.
*/
+ @VisibleForTesting
public AppData(String siteUrl, String packageName) {
mSiteUrl = siteUrl;
mPackageName = packageName;
@@ -107,6 +110,7 @@ public class AppData {
* @param detailsIntent Intent to fire to launch the details page for the app
* @param installIntent Intent to fire to trigger the purchase/install process.
*/
+ @VisibleForTesting
public void setPackageInfo(String title, String imageUrl, float rating,
String installButtonText, PendingIntent detailsIntent, Intent installIntent) {
mTitle = title;
« no previous file with comments | « chrome/android/BUILD.gn ('k') | chrome/android/java/src/org/chromium/chrome/browser/banners/AppDetailsDelegate.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698