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

Side by Side Diff: chrome/browser/extensions/external_policy_extension_loader_unittest.cc

Issue 6370009: Add in-profile external_extensions.json file for Chrome OS OEM customization. (Closed) Base URL: http://git.chromium.org/git/chromium.git
Patch Set: Address review comments. Created 9 years, 11 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
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 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 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 <string> 5 #include <string>
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/message_loop.h" 8 #include "base/message_loop.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "base/version.h" 10 #include "base/version.h"
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 // Provider returns the correct location when asked. 78 // Provider returns the correct location when asked.
79 Extension::Location location1; 79 Extension::Location location1;
80 scoped_ptr<Version> version1; 80 scoped_ptr<Version> version1;
81 provider_->GetExtensionDetails(id, &location1, &version1); 81 provider_->GetExtensionDetails(id, &location1, &version1);
82 EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, location1); 82 EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, location1);
83 EXPECT_FALSE(version1.get()); 83 EXPECT_FALSE(version1.get());
84 84
85 // Remove the extension from our list. 85 // Remove the extension from our list.
86 StringValue ext_str(id + ";" + update_url.spec()); 86 StringValue ext_str(id + ";" + update_url.spec());
87 EXPECT_NE(-1, remaining_extensions->Remove(ext_str)); 87 EXPECT_NE(-1, remaining_extensions->Remove(ext_str));
88 } 88 }
89 89
90 virtual void OnExternalProviderReady() { 90 virtual void OnExternalProviderReady() {
91 EXPECT_TRUE(provider_->IsReady()); 91 EXPECT_TRUE(provider_->IsReady());
92 } 92 }
93 93
94 private: 94 private:
95 ListValue* remaining_extensions; 95 ListValue* remaining_extensions;
96 96
97 scoped_ptr<TestingProfile> profile_; 97 scoped_ptr<TestingProfile> profile_;
98 98
99 scoped_ptr<ExternalExtensionProviderImpl> provider_; 99 scoped_ptr<ExternalExtensionProviderImpl> provider_;
100 100
101 DISALLOW_COPY_AND_ASSIGN(MockExternalPolicyExtensionProviderVisitor); 101 DISALLOW_COPY_AND_ASSIGN(MockExternalPolicyExtensionProviderVisitor);
102 }; 102 };
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa;http#//www.example.com/crx")); 136 "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa;http#//www.example.com/crx"));
137 forced_extensions.Append(Value::CreateStringValue( 137 forced_extensions.Append(Value::CreateStringValue(
138 "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa")); 138 "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"));
139 forced_extensions.Append(Value::CreateStringValue( 139 forced_extensions.Append(Value::CreateStringValue(
140 "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaahttp#//www.example.com/crx")); 140 "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaahttp#//www.example.com/crx"));
141 141
142 MockExternalPolicyExtensionProviderVisitor mv; 142 MockExternalPolicyExtensionProviderVisitor mv;
143 std::set<std::string> empty; 143 std::set<std::string> empty;
144 mv.Visit(&forced_extensions, &valid_extensions, empty); 144 mv.Visit(&forced_extensions, &valid_extensions, empty);
145 } 145 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698