Add "Open with <ARC-app-name>" items to the context menu

components/arc/intent_helper/arc_intent_helper_bridge.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/intent_helper/arc_intent_helper_bridge.h"
 
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
+#include "components/arc/intent_helper/link_handler_model_impl.h"
 #include "url/gurl.h"
 
 namespace arc {
 
 ArcIntentHelperBridge::ArcIntentHelperBridge(ArcBridgeService* bridge_service)
     : ArcService(bridge_service), binding_(this) {
   arc_bridge_service()->AddObserver(this);
 }
 
 ArcIntentHelperBridge::~ArcIntentHelperBridge() {
   arc_bridge_service()->RemoveObserver(this);
 }
 
 void ArcIntentHelperBridge::OnIntentHelperInstanceReady() {
+  ash::Shell::GetInstance()->open_with_menu_controller()->SetDelegate(this);
   arc_bridge_service()->intent_helper_instance()->Init(
       binding_.CreateInterfacePtrAndBind());
 }
 
+void ArcIntentHelperBridge::OnIntentHelperInstanceClosed() {
+  ash::Shell::GetInstance()->open_with_menu_controller()->SetDelegate(nullptr);

[hidehiko, 2016/04/20 18:10:18]

Could you verify this is always called before this instance's destruction?

Could you delete the instance returned by CreateModel?
It has the reference to ArcBridgeService via raw ptr, but there looks no path to
destruct it on ABS's destruction?

[Yusuke Sato, 2016/04/22 05:27:48]

Done. Copied line 30 to the destructor following a suggestion from Luis.

(Luis: since calling the ash method with nullptr twice is okay, I didn't
introduce any local boolean flags for this.)

As for the second question, I stopped passing ABS* raw pointer to the model impl
class (also based on Luis' suggestion.)

+}
+
 void ArcIntentHelperBridge::OnOpenUrl(const mojo::String& url) {
   GURL gurl(url.get());
   ash::Shell::GetInstance()->delegate()->OpenUrl(gurl);
 }
 
+std::unique_ptr<ash::LinkHandlerModel> ArcIntentHelperBridge::CreateModel(
+    const GURL& url) {
+  std::unique_ptr<LinkHandlerModelImpl> impl(
+      new LinkHandlerModelImpl(arc_bridge_service()));
+  if (!impl->Init(url))
+    return nullptr;
+  return std::move(impl);
+}
+
 }  // namespace arc