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

Unified Diff: chrome/browser/net/chrome_network_delegate.cc

Issue 1459793002: Android: Allow compiling browser without Java UI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years 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 | « chrome/browser/metrics/chrome_metrics_service_client.cc ('k') | chrome/browser/net/net_error_tab_helper.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/chrome_network_delegate.cc
diff --git a/chrome/browser/net/chrome_network_delegate.cc b/chrome/browser/net/chrome_network_delegate.cc
index 8fd4f9bb1ffa69ece8a5fe5d652b8516e338d4c8..b7fc936f599c7b738d145253a4ff16155ab5a97d 100644
--- a/chrome/browser/net/chrome_network_delegate.cc
+++ b/chrome/browser/net/chrome_network_delegate.cc
@@ -35,6 +35,7 @@
#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/browser/task_management/task_manager_interface.h"
#include "chrome/common/chrome_constants.h"
+#include "chrome/common/features.h"
#include "chrome/common/pref_names.h"
#include "components/content_settings/core/browser/cookie_settings.h"
#include "components/data_usage/core/data_use_aggregator.h"
@@ -56,7 +57,7 @@
#include "net/log/net_log.h"
#include "net/url_request/url_request.h"
-#if defined(OS_ANDROID)
+#if BUILDFLAG(ANDROID_JAVA_UI)
#include "chrome/browser/io_thread.h"
#include "chrome/browser/precache/precache_manager_factory.h"
#include "components/precache/content/precache_manager.h"
@@ -106,7 +107,7 @@ void ForceGoogleSafeSearchCallbackWrapper(
callback.Run(rv);
}
-#if defined(OS_ANDROID)
+#if BUILDFLAG(ANDROID_JAVA_UI)
void RecordPrecacheStatsOnUIThread(const GURL& url,
const GURL& referrer,
base::TimeDelta latency,
@@ -129,7 +130,7 @@ void RecordPrecacheStatsOnUIThread(const GURL& url,
precache_manager->RecordStatsForFetch(url, referrer, latency, fetch_time,
size, was_cached);
}
-#endif // defined(OS_ANDROID)
+#endif // BUILDFLAG(ANDROID_JAVA_UI)
void ReportInvalidReferrerSendOnUI() {
base::RecordAction(
@@ -526,7 +527,7 @@ void ChromeNetworkDelegate::OnCompleted(net::URLRequest* request,
}
if (request->status().status() == net::URLRequestStatus::SUCCESS) {
-#if defined(OS_ANDROID)
+#if BUILDFLAG(ANDROID_JAVA_UI)
// For better accuracy, we use the actual bytes read instead of the length
// specified with the Content-Length header, which may be inaccurate,
// or missing, as is the case with chunked encoding.
@@ -540,7 +541,7 @@ void ChromeNetworkDelegate::OnCompleted(net::URLRequest* request,
base::Bind(&RecordPrecacheStatsOnUIThread, request->url(),
GURL(request->referrer()), latency, base::Time::Now(),
received_content_length, request->was_cached(), profile_));
-#endif // defined(OS_ANDROID)
+#endif // BUILDFLAG(ANDROID_JAVA_UI)
extensions_delegate_->OnCompleted(request, started);
} else if (request->status().status() == net::URLRequestStatus::FAILED ||
request->status().status() == net::URLRequestStatus::CANCELED) {
« no previous file with comments | « chrome/browser/metrics/chrome_metrics_service_client.cc ('k') | chrome/browser/net/net_error_tab_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698