Add report URL to safe search API functionality.

chrome/browser/supervised_user/supervised_user_service.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_
 #define CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_
 
 #include <stddef.h>
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/observer_list.h"
 #include "base/scoped_observer.h"
 #include "base/strings/string16.h"
 #include "build/build_config.h"
+#include "chrome/browser/supervised_user/experimental/safe_search_url_reporter.h"
 #include "chrome/browser/supervised_user/experimental/supervised_user_blacklist.h"
 #include "chrome/browser/supervised_user/supervised_user_url_filter.h"
 #include "chrome/browser/supervised_user/supervised_users.h"
 #include "chrome/browser/ui/browser_list_observer.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "components/prefs/pref_change_registrar.h"
 #include "components/sync_driver/sync_service_observer.h"
 #include "components/sync_driver/sync_type_preference_provider.h"
 #include "net/url_request/url_request_context_getter.h"
 
@@ skipping 80 matching lines @@
     return whitelists_;
   }
 
   // Whether the user can request to get access to blocked URLs or to new
   // extensions.
   bool AccessRequestsEnabled();
 
   // Adds an access request for the given URL.
   void AddURLAccessRequest(const GURL& url, const SuccessCallback& callback);
 
+  // Reports url to the SafeSearch API.

[Marc Treib, 2016/03/21 12:32:21]

|url| (pipes around the variable name)

I'd also add some explanation on the semantics of "report", i.e. the user thinks
this is an inappropriate URL.

[atanasova, 2016/03/22 15:45:41]

Done.

+  void ReportUrl(const GURL& url, const SuccessCallback& callback);

[Marc Treib, 2016/03/21 12:32:21]

nit: ReportURL (capitalized), to be consistent with the method above.

[atanasova, 2016/03/22 15:45:41]

Done.

+
   // Adds an update request for the given WebStore item (App/Extension).
   void AddExtensionUpdateRequest(const std::string& extension_id,
                                  const base::Version& version,
                                  const SuccessCallback& callback);
 
   // Same as above, but without a callback, just logging errors on failure.
   void AddExtensionUpdateRequest(const std::string& extension_id,
                                  const base::Version& version);
 
   // Get the string used to identify an extension update request. Public for
@@ skipping 40 matching lines @@
 
   void AddNavigationBlockedCallback(const NavigationBlockedCallback& callback);
   void DidBlockNavigation(content::WebContents* web_contents);
 
   void AddObserver(SupervisedUserServiceObserver* observer);
   void RemoveObserver(SupervisedUserServiceObserver* observer);
 
   void AddPermissionRequestCreator(
       scoped_ptr<PermissionRequestCreator> creator);
 
+  void SetSafeSearchUrlReporter(scoped_ptr<SafeSearchUrlReporter> creator);

[Marc Treib, 2016/03/21 12:32:21]

s/creator/reporter/

[atanasova, 2016/03/22 15:45:41]

Done.

+
   // ProfileKeyedService override:
   void Shutdown() override;
 
   // SyncTypePreferenceProvider implementation:
   syncer::ModelTypeSet GetPreferredDataTypes() const override;
 
 #if !defined(OS_ANDROID)
   // sync_driver::SyncServiceObserver implementation:
   void OnStateChanged() override;
 
@@ skipping 198 matching lines @@
   SupervisedUserBlacklist blacklist_;
   scoped_ptr<FileDownloader> blacklist_downloader_;
 
   scoped_ptr<SupervisedUserWhitelistService> whitelist_service_;
 
   std::vector<scoped_refptr<SupervisedUserSiteList>> whitelists_;
 
   // Used to create permission requests.
   ScopedVector<PermissionRequestCreator> permissions_creators_;
 
+  // Used to create report url requests.

[Marc Treib, 2016/03/21 12:32:21]

"Used to report inappropriate URLs to the SafeSearch API"?

[atanasova, 2016/03/22 15:45:41]

Done.

+  scoped_ptr<SafeSearchUrlReporter> url_reporter_;
+
   base::ObserverList<SupervisedUserServiceObserver> observer_list_;
 
   base::WeakPtrFactory<SupervisedUserService> weak_ptr_factory_;
 };
 
 #endif  // CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_