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

Unified Diff: chrome/browser/android/offline_pages/prerendering_loader.cc

Issue 2104393002: Adds UMA for PrerenderingOffliner request processing result status. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Revised histogram naming from OfflinerStatus to OfflinerRequestStatus Created 4 years, 6 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/browser/android/offline_pages/prerendering_loader.cc
diff --git a/chrome/browser/android/offline_pages/prerendering_loader.cc b/chrome/browser/android/offline_pages/prerendering_loader.cc
index b82a149ab9ddb0be4d9b23bb4c051f37614d5029..2c9aa46398ed0ab5e8191fced176e24a942e86f9 100644
--- a/chrome/browser/android/offline_pages/prerendering_loader.cc
+++ b/chrome/browser/android/offline_pages/prerendering_loader.cc
@@ -164,8 +164,8 @@ void PrerenderingLoader::HandleLoadingStopped() {
// Request status depends on whether we are still loading (failed) or
// did load and then loading was stopped (cancel - from prerender stack).
Offliner::RequestStatus request_status =
- IsLoaded() ? Offliner::RequestStatus::CANCELED
- : Offliner::RequestStatus::FAILED;
+ IsLoaded() ? Offliner::RequestStatus::LOAD_CANCELED
+ : Offliner::RequestStatus::LOAD_FAILED;
// TODO(dougarnett): For failure, determine from final status if retry-able
// and report different failure statuses if retry-able or not.
snapshot_controller_.reset(nullptr);
@@ -184,7 +184,7 @@ void PrerenderingLoader::CancelPrerender() {
if (!IsLoaded() && !IsIdle()) {
base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE,
- base::Bind(callback_, Offliner::RequestStatus::CANCELED, nullptr));
+ base::Bind(callback_, Offliner::RequestStatus::LOAD_CANCELED, nullptr));
}
state_ = State::IDLE;
}

Powered by Google App Engine
This is Rietveld 408576698