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

Unified Diff: content/browser/loader/offline_policy.cc

Issue 271543005: Removed offline policy; it's not being used, and we have no plans to use it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to TOT. Created 6 years, 7 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 | « content/browser/loader/offline_policy.h ('k') | content/browser/loader/offline_policy_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/loader/offline_policy.cc
diff --git a/content/browser/loader/offline_policy.cc b/content/browser/loader/offline_policy.cc
deleted file mode 100644
index 9c9361f5258c1282e36ef9d80d4d03761b588b03..0000000000000000000000000000000000000000
--- a/content/browser/loader/offline_policy.cc
+++ /dev/null
@@ -1,96 +0,0 @@
-// Copyright (c) 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "content/browser/loader/offline_policy.h"
-
-#include "base/command_line.h"
-#include "base/metrics/histogram.h"
-#include "content/public/common/content_switches.h"
-#include "net/base/load_flags.h"
-#include "net/http/http_response_info.h"
-#include "net/url_request/url_request.h"
-
-namespace content {
-
-OfflinePolicy::OfflinePolicy()
- : enabled_(CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kEnableOfflineCacheAccess)),
- state_(INIT),
- resource_loads_initiated_(0),
- resource_loads_successfully_started_(0) {}
-
-OfflinePolicy::~OfflinePolicy() {
- RecordAndResetStats();
-}
-
-void OfflinePolicy::RecordAndResetStats() {
- if (enabled_ && OFFLINE == state_ && 0 != resource_loads_initiated_) {
- UMA_HISTOGRAM_PERCENTAGE(
- "OfflinePolicy.SuccessfulResourceLoadPercentage",
- (resource_loads_successfully_started_ * 100 /
- resource_loads_initiated_));
- }
- resource_loads_initiated_ = 0;
- resource_loads_successfully_started_ = 0;
-}
-
-int OfflinePolicy::GetAdditionalLoadFlags(int current_flags,
- bool reset_state) {
- // Don't do anything if offline mode is disabled.
- if (!enabled_)
- return 0;
-
- if (reset_state) {
- RecordAndResetStats();
- state_ = INIT;
- }
-
- ++resource_loads_initiated_;
-
- // If a consumer has requested something contradictory, it wins; we
- // don't modify the load flags.
- if (current_flags &
- (net::LOAD_BYPASS_CACHE | net::LOAD_PREFERRING_CACHE |
- net::LOAD_ONLY_FROM_CACHE | net::LOAD_FROM_CACHE_IF_OFFLINE |
- net::LOAD_DISABLE_CACHE)) {
- return 0;
- }
-
- switch(state_) {
- case INIT:
- return net::LOAD_FROM_CACHE_IF_OFFLINE;
- case ONLINE:
- return 0;
- case OFFLINE:
- return net::LOAD_ONLY_FROM_CACHE;
- }
- NOTREACHED();
- return 0;
-}
-
-void OfflinePolicy::UpdateStateForSuccessfullyStartedRequest(
- const net::HttpResponseInfo& response_info) {
- // Don't do anything if offline mode is disabled.
- if (!enabled_)
- return;
-
- // If we get here, we're going to be providing some amount of information
- // to the renderer.
- ++resource_loads_successfully_started_;
-
- if (state_ != INIT)
- // We've already made the decision for the rest of this set
- // of navigations.
- return;
-
- if (response_info.server_data_unavailable) {
- state_ = OFFLINE;
- } else if (response_info.network_accessed) {
- // If we got the response from the network or validated it as part
- // of this request, that means our connection to the host is working.
- state_ = ONLINE;
- }
-}
-
-} // namespace content
« no previous file with comments | « content/browser/loader/offline_policy.h ('k') | content/browser/loader/offline_policy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698