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

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

Issue 146057: Making browser test work (again) on Linux (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 6 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 | « base/path_service.cc ('k') | chrome/test/in_process_browser_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ssl/ssl_browser_tests.cc
===================================================================
--- chrome/browser/ssl/ssl_browser_tests.cc (revision 19071)
+++ chrome/browser/ssl/ssl_browser_tests.cc (working copy)
@@ -73,7 +73,6 @@
DISALLOW_COPY_AND_ASSIGN(SSLUITest);
};
-#if defined(OS_WIN)
// Visits a regular page over http.
IN_PROC_BROWSER_TEST_F(SSLUITest, TestHTTP) {
scoped_refptr<HTTPTestServer> server = PlainServer();
@@ -712,21 +711,7 @@
content_frame_xpath, is_frame_evil_js, &is_content_evil));
EXPECT_FALSE(is_content_evil);
}
-#else
-// TODO(port): enable the real tests.
-IN_PROC_BROWSER_TEST_F(SSLUITest, PhonyTest1) {
- EXPECT_TRUE(true);
-}
-IN_PROC_BROWSER_TEST_F(SSLUITest, PhonyTest2) {
- EXPECT_TRUE(false);
-}
-
-IN_PROC_BROWSER_TEST_F(SSLUITest, PhonyTest3) {
- EXPECT_TRUE(true);
-}
-#endif
-
// TODO(jcampan): more tests to do below.
// Visit a page over https that contains a frame with a redirect.
« no previous file with comments | « base/path_service.cc ('k') | chrome/test/in_process_browser_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698