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

Side by Side Diff: chrome/browser/prerender/prerender_util_unittest.cc

Issue 7289020: Updating histograms to allow for experiments & log origin-based histograms. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/prerender/prerender_util.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/prerender/prerender_util.h"
6 #include "testing/gtest/include/gtest/gtest.h"
7
8 namespace prerender {
9
10 class PrerenderUtilTest : public testing::Test {
11 public:
12 PrerenderUtilTest() {
13 }
14 };
15
16 // Ensure that extracting a urlencoded URL in the url= query string component
17 // works.
18 TEST_F(PrerenderUtilTest, ExtractURLInQueryStringTest) {
19 GURL result;
20 EXPECT_TRUE(MaybeGetQueryStringBasedAliasURL(
21 GURL("http://www.google.com/url?sa=t&source=web&cd=1&ved=0CBcQFjAA&url=htt p%3A%2F%2Fwww.abercrombie.com%2Fwebapp%2Fwcs%2Fstores%2Fservlet%2FStoreLocator%3 FcatalogId%3D%26storeId%3D10051%26langId%3D-1&rct=j&q=allinurl%3A%26&ei=KLyUTYGS EdTWiAKUmLCdCQ&usg=AFQjCNF8nJ2MpBFfr1ijO39_f22bcKyccw&sig2=2ymyGpO0unJwU1d4kdCUj Q"),
22 &result));
23 ASSERT_EQ(GURL("http://www.abercrombie.com/webapp/wcs/stores/servlet/StoreLoca tor?catalogId=&storeId=10051&langId=-1").spec(), result.spec());
24 EXPECT_FALSE(MaybeGetQueryStringBasedAliasURL(
25 GURL("http://www.google.com/url?sadf=test&blah=blahblahblah"), &result));
26 EXPECT_FALSE(MaybeGetQueryStringBasedAliasURL(
27 GURL("http://www.google.com/?url=INVALIDurlsAREsoMUCHfun.com"), &result));
28 EXPECT_TRUE(MaybeGetQueryStringBasedAliasURL(
29 GURL("http://www.google.com/?url=http://validURLSareGREAT.com"),
30 &result));
31 ASSERT_EQ(GURL("http://validURLSareGREAT.com").spec(), result.spec());
32 }
33
34 // Ensure that extracting an experiment in the lpe= query string component
35 // works.
36 TEST_F(PrerenderUtilTest, ExtractExperimentInQueryStringTest) {
37 GURL result;
38 EXPECT_EQ(GetQueryStringBasedExperiment(
39 GURL("http://www.google.com/url?sa=t&source=web&cd=1&ved=0CBcQFjAA&url=htt p%3A%2F%2Fwww.abercrombie.com%2Fwebapp%2Fwcs%2Fstores%2Fservlet%2FStoreLocator%3 FcatalogId%3D%26storeId%3D10051%26langId%3D-1&rct=j&q=allinurl%3A%26&ei=KLyUTYGS EdTWiAKUmLCdCQ&usg=AFQjCNF8nJ2MpBFfr1ijO39_f22bcKyccw&sig2=2ymyGpO0unJwU1d4kdCUj Q&lpe=4&asdf=test")), 4);
40 EXPECT_EQ(GetQueryStringBasedExperiment(
41 GURL("http://www.google.com/test.php?a=b")), kNoExperiment);
42 EXPECT_EQ(GetQueryStringBasedExperiment(
43 GURL("http://www.google.com/test.php?lpe=5")), 5);
44 EXPECT_EQ(GetQueryStringBasedExperiment(
45 GURL("http://www.google.com/test.php?lpe=50")), kNoExperiment);
46 EXPECT_EQ(GetQueryStringBasedExperiment(
47 GURL("http://www.google.com/test.php?lpe=0")), kNoExperiment);
48 EXPECT_EQ(GetQueryStringBasedExperiment(
49 GURL("http://www.google.com/test.php?lpe=10")), kNoExperiment);
50 }
51
52 } // namespace prerender
OLDNEW
« no previous file with comments | « chrome/browser/prerender/prerender_util.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698