Read share targets from prefstore, and filter by engagement.

chrome/browser/webshare/share_service_impl_unittest.cc

 // Copyright 2016 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.
 
 #include <memory>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/webshare/share_service_impl.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
+#include "components/prefs/pref_registry_simple.h"
+#include "components/prefs/scoped_user_pref_update.h"
+#include "components/prefs/testing_pref_service.h"
 #include "mojo/public/cpp/bindings/interface_request.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 namespace {
 
 constexpr char kTitle[] = "My title";
 constexpr char kText[] = "My text";
 constexpr char kUrlSpec[] = "https://www.google.com/";
 
 class ShareServiceTestImpl : public ShareServiceImpl {
  public:
   static ShareServiceTestImpl* Create(
       blink::mojom::ShareServiceRequest request) {
     std::unique_ptr<ShareServiceTestImpl> share_service_helper =
         base::MakeUnique<ShareServiceTestImpl>();
     ShareServiceTestImpl* share_service_helper_raw = share_service_helper.get();
     mojo::MakeStrongBinding(std::move(share_service_helper),
                             std::move(request));
     return share_service_helper_raw;
   }
 
   void set_picker_result(base::Optional<base::string16> result) {
     picker_result_ = result;
   }
 
+  void SetUpPrefs() {
+    pref_service_.reset(new TestingPrefServiceSimple());
+    pref_service_->registry()->RegisterDictionaryPref(
+        prefs::kWebShareVisitedTargets);
+  }
+
+  void AddShareTarget(std::string manifest_url, std::string url_template) {
+    DictionaryPrefUpdate update(GetPrefService(),
+                                prefs::kWebShareVisitedTargets);
+    base::DictionaryValue* share_target_dict = update.Get();
+
+    std::unique_ptr<base::DictionaryValue> origin_dict(
+        new base::DictionaryValue);
+
+    constexpr char kUrlTemplateKey[] = "url_template";
+    origin_dict->SetStringWithoutPathExpansion(kUrlTemplateKey, url_template);
+
+    share_target_dict->SetWithoutPathExpansion(manifest_url,
+                                               std::move(origin_dict));
+  }
+
+  void AddEngagementForTarget(std::string manifest_url,
+                              blink::mojom::EngagementLevel level) {
+    engagement_map[manifest_url] = level;
+  }
+
+  void ClearEngagements() { engagement_map.clear(); }
+
   const std::string& GetLastUsedTargetURL() { return last_used_target_url_; }
 
  private:
   base::Optional<base::string16> picker_result_ = base::nullopt;
   std::string last_used_target_url_;
+  std::unique_ptr<TestingPrefServiceSimple> pref_service_;
+  std::map<std::string, blink::mojom::EngagementLevel> engagement_map;
 
   void ShowPickerDialog(
       const std::vector<base::string16>& targets,
       const base::Callback<void(base::Optional<base::string16>)>& callback)
       override {
+    // TODO check targets were what was expected.

[Matt Giuca, 2017/02/01 07:30:20]

Should this be done?

[constantina, 2017/02/02 00:43:48]

Yep, will do in next patch.

     callback.Run(picker_result_);
   }
 
   void OpenTargetURL(const GURL& target_url) override {
     last_used_target_url_ = target_url.spec();
   }
+
+  PrefService* GetPrefService() override { return pref_service_.get(); }
+
+  blink::mojom::EngagementLevel GetEngagementLevel(GURL url) override {
+    return engagement_map[url.spec()];
+  }
 };
 
 class ShareServiceImplUnittest : public ChromeRenderViewHostTestHarness {
  public:
   ShareServiceImplUnittest() = default;
   ~ShareServiceImplUnittest() override = default;
 
   void SetUp() override {
     ChromeRenderViewHostTestHarness::SetUp();
 
     share_service_helper_ =
         ShareServiceTestImpl::Create(mojo::MakeRequest(&share_service_));
+    share_service_helper_->SetUpPrefs();
   }
 
-  void TearDown() override { ChromeRenderViewHostTestHarness::TearDown(); }
+  void TearDown() override {
+    ChromeRenderViewHostTestHarness::TearDown();
+    share_service_helper_->ClearEngagements();
+  }
 
   void DidShare(const std::string& expected_target_url,
                 const base::Optional<std::string>& expected_param,
                 const base::Optional<std::string>& param) {
     EXPECT_EQ(expected_param, param);
     std::string target_url = share_service_helper_->GetLastUsedTargetURL();
     EXPECT_EQ(expected_target_url, target_url);
 
     if (!on_callback_.is_null())
       on_callback_.Run();
   }
 
   blink::mojom::ShareServicePtr share_service_;
   ShareServiceTestImpl* share_service_helper_;
   base::Closure on_callback_;
 };
 
 }  // namespace
 
 // Basic test to check the Share method calls the callback with the expected
 // parameters.
 TEST_F(ShareServiceImplUnittest, ShareCallbackParams) {
+  std::string base_url = "https://wicg.github.io/web-share-target/demos/";
+  std::string url_template =
+      "sharetarget.html?title={title}&text={text}&url={url}";
   std::string expected_url =
       "https://wicg.github.io/web-share-target/demos/"
       "sharetarget.html?title=My%20title&text=My%20text&url=https%3A%2F%2Fwww."
       "google.com%2F";
-  share_service_helper_->set_picker_result(
-      base::ASCIIToUTF16("https://wicg.github.io/web-share-target/demos/"));
+  share_service_helper_->set_picker_result(base::ASCIIToUTF16(base_url));
+  share_service_helper_->AddShareTarget(base_url, url_template);
+  share_service_helper_->AddEngagementForTarget(
+      base_url, blink::mojom::EngagementLevel::LOW);
 
-  const GURL url(kUrlSpec);
   base::Callback<void(const base::Optional<std::string>&)> callback =
       base::Bind(&ShareServiceImplUnittest::DidShare, base::Unretained(this),
                  expected_url, base::Optional<std::string>());
 
   base::RunLoop run_loop;
   on_callback_ = run_loop.QuitClosure();
 
+  const GURL url(kUrlSpec);
   share_service_->Share(kTitle, kText, url, callback);
 
   run_loop.Run();
 }
 
 // Tests the result of cancelling the share in the picker UI.
 TEST_F(ShareServiceImplUnittest, ShareCancel) {
   // Ask that the dialog be cancelled.
   share_service_helper_->set_picker_result(base::nullopt);
 
@@ skipping 208 matching lines @@
   EXPECT_TRUE(succeeded);
   EXPECT_EQ("%C3%A9", url_template_filled);
 
   // U+1F4A9
   url_template = "{title}";
   succeeded = ShareServiceImpl::ReplacePlaceholders(
       url_template, "\xf0\x9f\x92\xa9", kText, url, &url_template_filled);
   EXPECT_TRUE(succeeded);
   EXPECT_EQ("%F0%9F%92%A9", url_template_filled);
 }