Skip to content
Snippets Groups Projects
Commit 92f132ae authored by ananta@chromium.org's avatar ananta@chromium.org
Browse files

Fix the ChromeFrame BeginningTransaction unit test failures which validate the HttpNegotiate

patch. On IE6/7/8 the chrome frame user agent is added to the ua section in the registry. This
causes the test matcher to fail as the actual UA string contains the string chromeframe followed
by the version of the dll.

Fix is to reduce the scope of the matcher to just look for the string chromeframe.

BUG=none
TEST=Covered by existing chrome frame BeginningTransaction test.
TBR=amit


Review URL: http://codereview.chromium.org/6045001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69591 0039d316-1c4b-4281-b951-d872f2087c98
parent f7093e0d
No related branches found
No related tags found
No related merge requests found
......@@ -70,7 +70,7 @@ TEST_F(HttpNegotiateTest, BeginningTransaction) {
std::wstring cf_tag(
ASCIIToWide(http_utils::GetChromeFrameUserAgent()));
EXPECT_NE(std::wstring::npos, cf_ua.find(cf_tag));
EXPECT_NE(std::wstring::npos, cf_ua.find(L"chromeframe/"));
struct TestCase {
const std::wstring original_headers_;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment