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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/feedback/FeedbackCollector.java

Issue 2362693003: Add framework for sending feedback for Blimp. (Closed)
Patch Set: Rebased for good measure 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
« no previous file with comments | « blimp/client/public/android/java/src/org/chromium/blimp_public/BlimpClientContext.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/feedback/FeedbackCollector.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/feedback/FeedbackCollector.java b/chrome/android/java/src/org/chromium/chrome/browser/feedback/FeedbackCollector.java
index 218a166a328ef7129fee36dc40e715ea0b5695ef..367923725a0e81d2a9194f4926d89e83fae5d731 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/feedback/FeedbackCollector.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/feedback/FeedbackCollector.java
@@ -12,6 +12,8 @@ import android.text.TextUtils;
import org.chromium.base.ThreadUtils;
import org.chromium.base.VisibleForTesting;
+import org.chromium.blimp_public.BlimpClientContext;
+import org.chromium.chrome.browser.blimp.BlimpClientContextFactory;
import org.chromium.chrome.browser.net.spdyproxy.DataReductionProxySettings;
import org.chromium.chrome.browser.profiles.Profile;
import org.chromium.components.variations.VariationsAssociatedData;
@@ -240,6 +242,7 @@ public class FeedbackCollector
addConnectivityData();
addDataReductionProxyData();
addVariationsData();
+ addBlimpData();
return asBundle();
}
@@ -267,6 +270,14 @@ public class FeedbackCollector
mData.putAll(VariationsAssociatedData.getFeedbackMap());
}
+ private void addBlimpData() {
+ if (mProfile.isOffTheRecord()) return;
+
+ BlimpClientContext blimpClientContext =
+ BlimpClientContextFactory.getBlimpClientContextForProfile(mProfile);
+ mData.putAll(blimpClientContext.getFeedbackMap());
+ }
+
private Bundle asBundle() {
Bundle bundle = new Bundle();
for (Map.Entry<String, String> entry : mData.entrySet()) {
« no previous file with comments | « blimp/client/public/android/java/src/org/chromium/blimp_public/BlimpClientContext.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698