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

Unified Diff: chrome/browser/ssl/ssl_browser_tests.cc

Issue 2593263003: Use a non-replaced inline container for image alt text (Closed)
Patch Set: Use a non-replaced inline container for image alt text 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
Index: chrome/browser/ssl/ssl_browser_tests.cc
diff --git a/chrome/browser/ssl/ssl_browser_tests.cc b/chrome/browser/ssl/ssl_browser_tests.cc
index cadbe9d34fc4e7b9c57b3012794b70983b6e9cfd..2006a6fb5955e9591f298202279849a5a2ae20b5 100644
--- a/chrome/browser/ssl/ssl_browser_tests.cc
+++ b/chrome/browser/ssl/ssl_browser_tests.cc
@@ -1716,12 +1716,11 @@ IN_PROC_BROWSER_TEST_F(SSLUITest, TestUnsafeContents) {
// The iframe attempts to open a popup window, but it shouldn't be able to.
// Previous popup is still open.
EXPECT_EQ(2u, chrome::GetBrowserCount(browser()->profile()));
- // Assume the broken image width is less than 100.
- int img_width = 0;
+ // The broken image width is zero.
+ int img_width = 99;
EXPECT_TRUE(content::ExecuteScriptAndExtractInt(
tab, "window.domAutomationController.send(ImageWidth());", &img_width));
- EXPECT_GT(img_width, 0);
- EXPECT_LT(img_width, 100);
+ EXPECT_EQ(img_width, 0);
// Check that variable |foo| is not set.
bool js_result = false;
EXPECT_TRUE(content::ExecuteScriptAndExtractBool(

Powered by Google App Engine
This is Rietveld 408576698