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

Unified Diff: ios/web/web_state/web_state_impl.mm

Issue 2644103004: Remove RequestTracker from WebStateImpl. (Closed)
Patch Set: Created 3 years, 11 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: ios/web/web_state/web_state_impl.mm
diff --git a/ios/web/web_state/web_state_impl.mm b/ios/web/web_state/web_state_impl.mm
index 4ab5963c38a782a324aceee37995cd9279a700bf..fdf4660463b6b096582b056dc734dcf8ddfd5258 100644
--- a/ios/web/web_state/web_state_impl.mm
+++ b/ios/web/web_state/web_state_impl.mm
@@ -14,7 +14,6 @@
#import "ios/web/navigation/crw_session_controller.h"
#import "ios/web/navigation/crw_session_entry.h"
#import "ios/web/navigation/navigation_item_impl.h"
-#import "ios/web/net/request_group_util.h"
#include "ios/web/public/browser_state.h"
#import "ios/web/public/image_fetcher/image_data_fetcher.h"
#import "ios/web/public/java_script_dialog_presenter.h"
@@ -92,8 +91,6 @@ WebStateImpl::~WebStateImpl() {
for (auto& observer : policy_deciders_)
observer.ResetWebState();
DCHECK(script_command_callbacks_.empty());
- if (request_tracker_.get())
- CloseRequestTracker();
SetDelegate(nullptr);
}
@@ -512,31 +509,6 @@ bool WebStateImpl::ShouldAllowResponse(NSURLResponse* response) {
#pragma mark - RequestTracker management
-void WebStateImpl::InitializeRequestTracker(
- id<CRWRequestTrackerDelegate> delegate) {
- BrowserState* browser_state = navigation_manager_.GetBrowserState();
- request_tracker_ = RequestTrackerImpl::CreateTrackerForRequestGroupID(
- GetRequestGroupID(), browser_state, browser_state->GetRequestContext(),
- delegate);
-}
-
-void WebStateImpl::CloseRequestTracker() {
- request_tracker_->Close();
- request_tracker_ = NULL;
-}
-
-RequestTrackerImpl* WebStateImpl::GetRequestTracker() {
- DCHECK(request_tracker_.get());
- return request_tracker_.get();
-}
-
-NSString* WebStateImpl::GetRequestGroupID() {
- if (request_group_id_.get() == nil)
- request_group_id_.reset([GenerateNewRequestGroupID() copy]);
-
- return request_group_id_;
-}
-
int WebStateImpl::DownloadImage(
const GURL& url,
bool is_favicon,

Powered by Google App Engine
This is Rietveld 408576698