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

Side by Side Diff: chrome/browser/supervised_user/supervised_user_service_unittest.cc

Issue 2054773002: Replace the WAS_INSTALLED_BY_CUSTODIAN creation flag with a pref (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixing the build Created 4 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chrome/browser/supervised_user/supervised_user_service.h" 5 #include "chrome/browser/supervised_user/supervised_user_service.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 23 matching lines...) Expand all
34 #include "components/signin/core/browser/fake_profile_oauth2_token_service.h" 34 #include "components/signin/core/browser/fake_profile_oauth2_token_service.h"
35 #include "components/signin/core/browser/signin_manager.h" 35 #include "components/signin/core/browser/signin_manager.h"
36 #include "components/version_info/version_info.h" 36 #include "components/version_info/version_info.h"
37 #include "content/public/test/test_browser_thread_bundle.h" 37 #include "content/public/test/test_browser_thread_bundle.h"
38 #include "content/public/test/test_utils.h" 38 #include "content/public/test/test_utils.h"
39 #include "testing/gtest/include/gtest/gtest.h" 39 #include "testing/gtest/include/gtest/gtest.h"
40 40
41 #if defined(ENABLE_EXTENSIONS) 41 #if defined(ENABLE_EXTENSIONS)
42 #include "chrome/browser/extensions/extension_service.h" 42 #include "chrome/browser/extensions/extension_service.h"
43 #include "chrome/browser/extensions/extension_service_test_base.h" 43 #include "chrome/browser/extensions/extension_service_test_base.h"
44 #include "chrome/browser/extensions/extension_util.h"
44 #include "chrome/browser/extensions/unpacked_installer.h" 45 #include "chrome/browser/extensions/unpacked_installer.h"
45 #include "chrome/common/extensions/features/feature_channel.h" 46 #include "chrome/common/extensions/features/feature_channel.h"
46 #include "extensions/common/extension.h" 47 #include "extensions/common/extension.h"
47 #include "extensions/common/extension_builder.h" 48 #include "extensions/common/extension_builder.h"
48 #include "extensions/common/manifest_constants.h" 49 #include "extensions/common/manifest_constants.h"
49 #endif 50 #endif
50 51
51 using content::MessageLoopRunner; 52 using content::MessageLoopRunner;
52 53
53 namespace { 54 namespace {
(...skipping 346 matching lines...) Expand 10 before | Expand all | Expand 10 after
400 builder.SetManifest(std::move(source)).Build(); 401 builder.SetManifest(std::move(source)).Build();
401 return extension; 402 return extension;
402 } 403 }
403 404
404 scoped_refptr<extensions::Extension> MakeExtension(bool by_custodian) { 405 scoped_refptr<extensions::Extension> MakeExtension(bool by_custodian) {
405 std::unique_ptr<base::DictionaryValue> manifest = 406 std::unique_ptr<base::DictionaryValue> manifest =
406 extensions::DictionaryBuilder() 407 extensions::DictionaryBuilder()
407 .Set(extensions::manifest_keys::kName, "Extension") 408 .Set(extensions::manifest_keys::kName, "Extension")
408 .Set(extensions::manifest_keys::kVersion, "1.0") 409 .Set(extensions::manifest_keys::kVersion, "1.0")
409 .Build(); 410 .Build();
410 int creation_flags = extensions::Extension::NO_FLAGS; 411
411 if (by_custodian)
412 creation_flags |= extensions::Extension::WAS_INSTALLED_BY_CUSTODIAN;
413 extensions::ExtensionBuilder builder; 412 extensions::ExtensionBuilder builder;
414 scoped_refptr<extensions::Extension> extension = 413 scoped_refptr<extensions::Extension> extension =
415 builder.SetManifest(std::move(manifest)) 414 builder.SetManifest(std::move(manifest))
416 .AddFlags(creation_flags)
417 .Build(); 415 .Build();
416 extensions::util::SetWasInstalledByCustodian(extension->id(),
417 profile_.get(), by_custodian);
418
418 return extension; 419 return extension;
419 } 420 }
420 421
421 bool is_supervised_; 422 bool is_supervised_;
422 extensions::ScopedCurrentChannel channel_; 423 extensions::ScopedCurrentChannel channel_;
423 SiteListObserver site_list_observer_; 424 SiteListObserver site_list_observer_;
424 SupervisedUserURLFilterObserver url_filter_observer_; 425 SupervisedUserURLFilterObserver url_filter_observer_;
425 }; 426 };
426 427
427 class SupervisedUserServiceExtensionTestUnsupervised 428 class SupervisedUserServiceExtensionTestUnsupervised
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
582 EXPECT_EQ(title2, site_list_observer_.site_lists()[0]->title()); 583 EXPECT_EQ(title2, site_list_observer_.site_lists()[0]->title());
583 EXPECT_TRUE(site_list_observer_.site_lists()[0]->entry_point().is_empty()); 584 EXPECT_TRUE(site_list_observer_.site_lists()[0]->entry_point().is_empty());
584 585
585 url_filter_observer_.Wait(); 586 url_filter_observer_.Wait();
586 EXPECT_EQ(SupervisedUserURLFilter::WARN, 587 EXPECT_EQ(SupervisedUserURLFilter::WARN,
587 url_filter->GetFilteringBehaviorForURL(youtube_url)); 588 url_filter->GetFilteringBehaviorForURL(youtube_url));
588 EXPECT_EQ(SupervisedUserURLFilter::ALLOW, 589 EXPECT_EQ(SupervisedUserURLFilter::ALLOW,
589 url_filter->GetFilteringBehaviorForURL(moose_url)); 590 url_filter->GetFilteringBehaviorForURL(moose_url));
590 } 591 }
591 #endif // defined(ENABLE_EXTENSIONS) 592 #endif // defined(ENABLE_EXTENSIONS)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698