Isolate ExternalProtocolHandlerTest from other tests in the TearDown method.

The issue is that these tests call LaunchUrlWithDelegate(), which sets
g_accept_requests to false unconditionally.  This can cause other tests, such
as AutocompleteInput.InputType, to fail if run with different sharding or
different ordering.

The solution is to call PermitLaunchUrl() from the TearDown method, which
resets g_accept_requests to true.

R=benwells@chromium.org
BUG=105574
TEST=Run unit_tests.exe with three shards and watch the DrMemory bots


Review URL: http://codereview.chromium.org/8763006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112404 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/external_protocol/external_protocol_handler_unittest.cc

@@ -102,6 +102,11 @@ class ExternalProtocolHandlerTest : public testing::Test {
     file_thread_.Start();
   }
 
+  virtual void TearDown() {
+    // Ensure that g_accept_requests gets set back to true after test execution.
+    ExternalProtocolHandler::PermitLaunchUrl();
+  }
+
   void DoTest(ExternalProtocolHandler::BlockState block_state,
               ShellIntegration::DefaultWebClientState os_state,
               bool should_prompt, bool should_launch, bool should_block) {