Commit ebbad604 authored by ananta@chromium.org's avatar ananta@chromium.org

Fix the newly added OverrideUserAgent net test so that it works in ChromeFrame where the

chromeframe suffix is added to the outgoing UA.

BUG=none
TEST=Existing OverideUserAgent test should pass in the regular and ChromeFrame case.
R=wtc

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72228 0039d316-1c4b-4281-b951-d872f2087c98
parent c051a1b5
......@@ -23,6 +23,7 @@
#include "base/process_util.h"
#include "base/string_number_conversions.h"
#include "base/string_piece.h"
#include "base/string_util.h"
#include "base/stringprintf.h"
#include "base/utf_string_conversions.h"
#include "net/base/cookie_monster.h"
......@@ -2669,5 +2670,8 @@ TEST_F(URLRequestTestHTTP, OverrideUserAgent) {
req.SetExtraRequestHeaders(headers);
req.Start();
MessageLoop::current()->Run();
EXPECT_EQ(std::string("Lynx (textmode)"), d.data_received());
// If the net tests are being run with ChromeFrame then we need to allow for
// the 'chromeframe' suffix which is added to the user agent in outgoing HTTP
// requests.
EXPECT_TRUE(StartsWithASCII(d.data_received(), "Lynx (textmode)", true));
}
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment