[Sharesheet] Close sharesheet on button press.

This CL closes the sharesheet once an app is launched.

Bug: 1097623
Change-Id: I72ab0f8ff39569a4fd14a1f3f3efd1df46af7f0a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2338154Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Melissa Zhang <melzhang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795725}

chrome/browser/sharesheet/sharesheet_controller.h

@@ -19,7 +19,7 @@ class SharesheetController {
 
   // Called by ShareAction to notify SharesheetBubbleView that ShareAction
   // has completed.
-  virtual void ShareActionCompleted() = 0;
+  virtual void CloseSharesheet() = 0;
 };
 
 }  // namespace sharesheet

chrome/browser/sharesheet/sharesheet_service.cc

@@ -100,6 +100,7 @@ void SharesheetService::OnTargetSelected(uint32_t delegate_id,
             WindowOpenDisposition::NEW_WINDOW,
             /*prefer_container=*/true),
         std::move(intent), launch_source, display::kDefaultDisplayId);
+    delegate->CloseSharesheet();
   }
 }
 

chrome/browser/sharesheet/sharesheet_service_delegate.cc

@@ -63,7 +63,7 @@ uint32_t SharesheetServiceDelegate::GetId() {
   return id_;
 }
 
-void SharesheetServiceDelegate::ShareActionCompleted() {
+void SharesheetServiceDelegate::CloseSharesheet() {
   sharesheet_bubble_view_->CloseBubble();
 }
 

chrome/browser/sharesheet/sharesheet_service_delegate.h

@@ -52,7 +52,7 @@ class SharesheetServiceDelegate : public SharesheetController {
 
   // SharesheetController overrides
   uint32_t GetId() override;
-  void ShareActionCompleted() override;
+  void CloseSharesheet() override;
 
  private:
   const uint32_t id_;

chrome/browser/ui/views/sharesheet_bubble_view.cc

@@ -190,7 +190,6 @@ void SharesheetBubbleView::ButtonPressed(views::Button* sender,
   delegate_->OnTargetSelected(targets_[sender->tag()].launch_name, type,
                               std::move(intent_), share_action_view_);
   intent_.reset();
-  RequestFocus();
 }
 
 std::unique_ptr<views::NonClientFrameView>