[WebLayer] Rename test method.

Rename method in BrowserImpl to clarify it's used for tests.

Change-Id: I39a2e07333b64540547f0fc164b5dc59788f973d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096583
Auto-Submit: Mugdha Lakhani <nator@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748964}

weblayer/browser/browser_impl.h

@@ -91,7 +91,8 @@ class BrowserImpl : public Browser {
 
   // Used by tests to specify a callback to listen to changes to visible
   // security state.
-  void SetVisibleSecurityStateChangedCallback(base::OnceClosure closure) {
+  void set_visible_security_state_callback_for_tests(
+      base::OnceClosure closure) {
     visible_security_state_changed_callback_for_tests_ = std::move(closure);
   }
 

weblayer/browser/url_bar/url_bar_browsertest.cc

@@ -38,7 +38,8 @@ class UrlBarBrowserTest : public WebLayerBrowserTest {
   GURL abort_url() { return embedded_test_server()->GetURL("/nocontent"); }
 
   void SetVisibleSecurityStateChangedCallback(base::OnceClosure closure) {
-    browser_impl()->SetVisibleSecurityStateChangedCallback(std::move(closure));
+    browser_impl()->set_visible_security_state_callback_for_tests(
+        std::move(closure));
   }
 
  protected: