Remove RequestTracker from WebStateImpl.

ios/web/web_state/web_state_impl.h

 // Copyright 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.
 
 #ifndef IOS_WEB_WEB_STATE_WEB_STATE_IMPL_H_
 #define IOS_WEB_WEB_STATE_WEB_STATE_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <map>
 #include <memory>
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/observer_list.h"
 #include "base/values.h"
 #import "ios/web/navigation/navigation_manager_delegate.h"
 #import "ios/web/navigation/navigation_manager_impl.h"
 #import "ios/web/net/request_tracker_impl.h"

[Eugene But (OOO till 7-30), 2017/01/19 19:05:55]

nit: Do we still need this include?

[sdefresne, 2017/01/20 11:31:28]

Removed.

 #import "ios/web/public/java_script_dialog_callback.h"
 #include "ios/web/public/java_script_dialog_type.h"
 #import "ios/web/public/web_state/web_state.h"
 #include "url/gurl.h"
 
 @protocol CRWRequestTrackerDelegate;
 @class CRWWebController;
 @protocol CRWWebViewProxy;
 @class NSURLRequest;
 @class NSURLResponse;
@@ skipping 152 matching lines @@
   // that is the point where MIME type is set from HTTP headers.
   void SetContentsMimeType(const std::string& mime_type);
 
   // Returns whether the navigation corresponding to |request| should be allowed
   // to continue by asking its policy deciders. Defaults to true.
   bool ShouldAllowRequest(NSURLRequest* request);
   // Returns whether the navigation corresponding to |response| should be
   // allowed to continue by asking its policy deciders. Defaults to true.
   bool ShouldAllowResponse(NSURLResponse* response);
 
-  // Request tracker management. For now, this exposes the RequestTracker for
-  // embedders to use.
-  // TODO(stuartmorgan): RequestTracker should become an internal detail of this
-  // class.
-
-  // Create a new tracker using |delegate| as its delegate.
-  void InitializeRequestTracker(id<CRWRequestTrackerDelegate> delegate);
-
-  // Close the request tracker and delete it.
-  void CloseRequestTracker();
-
-  // Returns the tracker for this WebStateImpl.
-  RequestTrackerImpl* GetRequestTracker();
-
-  // Lazily creates (if necessary) and returns |request_group_id_|.
-  // IMPORTANT: This should not be used for anything other than associating this
-  // instance to network requests.
-  // This function is only intended to be used in web/.
-  // TODO(stuartmorgan): Move this method in an implementation file in web/.
-  NSString* GetRequestGroupID();
-
   // WebState:
   WebStateDelegate* GetDelegate() override;
   void SetDelegate(WebStateDelegate* delegate) override;
   bool IsWebUsageEnabled() const override;
   void SetWebUsageEnabled(bool enabled) override;
   bool ShouldSuppressDialogs() const override;
   void SetShouldSuppressDialogs(bool should_suppress) override;
   UIView* GetView() override;
   BrowserState* GetBrowserState() const override;
   void OpenURL(const WebState::OpenURLParams& params) override;
@@ skipping 118 matching lines @@
   scoped_refptr<net::HttpResponseHeaders> http_response_headers_;
   std::string mime_type_;
   std::string content_language_header_;
 
   // Weak pointer to the interstitial page being displayed, if any.
   WebInterstitialImpl* interstitial_;
 
   // Returned by reference.
   base::string16 empty_string16_;
 
-  // Request tracker associted with this object.
-  scoped_refptr<RequestTrackerImpl> request_tracker_;
-
-  // A number identifying this object. This number is injected into the user
-  // agent to allow the network layer to know which web view requests originated
-  // from.
-  base::scoped_nsobject<NSString> request_group_id_;
-
   // Callbacks associated to command prefixes.
   std::map<std::string, ScriptCommandCallback> script_command_callbacks_;
 
   // Member variables should appear before the WeakPtrFactory<> to ensure that
   // any WeakPtrs to WebStateImpl are invalidated before its member variable's
   // destructors are executed, rendering them invalid.
   base::WeakPtrFactory<WebState> weak_factory_;
 
   // Mojo interface registry for this WebState.
   std::unique_ptr<service_manager::InterfaceRegistry> mojo_interface_registry_;
 
   // Image Fetcher used to images.
   std::unique_ptr<ImageDataFetcher> image_fetcher_;
 
   DISALLOW_COPY_AND_ASSIGN(WebStateImpl);
 };
 
 }  // namespace web
 
 #endif  // IOS_WEB_WEB_STATE_WEB_STATE_IMPL_H_