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

Side by Side Diff: chrome/browser/chromeos/login/saml/saml_browsertest.cc

Issue 2732113003: Disabling SAMLPolicyTest.SAMLInterstitialNext due to flakiness (Closed)
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <cstring> 5 #include <cstring>
6 #include <memory> 6 #include <memory>
7 #include <string> 7 #include <string>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 1389 matching lines...) Expand 10 before | Expand all | Expand 10 after
1400 // Expects that only the gaia signin frame is visible and shown. 1400 // Expects that only the gaia signin frame is visible and shown.
1401 JsExpect("$('signin-frame').classList.contains('show')"); 1401 JsExpect("$('signin-frame').classList.contains('show')");
1402 JsExpect("$('signin-frame').hidden == false"); 1402 JsExpect("$('signin-frame').hidden == false");
1403 JsExpect("$('offline-gaia').hidden == true"); 1403 JsExpect("$('offline-gaia').hidden == true");
1404 JsExpect("$('saml-interstitial').hidden == true"); 1404 JsExpect("$('saml-interstitial').hidden == true");
1405 } 1405 }
1406 1406
1407 // Tests that clicking "Next" in the SAML interstitial page successfully 1407 // Tests that clicking "Next" in the SAML interstitial page successfully
1408 // triggers a SAML redirect request, and the SAML IdP authentication page is 1408 // triggers a SAML redirect request, and the SAML IdP authentication page is
1409 // loaded and authenticaing there is successful. 1409 // loaded and authenticaing there is successful.
1410 IN_PROC_BROWSER_TEST_F(SAMLPolicyTest, SAMLInterstitialNext) { 1410 // Disabled due to flakiness, see crbug.com/699228
1411 IN_PROC_BROWSER_TEST_F(SAMLPolicyTest, DISABLED_SAMLInterstitialNext) {
1411 fake_saml_idp()->SetLoginHTMLTemplate("saml_login.html"); 1412 fake_saml_idp()->SetLoginHTMLTemplate("saml_login.html");
1412 fake_gaia_->SetFakeMergeSessionParams( 1413 fake_gaia_->SetFakeMergeSessionParams(
1413 kFirstSAMLUserEmail, kTestAuthSIDCookie1, kTestAuthLSIDCookie1); 1414 kFirstSAMLUserEmail, kTestAuthSIDCookie1, kTestAuthLSIDCookie1);
1414 SetLoginBehaviorPolicyToSAMLInterstitial(); 1415 SetLoginBehaviorPolicyToSAMLInterstitial();
1415 WaitForSigninScreen(); 1416 WaitForSigninScreen();
1416 1417
1417 ShowSAMLInterstitial(); 1418 ShowSAMLInterstitial();
1418 ClickNextOnSAMLInterstitialPage(); 1419 ClickNextOnSAMLInterstitialPage();
1419 1420
1420 SetSignFormField("Email", "fake_user"); 1421 SetSignFormField("Email", "fake_user");
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
1479 url3, url3, CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA, std::string(), 1480 url3, url3, CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA, std::string(),
1480 CONTENT_SETTING_ALLOW); 1481 CONTENT_SETTING_ALLOW);
1481 1482
1482 MediaPermission permission(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA, url3, 1483 MediaPermission permission(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA, url3,
1483 url3, profile, web_contents); 1484 url3, profile, web_contents);
1484 EXPECT_EQ(CONTENT_SETTING_BLOCK, permission.GetPermissionStatus(&reason)); 1485 EXPECT_EQ(CONTENT_SETTING_BLOCK, permission.GetPermissionStatus(&reason));
1485 } 1486 }
1486 } 1487 }
1487 1488
1488 } // namespace chromeos 1489 } // namespace chromeos
OLDNEW
« 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