DevTools: make network conditions emulation scoped (browser)

chrome/browser/devtools/devtools_network_controller_unittest.cc

 // Copyright 2014 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 <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "chrome/browser/devtools/devtools_network_conditions.h"
 #include "chrome/browser/devtools/devtools_network_controller.h"
 #include "chrome/browser/devtools/devtools_network_transaction.h"
 #include "net/http/http_transaction_test_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 namespace test {
 
-const char kHttpDotCom[] = "http://dot.com";
-const char kHttpDotOrg[] = "http://dot.org";
-const char kCom[] = "com";
+const char kClientId[] = "42";
+const char kAnotherClientId[] = "24";
 
 class TestCallback {
  public:
   TestCallback() : run_count_(0), value_(0) {}
   void Run(int value) {
     run_count_++;
     value_ = value;
   }
   int run_count() { return run_count_; }
   int value() { return value_; }
 
  private:
   int run_count_;
   int value_;
 };
 
 class DevToolsNetworkControllerHelper {
  public:
   DevToolsNetworkControllerHelper() :
       completion_callback_(
           base::Bind(&TestCallback::Run, base::Unretained(&callback_))),
       mock_transaction_(kSimpleGET_Transaction),
       buffer_(new net::IOBuffer(64)) {
     mock_transaction_.test_mode = TEST_MODE_SYNC_NET_START;
-    mock_transaction_.url = kHttpDotCom;
+    mock_transaction_.url = "http://dot.com";
+    mock_transaction_.request_headers =
+        "X-DevTools-Emulate-Network-Conditions-Client-Id: 42\r\n";
     AddMockTransaction(&mock_transaction_);
 
     scoped_ptr<net::HttpTransaction> network_transaction;
     network_layer_.CreateTransaction(
         net::DEFAULT_PRIORITY, &network_transaction);
     transaction_.reset(new DevToolsNetworkTransaction(
         &controller_, network_transaction.Pass()));
   }
 
   net::HttpRequestInfo* GetRequest() {
     if (!request_)
       request_.reset(new MockHttpRequest(mock_transaction_));
     return request_.get();
   }
 
-  void SetNetworkState(bool offline) {
-    std::vector<std::string> domains;
-    domains.push_back(kCom);
+  void SetNetworkState(const std::string id, bool offline) {
     scoped_refptr<DevToolsNetworkConditions> conditions;
     if (offline)
-      conditions = new DevToolsNetworkConditions(domains, 0.0);
-    controller_.SetNetworkStateOnIO(conditions);
+      conditions = new DevToolsNetworkConditions(0, 0, 0);
+    controller_.SetNetworkStateOnIO(id, conditions);
   }
 
   int Start() {
     return transaction_->Start(
         GetRequest(), completion_callback_, net::BoundNetLog());
   }
 
   int Read() {
     return transaction_->Read(buffer_.get(), 64, completion_callback_);
   }
@@ skipping 15 matching lines @@
   MockTransaction mock_transaction_;
   DevToolsNetworkController controller_;
   scoped_ptr<DevToolsNetworkTransaction> transaction_;
   scoped_refptr<net::IOBuffer> buffer_;
   scoped_ptr<MockHttpRequest> request_;
 };
 
 TEST(DevToolsNetworkControllerTest, SingleDisableEnable) {
   DevToolsNetworkControllerHelper helper;
   DevToolsNetworkController* controller = helper.controller();
-  net::HttpRequestInfo* request = helper.GetRequest();
+  helper.Start();
 
-  EXPECT_FALSE(controller->ShouldFail(request));
-  helper.SetNetworkState(true);
-  EXPECT_TRUE(controller->ShouldFail(request));
-  helper.SetNetworkState(false);
-  EXPECT_FALSE(controller->ShouldFail(request));
+  EXPECT_FALSE(controller->ShouldFail(helper.transaction()));
+  helper.SetNetworkState(kClientId, true);
+  EXPECT_TRUE(controller->ShouldFail(helper.transaction()));
+  helper.SetNetworkState(kClientId, false);
+  EXPECT_FALSE(controller->ShouldFail(helper.transaction()));
+
+  base::RunLoop().RunUntilIdle();
+}
+
+TEST(DevToolsNetworkControllerTest, RealmIsolation) {
+  DevToolsNetworkControllerHelper helper;
+  DevToolsNetworkController* controller = helper.controller();
+  helper.Start();
+
+  EXPECT_FALSE(controller->ShouldFail(helper.transaction()));
+  helper.SetNetworkState(kAnotherClientId, true);
+  EXPECT_FALSE(controller->ShouldFail(helper.transaction()));
+  helper.SetNetworkState(kClientId, true);
+  EXPECT_TRUE(controller->ShouldFail(helper.transaction()));
+
+  helper.SetNetworkState(kAnotherClientId, false);
+  helper.SetNetworkState(kClientId, false);
+  base::RunLoop().RunUntilIdle();
 }
 
 TEST(DevToolsNetworkControllerTest, FailOnStart) {
   DevToolsNetworkControllerHelper helper;
-  helper.SetNetworkState(true);
+  helper.SetNetworkState(kClientId, true);
 
   int rv = helper.Start();
   EXPECT_EQ(rv, net::ERR_INTERNET_DISCONNECTED);
 
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(helper.callback()->run_count(), 0);
+
+  helper.SetNetworkState(kClientId, false);
 }
 
 TEST(DevToolsNetworkControllerTest, FailRunningTransaction) {
   DevToolsNetworkControllerHelper helper;
   TestCallback* callback = helper.callback();
 
   int rv = helper.Start();
   EXPECT_EQ(rv, net::OK);
 
   scoped_refptr<net::IOBuffer> buffer(new net::IOBuffer(64));
   rv = helper.Read();
   EXPECT_EQ(rv, net::ERR_IO_PENDING);
   EXPECT_EQ(callback->run_count(), 0);
 
-  helper.SetNetworkState(true);
+  helper.SetNetworkState(kClientId, true);
   EXPECT_EQ(callback->run_count(), 1);
   EXPECT_EQ(callback->value(), net::ERR_INTERNET_DISCONNECTED);
 
   // Wait until HttpTrancation completes reading and invokes callback.
   // DevToolsNetworkTransaction should ignore callback, because it has
   // reported network error already.
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(callback->run_count(), 1);
 
   // Check that transaction in not failed second time.
-  helper.SetNetworkState(false);
-  helper.SetNetworkState(true);
+  helper.SetNetworkState(kClientId, false);
+  helper.SetNetworkState(kClientId, true);
   EXPECT_EQ(callback->run_count(), 1);
+
+  helper.SetNetworkState(kClientId, false);
 }
 
 TEST(DevToolsNetworkControllerTest, ReadAfterFail) {
   DevToolsNetworkControllerHelper helper;
 
   int rv = helper.Start();
   EXPECT_EQ(rv, net::OK);
   EXPECT_TRUE(helper.transaction()->request());
 
-  helper.SetNetworkState(true);
+  helper.SetNetworkState(kClientId, true);
   EXPECT_TRUE(helper.transaction()->failed());
 
   scoped_refptr<net::IOBuffer> buffer(new net::IOBuffer(64));
   rv = helper.Read();
   EXPECT_EQ(rv, net::ERR_INTERNET_DISCONNECTED);
 
   // Check that callback is never invoked.
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(helper.callback()->run_count(), 0);
+
+  helper.SetNetworkState(kClientId, false);
 }
 
 TEST(DevToolsNetworkControllerTest, AllowsDevToolsRequests) {
   DevToolsNetworkControllerHelper helper;
   helper.mock_transaction()->request_headers =
+      "X-DevTools-Emulate-Network-Conditions-Client-Id: 42\r\n"
       "X-DevTools-Request-Initiator: frontend\r\n";
   DevToolsNetworkController* controller = helper.controller();
-  net::HttpRequestInfo* request = helper.GetRequest();
 
-  EXPECT_FALSE(controller->ShouldFail(request));
-  helper.SetNetworkState(true);
-  EXPECT_FALSE(controller->ShouldFail(request));
-}
+  EXPECT_FALSE(controller->ShouldFail(helper.transaction()));
+  helper.SetNetworkState(kClientId, true);
+  EXPECT_FALSE(controller->ShouldFail(helper.transaction()));
 
-TEST(DevToolsNetworkControllerTest, AllowsNotMatchingRequests) {
-  DevToolsNetworkControllerHelper helper;
-  helper.mock_transaction()->url = kHttpDotOrg;
-  DevToolsNetworkController* controller = helper.controller();
-  net::HttpRequestInfo* request = helper.GetRequest();
-
-  EXPECT_FALSE(controller->ShouldFail(request));
-  helper.SetNetworkState(true);
-  EXPECT_FALSE(controller->ShouldFail(request));
+  helper.SetNetworkState(kClientId, false);
 }
 
 }  // namespace test