summaryrefslogtreecommitdiffstats
path: root/chrome/test/automation
diff options
context:
space:
mode:
authorbrettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-07-31 17:29:25 +0000
committerbrettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-07-31 17:29:25 +0000
commite83326f8400791e92875546b2fd1885a3a17d1b1 (patch)
treeedbe773208b1a9f6965b45b55da10afd210ea7bb /chrome/test/automation
parent8e0a03bf3b1aacaa7a2bc2561d8eb1b83eb9c2e5 (diff)
downloadchromium_src-e83326f8400791e92875546b2fd1885a3a17d1b1.zip
chromium_src-e83326f8400791e92875546b2fd1885a3a17d1b1.tar.gz
chromium_src-e83326f8400791e92875546b2fd1885a3a17d1b1.tar.bz2
Convert more callers of the integer/string functions to using
string_number_conversions.h TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3013046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54454 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/automation')
-rw-r--r--chrome/test/automation/dom_automation_browsertest.cc19
-rw-r--r--chrome/test/automation/extension_proxy.cc3
2 files changed, 15 insertions, 7 deletions
diff --git a/chrome/test/automation/dom_automation_browsertest.cc b/chrome/test/automation/dom_automation_browsertest.cc
index 7085395..dbaa1ef 100644
--- a/chrome/test/automation/dom_automation_browsertest.cc
+++ b/chrome/test/automation/dom_automation_browsertest.cc
@@ -3,6 +3,7 @@
// found in the LICENSE file.
#include "base/ref_counted.h"
+#include "base/string_number_conversions.h"
#include "base/utf_string_conversions.h"
#include "chrome/test/automation/dom_element_proxy.h"
#include "chrome/test/automation/javascript_execution_controller.h"
@@ -45,8 +46,10 @@ IN_PROC_BROWSER_TEST_F(DOMAutomationTest, FindByXPath) {
std::vector<DOMElementProxyRef> elements;
ASSERT_TRUE(main_doc->FindElements(By::XPath("//div"), &elements));
ASSERT_EQ(2u, elements.size());
- for (size_t i = 0; i < elements.size(); i++)
- ASSERT_NO_FATAL_FAILURE(elements[i]->EnsureNameMatches(UintToString(i)));
+ for (size_t i = 0; i < elements.size(); i++) {
+ ASSERT_NO_FATAL_FAILURE(elements[i]->EnsureNameMatches(
+ base::UintToString(i)));
+ }
// Find 0 elements.
ASSERT_FALSE(main_doc->FindElement(By::XPath("//nosuchtag")));
@@ -90,8 +93,10 @@ IN_PROC_BROWSER_TEST_F(DOMAutomationTest, FindBySelectors) {
std::vector<DOMElementProxyRef> elements;
ASSERT_TRUE(main_doc->FindElements(By::Selectors(".myclass"), &elements));
ASSERT_EQ(2u, elements.size());
- for (size_t i = 0; i < elements.size(); i++)
- ASSERT_NO_FATAL_FAILURE(elements[i]->EnsureNameMatches(UintToString(i)));
+ for (size_t i = 0; i < elements.size(); i++) {
+ ASSERT_NO_FATAL_FAILURE(elements[i]->EnsureNameMatches(
+ base::UintToString(i)));
+ }
// Find 0 elements.
ASSERT_FALSE(main_doc->FindElement(By::Selectors("#nosuchid")));
@@ -131,8 +136,10 @@ IN_PROC_BROWSER_TEST_F(DOMAutomationTest, FindByText) {
std::vector<DOMElementProxyRef> elements;
ASSERT_TRUE(main_doc->FindElements(By::Text("div_text"), &elements));
ASSERT_EQ(2u, elements.size());
- for (size_t i = 0; i < elements.size(); i++)
- ASSERT_NO_FATAL_FAILURE(elements[i]->EnsureNameMatches(UintToString(i)));
+ for (size_t i = 0; i < elements.size(); i++) {
+ ASSERT_NO_FATAL_FAILURE(elements[i]->EnsureNameMatches(
+ base::UintToString(i)));
+ }
// Find 0 elements.
ASSERT_FALSE(main_doc->FindElement(By::Text("nosuchtext")));
diff --git a/chrome/test/automation/extension_proxy.cc b/chrome/test/automation/extension_proxy.cc
index 66aa8e8..29fd54c 100644
--- a/chrome/test/automation/extension_proxy.cc
+++ b/chrome/test/automation/extension_proxy.cc
@@ -4,6 +4,7 @@
#include "chrome/test/automation/extension_proxy.h"
+#include "base/string_number_conversions.h"
#include "chrome/test/automation/automation_messages.h"
#include "chrome/test/automation/automation_proxy.h"
#include "chrome/test/automation/browser_proxy.h"
@@ -94,7 +95,7 @@ bool ExtensionProxy::GetBrowserActionIndex(int* index) {
// Do not modify |index| until we are sure we can get the value, just to be
// nice to the caller.
int converted_index;
- if (!StringToInt(index_string, &converted_index)) {
+ if (!base::StringToInt(index_string, &converted_index)) {
LOG(ERROR) << "Received index string could not be converted to int: "
<< index_string;
return false;