From 3969d2b5bf878d0d70f7790da335a6fb0995cb0d Mon Sep 17 00:00:00 2001 From: "mdm@chromium.org" Date: Wed, 15 Feb 2012 03:59:31 +0000 Subject: Linux: fix bookmarks with & in them not showing correctly in the wrench menu. As a side effect, also fix a bug in ReplaceChars() where it would go into an infinite loop (and probably also eventually run out of memory) if the replacement string contained any of the characters to be replaced. BUG=109546 Review URL: http://codereview.chromium.org/9348106 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122022 0039d316-1c4b-4281-b951-d872f2087c98 --- base/string_util_unittest.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'base/string_util_unittest.cc') diff --git a/base/string_util_unittest.cc b/base/string_util_unittest.cc index a028882..5c51c2a 100644 --- a/base/string_util_unittest.cc +++ b/base/string_util_unittest.cc @@ -1,4 +1,4 @@ -// 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. @@ -1087,6 +1087,7 @@ TEST(StringUtilTest, ReplaceChars) { { "test", "et", "!>", "!>!>s!>", true }, { "test", "zest", "!", "!!!!", true }, { "test", "szt", "!", "!e!!", true }, + { "test", "t", "test", "testestest", true }, }; for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { -- cgit v1.1