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

Unified Diff: chrome/browser/policy/policy_browsertest.cc

Issue 12277011: New network error pages, part 1. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Go back to using <if> for font sizes Created 7 years, 9 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
Index: chrome/browser/policy/policy_browsertest.cc
===================================================================
--- chrome/browser/policy/policy_browsertest.cc (revision 184957)
+++ chrome/browser/policy/policy_browsertest.cc (working copy)
@@ -239,14 +239,11 @@
EXPECT_EQ(title, contents->GetTitle());
// Verify that the expected error page is being displayed.
- // (error 138 == NETWORK_ACCESS_DENIED)
bool result = false;
EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
contents,
- "var hasError = false;"
- "var error = document.getElementById('errorDetails');"
- "if (error)"
- " hasError = error.textContent.indexOf('Error 138') == 0;"
+ "var textContent = document.body.textContent;"
+ "var hasError = textContent.indexOf('NETWORK_ACCESS_DENIED') != -1;"
"domAutomationController.send(hasError);",
&result));
EXPECT_TRUE(result);

Powered by Google App Engine
This is Rietveld 408576698