One more PhishingDOMFeatureExtractorTest suppression, for the IPC channel name.

chrome/renderer/safe_browsing/phishing_dom_feature_extractor_unittest.cc

 // Copyright (c) 2010 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 "chrome/renderer/safe_browsing/phishing_dom_feature_extractor.h"
 
 #include <string.h>  // for memcpy()
 #include <map>
 #include <string>
 
@@ skipping 51 matching lines @@
     // but we use a real RenderThread so that we can use the ResourceDispatcher
     // to fetch network resources.  These are then served canned content
     // in OnRequestResource().
     sandbox_init_wrapper_.reset(new SandboxInitWrapper);
     command_line_.reset(new CommandLine(CommandLine::ARGUMENTS_ONLY));
     params_.reset(new MainFunctionParams(*command_line_,
                                          *sandbox_init_wrapper_, NULL));
     platform_.reset(new RendererMainPlatformDelegate(*params_));
     platform_->PlatformInitialize();
 
-    // We use a new IPC channel name for each test that runs.
-    // This is necessary because the renderer-side IPC channel is not
-    // shut down when the RenderThread goes away, so attempting to reuse
-    // the channel name gives an error (see ChildThread::~ChildThread()).
-    std::string thread_name = StringPrintf(
-        "phishing_dom_feature_Extractor_unittest.%d",
-        next_thread_id_++);
+    std::string thread_name = GetNextThreadName();
     channel_.reset(new IPC::Channel(thread_name,
                                     IPC::Channel::MODE_SERVER, this));
     ASSERT_TRUE(channel_->Connect());
 
     webkit_glue::SetJavaScriptFlags("--expose-gc");
     mock_process_.reset(new MockRenderProcess);
     render_thread_ = new RenderThread(thread_name);
     mock_process_->set_main_thread(render_thread_);
 
     // Tell the renderer to create a view, then wait until it's ready.
@@ skipping 110 matching lines @@
 
   // Notification that the render view we've created is ready to use.
   void OnRenderViewReady() {
     // Grab a pointer to the new view using RenderViewVisitor.
     ASSERT_TRUE(!view_);
     RenderView::ForEach(this);
     ASSERT_TRUE(view_);
     msg_loop_.Quit();
   }
 
+  // We use a new IPC channel name for each test that runs.
+  // This is necessary because the renderer-side IPC channel is not
+  // shut down when the RenderThread goes away, so attempting to reuse
+  // the channel name gives an error (see ChildThread::~ChildThread()).
+  static std::string GetNextThreadName() {
+    return StringPrintf(
+        "phishing_dom_feature_Extractor_unittest.%d",
+        next_thread_id_++);
+  }
+
   static int next_thread_id_;  // incrementing counter for thread ids
   static const int32 kViewId = 5;  // arbitrary id for our testing view
 
   MessageLoopForIO msg_loop_;
   // channel that the renderer uses to talk to the browser.
   // For this test, we will handle the browser end of the channel.
   scoped_ptr<IPC::Channel> channel_;
   RenderThread* render_thread_;  // owned by mock_process_
   scoped_ptr<MockRenderProcess> mock_process_;
   RenderView* view_;  // not owned, deletes itself on close
@@ skipping 182 matching lines @@
   FeatureMap features;
   LoadURL("http://host.com/");
   ASSERT_TRUE(ExtractFeatures(&features));
   EXPECT_THAT(features.features(), ContainerEq(expected_features.features()));
 }
 
 // TODO(bryner): Test extraction with multiple passes, including the case where
 // the node we stopped on is removed from the document.
 
 }  // namespace safe_browsing