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

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

Issue 1443033004: Supervised User whitelists: update to json format v2 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove l10n Created 5 years, 1 month 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_whitelist_service.h" 5 #include "chrome/browser/supervised_user/supervised_user_whitelist_service.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 104
105 const bool result = whitelists.insert(std::make_pair(id, name)).second; 105 const bool result = whitelists.insert(std::make_pair(id, name)).second;
106 DCHECK(result); 106 DCHECK(result);
107 } 107 }
108 108
109 return whitelists; 109 return whitelists;
110 } 110 }
111 111
112 void SupervisedUserWhitelistService::LoadWhitelistForTesting( 112 void SupervisedUserWhitelistService::LoadWhitelistForTesting(
113 const std::string& id, 113 const std::string& id,
114 const base::string16& title,
114 const base::FilePath& path) { 115 const base::FilePath& path) {
115 bool result = registered_whitelists_.insert(id).second; 116 bool result = registered_whitelists_.insert(id).second;
116 DCHECK(result); 117 DCHECK(result);
117 OnWhitelistReady(id, path); 118 OnWhitelistReady(id, title, path);
118 } 119 }
119 120
120 void SupervisedUserWhitelistService::UnloadWhitelist(const std::string& id) { 121 void SupervisedUserWhitelistService::UnloadWhitelist(const std::string& id) {
121 bool result = registered_whitelists_.erase(id) > 0u; 122 bool result = registered_whitelists_.erase(id) > 0u;
122 DCHECK(result); 123 DCHECK(result);
123 loaded_whitelists_.erase(id); 124 loaded_whitelists_.erase(id);
124 NotifyWhitelistsChanged(); 125 NotifyWhitelistsChanged();
125 } 126 }
126 127
127 // static 128 // static
(...skipping 193 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 void SupervisedUserWhitelistService::NotifyWhitelistsChanged() { 322 void SupervisedUserWhitelistService::NotifyWhitelistsChanged() {
322 std::vector<scoped_refptr<SupervisedUserSiteList>> whitelists; 323 std::vector<scoped_refptr<SupervisedUserSiteList>> whitelists;
323 GetLoadedWhitelists(&whitelists); 324 GetLoadedWhitelists(&whitelists);
324 325
325 for (const auto& callback : site_lists_changed_callbacks_) 326 for (const auto& callback : site_lists_changed_callbacks_)
326 callback.Run(whitelists); 327 callback.Run(whitelists);
327 } 328 }
328 329
329 void SupervisedUserWhitelistService::OnWhitelistReady( 330 void SupervisedUserWhitelistService::OnWhitelistReady(
330 const std::string& id, 331 const std::string& id,
332 const base::string16& title,
331 const base::FilePath& whitelist_path) { 333 const base::FilePath& whitelist_path) {
332 // If we did not register the whitelist or it has been unregistered in the 334 // If we did not register the whitelist or it has been unregistered in the
333 // mean time, ignore it. 335 // mean time, ignore it.
334 if (registered_whitelists_.count(id) == 0u) 336 if (registered_whitelists_.count(id) == 0u)
335 return; 337 return;
336 338
337 SupervisedUserSiteList::Load( 339 SupervisedUserSiteList::Load(
338 whitelist_path, 340 title, whitelist_path,
339 base::Bind(&SupervisedUserWhitelistService::OnWhitelistLoaded, 341 base::Bind(&SupervisedUserWhitelistService::OnWhitelistLoaded,
340 weak_ptr_factory_.GetWeakPtr(), id, base::TimeTicks::Now())); 342 weak_ptr_factory_.GetWeakPtr(), id, base::TimeTicks::Now()));
341 } 343 }
342 344
343 void SupervisedUserWhitelistService::OnWhitelistLoaded( 345 void SupervisedUserWhitelistService::OnWhitelistLoaded(
344 const std::string& id, 346 const std::string& id,
345 base::TimeTicks start_time, 347 base::TimeTicks start_time,
346 const scoped_refptr<SupervisedUserSiteList>& whitelist) { 348 const scoped_refptr<SupervisedUserSiteList>& whitelist) {
347 if (!whitelist) { 349 if (!whitelist) {
348 LOG(WARNING) << "Couldn't load whitelist " << id; 350 LOG(WARNING) << "Couldn't load whitelist " << id;
349 return; 351 return;
350 } 352 }
351 353
352 UMA_HISTOGRAM_TIMES("ManagedUsers.Whitelist.TotalLoadDuration", 354 UMA_HISTOGRAM_TIMES("ManagedUsers.Whitelist.TotalLoadDuration",
353 base::TimeTicks::Now() - start_time); 355 base::TimeTicks::Now() - start_time);
354 356
355 // If the whitelist has been unregistered in the mean time, ignore it. 357 // If the whitelist has been unregistered in the mean time, ignore it.
356 if (registered_whitelists_.count(id) == 0u) 358 if (registered_whitelists_.count(id) == 0u)
357 return; 359 return;
358 360
359 loaded_whitelists_[id] = whitelist; 361 loaded_whitelists_[id] = whitelist;
360 NotifyWhitelistsChanged(); 362 NotifyWhitelistsChanged();
361 } 363 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698