chrome/browser/component_updater/test/component_updater_service_unittest.cc - Issue 100543005: Update all users of base::Version to explicitly specify the namespace, and clean up the header.

Unified Diff: chrome/browser/component_updater/test/component_updater_service_unittest.cc

Issue 100543005: Update all users of base::Version to explicitly specify the namespace, and clean up the header. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: someday it will work Created 7 years 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 | « chrome/browser/component_updater/test/component_updater_service_unittest.h ('k') | chrome/browser/component_updater/test/test_installer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/component_updater/test/component_updater_service_unittest.cc

diff --git a/chrome/browser/component_updater/test/component_updater_service_unittest.cc b/chrome/browser/component_updater/test/component_updater_service_unittest.cc

index ccabb615a63736747707bf15fde91f8351c3a578..898b9034b17df0afcf7e1c376e3266d27db63147 100644

--- a/chrome/browser/component_updater/test/component_updater_service_unittest.cc

+++ b/chrome/browser/component_updater/test/component_updater_service_unittest.cc

@@ -209,7 +209,7 @@ TestConfigurator* ComponentUpdaterTest::test_configurator() {

ComponentUpdateService::Status ComponentUpdaterTest::RegisterComponent(

CrxComponent* com,

TestComponents component,

- const Version& version,

+ const base::Version& version,

TestInstaller* installer) {

if (component == kTestComponent_abag) {

com->name = "test_abag";

@@ -288,7 +288,7 @@ TEST_F(ComponentUpdaterTest, CheckCrxSleep) {

EXPECT_EQ(ComponentUpdateService::kOk,

RegisterComponent(&com,

kTestComponent_abag,

- Version("1.1"),

+ base::Version("1.1"),

&installer));

// We loop twice, but there are no updates so we expect two sleep messages.

@@ -426,11 +426,13 @@ TEST_F(ComponentUpdaterTest, InstallCrx) {

TestInstaller installer1;

CrxComponent com1;

com1.observer = &observer1;

- RegisterComponent(&com1, kTestComponent_jebg, Version("0.9"), &installer1);

+ RegisterComponent(

+ &com1, kTestComponent_jebg, base::Version("0.9"), &installer1);

TestInstaller installer2;

CrxComponent com2;

com2.observer = &observer2;

- RegisterComponent(&com2, kTestComponent_abag, Version("2.2"), &installer2);

+ RegisterComponent(

+ &com2, kTestComponent_abag, base::Version("2.2"), &installer2);

test_configurator()->SetLoopCount(2);

component_updater()->Start();

@@ -495,7 +497,8 @@ TEST_F(ComponentUpdaterTest, ProdVersionCheck) {

TestInstaller installer;

CrxComponent com;

- RegisterComponent(&com, kTestComponent_jebg, Version("0.9"), &installer);

+ RegisterComponent(

+ &com, kTestComponent_jebg, base::Version("0.9"), &installer);

test_configurator()->SetLoopCount(1);

component_updater()->Start();

@@ -585,11 +588,13 @@ TEST_F(ComponentUpdaterTest, OnDemandUpdate) {

TestInstaller installer1;

CrxComponent com1;

com1.observer = &observer1;

- RegisterComponent(&com1, kTestComponent_abag, Version("2.2"), &installer1);

+ RegisterComponent(

+ &com1, kTestComponent_abag, base::Version("2.2"), &installer1);

TestInstaller installer2;

CrxComponent com2;

com2.observer = &observer2;

- RegisterComponent(&com2, kTestComponent_jebg, Version("0.9"), &installer2);

+ RegisterComponent(

+ &com2, kTestComponent_jebg, base::Version("0.9"), &installer2);

// No update normally.

test_configurator()->SetLoopCount(1);

@@ -810,11 +815,13 @@ TEST_F(ComponentUpdaterTest, CheckReRegistration) {

TestInstaller installer1;

CrxComponent com1;

com1.observer = &observer1;

- RegisterComponent(&com1, kTestComponent_jebg, Version("0.9"), &installer1);

+ RegisterComponent(

+ &com1, kTestComponent_jebg, base::Version("0.9"), &installer1);

TestInstaller installer2;

CrxComponent com2;

com2.observer = &observer2;

- RegisterComponent(&com2, kTestComponent_abag, Version("2.2"), &installer2);

+ RegisterComponent(

+ &com2, kTestComponent_abag, base::Version("2.2"), &installer2);

// Loop twice to issue two checks: (1) with original 0.9 version, update to

// 1.0, and do the second check (2) with the updated 1.0 version.

@@ -884,7 +891,7 @@ TEST_F(ComponentUpdaterTest, CheckReRegistration) {

EXPECT_EQ(ComponentUpdateService::kReplaced,

RegisterComponent(&com1,

kTestComponent_jebg,

- Version("2.2"),

+ base::Version("2.2"),

&installer3));

// Loop once just to notice the check happening with the re-register version.

@@ -942,7 +949,8 @@ TEST_F(ComponentUpdaterTest, DifferentialUpdate) {

VersionedTestInstaller installer;

CrxComponent com;

- RegisterComponent(&com, kTestComponent_ihfo, Version("0.0"), &installer);

+ RegisterComponent(

+ &com, kTestComponent_ihfo, base::Version("0.0"), &installer);

test_configurator()->SetLoopCount(3);

component_updater()->Start();

@@ -1013,7 +1021,8 @@ TEST_F(ComponentUpdaterTest, DifferentialUpdateFails) {

TestInstaller installer;

CrxComponent com;

- RegisterComponent(&com, kTestComponent_ihfo, Version("1.0"), &installer);

+ RegisterComponent(

+ &com, kTestComponent_ihfo, base::Version("1.0"), &installer);

test_configurator()->SetLoopCount(2);

component_updater()->Start();

@@ -1073,7 +1082,8 @@ TEST_F(ComponentUpdaterTest, CheckFailedInstallPing) {

// Loop twice to issue two checks: (1) with original 0.9 version

// and (2), which should retry with 0.9.

CrxComponent com;

- RegisterComponent(&com, kTestComponent_jebg, Version("0.9"), &installer);

+ RegisterComponent(

+ &com, kTestComponent_jebg, base::Version("0.9"), &installer);

test_configurator()->SetLoopCount(2);

component_updater()->Start();

@@ -1161,7 +1171,8 @@ TEST_F(ComponentUpdaterTest, DifferentialUpdateFailErrorcode) {

VersionedTestInstaller installer;

CrxComponent com;

- RegisterComponent(&com, kTestComponent_ihfo, Version("0.0"), &installer);

+ RegisterComponent(

+ &com, kTestComponent_ihfo, base::Version("0.0"), &installer);

test_configurator()->SetLoopCount(3);

component_updater()->Start();

@@ -1266,7 +1277,7 @@ TEST_F(ComponentUpdaterTest, ResourceThrottleDeletedNoUpdate) {

EXPECT_EQ(ComponentUpdateService::kOk,

RegisterComponent(&com,

kTestComponent_abag,

- Version("1.1"),

+ base::Version("1.1"),

&installer));

// The following two calls ensure that we don't do an update check via the

// timer, so the only update check should be the on-demand one.

@@ -1351,7 +1362,7 @@ TEST_F(ComponentUpdaterTest, ResourceThrottleLiveNoUpdate) {

EXPECT_EQ(ComponentUpdateService::kOk,

RegisterComponent(&com,

kTestComponent_abag,

- Version("1.1"),

+ base::Version("1.1"),

&installer));

// The following two calls ensure that we don't do an update check via the

// timer, so the only update check should be the on-demand one.