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

Unified Diff: chrome/browser/devtools/devtools_network_controller.h

Issue 342473004: DevTools: make network conditions emulation scoped (browser) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rename + fix nits + polish Created 6 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/devtools/devtools_network_controller.h
diff --git a/chrome/browser/devtools/devtools_network_controller.h b/chrome/browser/devtools/devtools_network_controller.h
index a9edd0fe31d5d10cac0f9bb15c603243018feb8c..6550da4fa656a01f17590fc87f77a8d8f257103a 100644
--- a/chrome/browser/devtools/devtools_network_controller.h
+++ b/chrome/browser/devtools/devtools_network_controller.h
@@ -5,34 +5,18 @@
#ifndef CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_CONTROLLER_H_
#define CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_CONTROLLER_H_
-#include <set>
#include <string>
-#include <vector>
+#include "base/containers/scoped_ptr_hash_map.h"
#include "base/macros.h"
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
#include "base/threading/thread_checker.h"
-#include "base/timer/timer.h"
class DevToolsNetworkConditions;
+class DevToolsNetworkInterceptor;
class DevToolsNetworkTransaction;
-class GURL;
-class Profile;
-
-namespace base {
-class TimeDelta;
-class TimeTicks;
-}
-
-namespace content {
-class ResourceContext;
-}
-
-namespace net {
-struct HttpRequestInfo;
-}
namespace test {
class DevToolsNetworkControllerHelper;
@@ -45,17 +29,13 @@ class DevToolsNetworkController {
DevToolsNetworkController();
virtual ~DevToolsNetworkController();
- void AddTransaction(DevToolsNetworkTransaction* transaction);
-
- void RemoveTransaction(DevToolsNetworkTransaction* transaction);
-
// Applies network emulation configuration.
void SetNetworkState(
+ const std::string& client_id,
const scoped_refptr<DevToolsNetworkConditions> conditions);
- bool ShouldFail(const net::HttpRequestInfo* request);
- bool ShouldThrottle(const net::HttpRequestInfo* request);
- void ThrottleTransaction(DevToolsNetworkTransaction* transaction);
+ base::WeakPtr<DevToolsNetworkInterceptor> GetInterceptor(
+ DevToolsNetworkTransaction* transaction);
protected:
friend class test::DevToolsNetworkControllerHelper;
@@ -64,24 +44,16 @@ class DevToolsNetworkController {
// Controller must be constructed on IO thread.
base::ThreadChecker thread_checker_;
- typedef scoped_refptr<DevToolsNetworkConditions> Conditions;
-
- void SetNetworkStateOnIO(const Conditions conditions);
-
- typedef std::set<DevToolsNetworkTransaction*> Transactions;
- Transactions transactions_;
-
- Conditions conditions_;
-
- void UpdateThrottles();
- void ArmTimer();
- void OnTimer();
+ void SetNetworkStateOnIO(
+ const std::string& client_id,
+ const scoped_refptr<DevToolsNetworkConditions> conditions);
- std::vector<DevToolsNetworkTransaction*> throttled_transactions_;
- base::OneShotTimer<DevToolsNetworkController> timer_;
- base::TimeTicks offset_;
- base::TimeDelta tick_length_;
- uint64_t last_tick_;
+ typedef scoped_ptr<DevToolsNetworkInterceptor> Interceptor;
+ Interceptor default_interceptor_;
+ Interceptor appcache_interceptor_;
+ typedef base::ScopedPtrHashMap<std::string, DevToolsNetworkInterceptor>
+ Interceptors;
+ Interceptors interceptors_;
base::WeakPtrFactory<DevToolsNetworkController> weak_ptr_factory_;

Powered by Google App Engine
This is Rietveld 408576698