Adjust signatures to pass string rather than char* since it ultimately constructs a GURL.

chrome/browser/ui/webui/bidi_checker_web_ui_test.h

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_BIDI_CHECKER_WEB_UI_TEST_H_
 #define CHROME_BROWSER_UI_WEBUI_BIDI_CHECKER_WEB_UI_TEST_H_
 #pragma once
 
+#include <string>
+
 #include "chrome/browser/ui/webui/web_ui_browsertest.h"
 #include "chrome/test/base/in_process_browser_test.h"
 
 // Base class for BidiChecker-based tests. Preloads the BidiChecker JS library
 // for each test.
 class WebUIBidiCheckerBrowserTest : public WebUIBrowserTest {
  public:
   virtual ~WebUIBidiCheckerBrowserTest();
 
  protected:
   WebUIBidiCheckerBrowserTest();
 
   // Runs the Bidi Checker on the given page URL. |isRTL| should be true when
   // the active page locale is RTL.
-  void RunBidiCheckerOnPage(const char pageURL[], bool isRTL);
+  void RunBidiCheckerOnPage(const std::string& pageURL, bool isRTL);

[tfarina, 2012/03/19 00:59:27]

nit: page_url, is_rtl here and elsewhere?

[Sheridan Rawlins, 2012/03/20 18:51:33]

Done.

 
   // Setup test path.
   virtual void SetUpInProcessBrowserTestFixture() OVERRIDE;
 };
 
 // Base class for BidiChecker-based tests that run with an LTR UI.
 class WebUIBidiCheckerBrowserTestLTR : public WebUIBidiCheckerBrowserTest {
  public:
-  void RunBidiCheckerOnPage(const char pageURL[]);
+  void RunBidiCheckerOnPage(const std::string& pageURL);
 };
 
 // Base class for BidiChecker-based tests that run with an RTL UI.
 class WebUIBidiCheckerBrowserTestRTL : public WebUIBidiCheckerBrowserTest {
  public:
-  void RunBidiCheckerOnPage(const char pageURL[]);
+  void RunBidiCheckerOnPage(const std::string& pageURL);
 
  protected:
   virtual void SetUpOnMainThread() OVERRIDE;
   virtual void CleanUpOnMainThread() OVERRIDE;
 
   // The app locale before we change it
   std::string app_locale_;
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_BIDI_CHECKER_WEB_UI_TEST_H_