Enable SiteIsolationExtensions trial for developers.

chrome/browser/extensions/api/networking_private/networking_private_service_client_apitest.cc

 // 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.
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
@@ skipping 86 matching lines @@
 
   void SetUpOnMainThread() override {
     ExtensionApiTest::SetUpOnMainThread();
     content::RunAllPendingInMessageLoop();
     NetworkingPrivateDelegateFactory::GetInstance()->SetTestingFactory(
         profile(), &CreateNetworkingPrivateServiceClient);
   }
 
   void TearDownOnMainThread() override {
     content::RunAllPendingInMessageLoop();
-    ExtensionApiTest::SetUpOnMainThread();

[Devlin, 2017/02/15 19:24:34]

Heh not the first time I've seen this.  It'd be great if there was a way we
could have a warning for this ("Hey, it looks like you're calling the wrong
super version of the method"), but I don't have a good heuristic... ah well.

[nasko, 2017/02/15 19:34:57]

Acknowledged.

+    ExtensionApiTest::TearDownOnMainThread();
   }
 
  protected:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateServiceClientApiTest);
 };
 
 // Place each subtest into a separate browser test so that the stub networking
 // library state is reset for each subtest run. This way they won't affect each
 // other.
 
@@ skipping 95 matching lines @@
                        VerifyAndEncryptCredentials) {
   EXPECT_TRUE(RunNetworkingSubtest("verifyAndEncryptCredentials")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(NetworkingPrivateServiceClientApiTest,
                        VerifyAndEncryptData) {
   EXPECT_TRUE(RunNetworkingSubtest("verifyAndEncryptData")) << message_;
 }
 
 }  // namespace