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/supervised_user_blacklist.h"
+#include "chrome/browser/supervised_user/report_url_request_creator.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"
 
 #if defined(ENABLE_EXTENSIONS)
@@ skipping 79 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);
 
+  // Adds a report url request to the SafeSearch API.
+  void AddReportUrlRequest(const GURL& url, const SuccessCallback& callback);
+
   // 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 AddReportUrlRequestCreator(scoped_ptr<ReportUrlRequestCreator> creator);

[Marc Treib, 2016/03/18 11:28:54]

All this complexity with a "RequestCreator" interface, and handling multiple
instances of them in the SUService, isn't necessary here. There will only ever
be one of them, and there is only one possible implementation (in contrast to
permission requests, which have different implementations for legacy SUs vs.
child accounts).

[atanasova, 2016/03/21 10:42:25]

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 11 matching lines @@
   FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest, ClearOmitOnRegistration);
   FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest,
                            ChangesIncludedSessionOnChangedSettings);
   FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest,
                            ChangesSyncSessionStateOnChangedSettings);
   FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceExtensionTest,
                            ExtensionManagementPolicyProvider);
 
   using CreatePermissionRequestCallback =
       base::Callback<void(PermissionRequestCreator*, const SuccessCallback&)>;
+  using CreateReportUrlRequestCallback =
+      base::Callback<void(ReportUrlRequestCreator*, const SuccessCallback&)>;
 
   // A bridge from the UI thread to the SupervisedUserURLFilters, one of which
   // lives on the IO thread. This class mediates access to them and makes sure
   // they are kept in sync.
   class URLFilterContext {
    public:
     URLFilterContext();
     ~URLFilterContext();
 
     SupervisedUserURLFilter* ui_url_filter() const;
@@ skipping 76 matching lines @@
   void AddPermissionRequestInternal(
       const CreatePermissionRequestCallback& create_request,
       const SuccessCallback& callback,
       size_t index);
   void OnPermissionRequestIssued(
       const CreatePermissionRequestCallback& create_request,
       const SuccessCallback& callback,
       size_t index,
       bool success);
 
+  void AddReportUrlRequestInternal(
+      const CreateReportUrlRequestCallback& report_url_request,
+      const SuccessCallback& callback,
+      size_t index);
+  void OnReportUrlRequestIssues(
+      const CreateReportUrlRequestCallback& report_url_request,
+      const SuccessCallback& callback,
+      size_t index,
+      bool success);
+
   void OnSupervisedUserIdChanged();
 
   void OnDefaultFilteringBehaviorChanged();
 
   void OnSafeSitesSettingChanged();
 
   void OnSiteListsChanged(
       const std::vector<scoped_refptr<SupervisedUserSiteList>>& site_lists);
 
   // Asynchronously loads a blacklist from a binary file at |path| and applies
@@ skipping 71 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.
+  std::vector<scoped_ptr<ReportUrlRequestCreator>> report_url_creators_;
+
   base::ObserverList<SupervisedUserServiceObserver> observer_list_;
 
   base::WeakPtrFactory<SupervisedUserService> weak_ptr_factory_;
 };
 
 #endif  // CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_