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

Unified Diff: components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java

Issue 500193002: Fix Cronet compilation and test errors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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: components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
diff --git a/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java b/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
index d067cf6559a912cf037252112008ddf399ca80b8..f49bc5f4a8f888410573c96fa4809a22740027a1 100644
--- a/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
+++ b/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
@@ -397,6 +397,7 @@ public class ChromiumUrlRequest implements HttpUrlRequest {
}
// All response headers.
+ @SuppressWarnings("unused")
mmenke 2014/08/25 19:46:44 Should this be using the called by native thing?
mef 2014/08/25 20:26:18 Well, it isn't really called by native. Per discu
mef 2014/08/25 22:12:42 Per discussion with clm@ this is not a fix, but ju
public Map<String, List<String>> getAllHeaders() {
validateHeadersAvailable();
ResponseHeadersMap result = new ResponseHeadersMap();

Powered by Google App Engine
This is Rietveld 408576698