Add missing return statement after ReceivedBadMessage call.

content/browser/child_process_security_policy_browsertest.cc

 // Copyright (c) 2012 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/files/file_path.h"
 #include "build/build_config.h"
+#include "content/browser/bad_message.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/common/result_codes.h"
+#include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "content/shell/browser/shell.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace content {
 
 class ChildProcessSecurityPolicyInProcessBrowserTest
     : public ContentBrowserTest {
  public:
@@ skipping 26 matching lines @@
 
   WebContents* web_contents = shell()->web_contents();
   web_contents->GetRenderProcessHost()->Shutdown(RESULT_CODE_KILLED, true);
 
   web_contents->GetController().Reload(ReloadType::NORMAL, true);
   EXPECT_EQ(
       1U,
       ChildProcessSecurityPolicyImpl::GetInstance()->security_state_.size());
 }
 
+IN_PROC_BROWSER_TEST_F(ChildProcessSecurityPolicyInProcessBrowserTest,
+                       RevokeOnBadMessage) {
+  GURL url = GetTestUrl("", "simple_page.html");
+
+  NavigateToURL(shell(), url);
+  int child_id = shell()->web_contents()->GetRenderProcessHost()->GetID();
+  base::FilePath file_path = GetTestFilePath("", "simple_page.html");
+  ChildProcessSecurityPolicyImpl::GetInstance()->GrantReadFile(child_id,
+                                                               file_path);
+  EXPECT_TRUE(ChildProcessSecurityPolicyImpl::GetInstance()->CanReadFile(
+      child_id, file_path));
+
+  RenderProcessHostWatcher process_exit_observer(
+      shell()->web_contents()->GetRenderProcessHost(),
+      RenderProcessHostWatcher::WATCH_FOR_PROCESS_EXIT);
+  bad_message::ReceivedBadMessage(child_id,
+                                  bad_message::RFPH_ILLEGAL_UPLOAD_PARAMS);
+  process_exit_observer.Wait();
+  EXPECT_FALSE(
+      ChildProcessSecurityPolicyImpl::GetInstance()->CanReadFile(child_id,

[Charlie Reis, 2017/05/31 17:28:23]

This is failing on the bots, but maybe that doesn't matter if we stick with the
early return approach for now.

[jam, 2017/05/31 23:26:15]

ok, I'll do this separately since it's a good point we don't have to do it on
branch.

The test passes on windows, must be a race condition that it fails elsewhere (or
maybe the error code doesn't get signalled correctly there).

+                                                                 file_path));
+}
+
 }  // namespace content