Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1296)

Unified Diff: chrome/browser/net/errorpage_browsertest.cc

Issue 2850293002: ErrorPage: use case-insensitive check for text (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/errorpage_browsertest.cc
diff --git a/chrome/browser/net/errorpage_browsertest.cc b/chrome/browser/net/errorpage_browsertest.cc
index dfd12f392b20b57efa41098c4cff8713d47c841c..ec0fdef5a5333b3dc856ed8f9c991a09569bad3f 100644
--- a/chrome/browser/net/errorpage_browsertest.cc
+++ b/chrome/browser/net/errorpage_browsertest.cc
@@ -95,8 +95,8 @@ namespace {
bool WARN_UNUSED_RESULT IsDisplayingText(Browser* browser,
const std::string& text) {
std::string command = base::StringPrintf(
- "var textContent = document.body.innerText;"
- "var hasText = textContent.indexOf('%s') >= 0;"
+ "var textContent = document.body.innerText.toLowerCase();"
+ "var hasText = textContent.indexOf('%s'.toLowerCase()) >= 0;"
"domAutomationController.send(hasText);",
text.c_str());
bool result = false;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698