arc: Implement more methods in FakeIntentHelperInstance.

components/arc/test/fake_intent_helper_instance.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 "components/arc/test/fake_intent_helper_instance.h"
 
+#include <utility>
+
+#include "base/bind.h"
+#include "base/threading/sequenced_task_runner_handle.h"
+
 namespace arc {
 
 FakeIntentHelperInstance::FakeIntentHelperInstance() {}
 
+FakeIntentHelperInstance::HandledIntent::HandledIntent(
+    const mojom::IntentInfoPtr& intent,
+    const mojom::ActivityNamePtr& activity)
+    : action(intent->action),
+      categories(intent->categories ? *(intent->categories)
+                                    : std::vector<std::string>()),

[Daniel Erat, 2016/12/16 02:29:03]

this is also a bit ugly. all of these fields are base::Optional.

+      data(intent->data ? *(intent->data) : std::string()),
+      type(intent->type ? *(intent->type) : std::string()),
+      clip_data_uri(intent->clip_data_uri ? *(intent->clip_data_uri)
+                                          : std::string()),
+      package(activity->package_name),
+      activity(activity->activity_name ? *(activity->activity_name)
+                                       : std::string()) {}
+
+FakeIntentHelperInstance::HandledIntent::HandledIntent(
+    const HandledIntent& other) = default;
+
+FakeIntentHelperInstance::HandledIntent::~HandledIntent() = default;
+
 FakeIntentHelperInstance::Broadcast::Broadcast(const std::string& action,
                                                const std::string& package_name,
                                                const std::string& cls,
                                                const std::string& extras)
     : action(action), package_name(package_name), cls(cls), extras(extras) {}
 
 FakeIntentHelperInstance::Broadcast::Broadcast(const Broadcast& broadcast)
     : action(broadcast.action),
       package_name(broadcast.package_name),
       cls(broadcast.cls),
       extras(broadcast.extras) {}
 
 FakeIntentHelperInstance::Broadcast::~Broadcast() {}
 
+void FakeIntentHelperInstance::SetIntentHandlers(
+    const std::string& action,
+    std::vector<mojom::IntentHandlerInfoPtr> handlers) {
+  intent_handlers_[action] = std::move(handlers);
+}
+
 FakeIntentHelperInstance::~FakeIntentHelperInstance() {}
 
 void FakeIntentHelperInstance::AddPreferredPackage(
     const std::string& package_name) {}
 
 void FakeIntentHelperInstance::GetFileSizeDeprecated(
     const std::string& url,
     const GetFileSizeDeprecatedCallback& callback) {}
 
 void FakeIntentHelperInstance::HandleIntent(mojom::IntentInfoPtr intent,
-                                            mojom::ActivityNamePtr activity) {}
+                                            mojom::ActivityNamePtr activity) {
+  handled_intents_.emplace_back(HandledIntent(intent, activity));

[hidehiko, 2016/12/16 04:27:12]

emplace_back(intent, activity);

Could you take a look at my comment in .h file, too?

[Daniel Erat, 2016/12/16 05:41:04]

thanks; done.

+}
 
 void FakeIntentHelperInstance::HandleUrl(const std::string& url,
                                          const std::string& package_name) {}
 
 void FakeIntentHelperInstance::HandleUrlList(
     std::vector<mojom::UrlWithMimeTypePtr> urls,
     mojom::ActivityNamePtr activity,
     mojom::ActionType action) {}
 
 void FakeIntentHelperInstance::Init(mojom::IntentHelperHostPtr host_ptr) {}
 
 void FakeIntentHelperInstance::OpenFileToReadDeprecated(
     const std::string& url,
     const OpenFileToReadDeprecatedCallback& callback) {}
 
 void FakeIntentHelperInstance::RequestActivityIcons(
     std::vector<mojom::ActivityNamePtr> activities,
     ::arc::mojom::ScaleFactor scale_factor,
     const RequestActivityIconsCallback& callback) {}
 
 void FakeIntentHelperInstance::RequestIntentHandlerList(
     mojom::IntentInfoPtr intent,
-    const RequestIntentHandlerListCallback& callback) {}
+    const RequestIntentHandlerListCallback& callback) {
+  std::vector<mojom::IntentHandlerInfoPtr> handlers;
+  auto it = intent_handlers_.find(intent->action);
+  if (it != intent_handlers_.end()) {
+    handlers = std::move(it->second);
+    intent_handlers_.erase(it);
+  }
+  base::SequencedTaskRunnerHandle::Get()->PostTask(

[hidehiko, 2016/12/16 04:27:12]

In ARC code, ThreadTaskRunnerHandle is used. How about to use it for
consistency?

[Daniel Erat, 2016/12/16 05:41:04]

sure. ThreadTaskRunnerHandle has a comment recommending using this instead, but
i agree consistency is better. (are these methods callable from any thread?)

+      FROM_HERE, base::Bind(callback, base::Passed(std::move(handlers))));

[Daniel Erat, 2016/12/16 02:29:03]

base::Passed(std::move()) looks very wrong to me, but i couldn't find any other
way to get this to compile. :-(

[hidehiko, 2016/12/16 04:27:12]

rvalue is actually needed for base::Passed, so it is necessary. Or,
alternatively,

base::Passed(&handlers)

[Daniel Erat, 2016/12/16 05:41:04]

Acknowledged.

+}
 
 void FakeIntentHelperInstance::RequestUrlHandlerList(
     const std::string& url,
     const RequestUrlHandlerListCallback& callback) {}
 
 void FakeIntentHelperInstance::RequestUrlListHandlerList(
     std::vector<mojom::UrlWithMimeTypePtr> urls,
     const RequestUrlListHandlerListCallback& callback) {}
 
 void FakeIntentHelperInstance::SendBroadcast(const std::string& action,
                                              const std::string& package_name,
                                              const std::string& cls,
                                              const std::string& extras) {
   broadcasts_.emplace_back(action, package_name, cls, extras);
 }
 
 }  // namespace arc