capture_mode: opens an image editor from an edit button.

Opens an image editor, an Essential App's Backlight from an edit button in a screenshot notification.

Important Note: the edit button is only available
for image screen capturing but not video.

Testing:
- Screen recording showing that edit button open the screenshot image in Backlight
https://bugs.chromium.org/p/chromium/issues/attachment?aid=469945&signed_aid=7kntFxv5qL2TWN8JYNxQ4Q==&inline=1

Bug: 1135247
Change-Id: Ia9f48cd82021b96fc3bbda8f140e90477b1a0152
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2457306
Commit-Queue: Cattalyya Nuengsigkapian <cattalyya@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815845}

ash/capture_mode/capture_mode_controller.cc

@@ -455,9 +455,10 @@ void CaptureModeController::HandleNotificationClicked(
     // Show the item in the folder.
     delegate_->ShowScreenCaptureItemInFolder(screen_capture_path);
   } else {
-    // TODO: fill in here.
+    // TODO(crbug.com/1136666): Remove edit button for video recording.
     switch (button_index.value()) {
       case NotificationButtonIndex::BUTTON_EDIT:
+        delegate_->OpenScreenshotInImageEditor(screen_capture_path);
         break;
       case NotificationButtonIndex::BUTTON_DELETE:
         DeleteFileAsync(screen_capture_path);

ash/capture_mode/test_capture_mode_delegate.cc

@@ -16,6 +16,9 @@ base::FilePath TestCaptureModeDelegate::GetActiveUserDownloadsDir() const {
 void TestCaptureModeDelegate::ShowScreenCaptureItemInFolder(
     const base::FilePath& file_path) {}
 
+void TestCaptureModeDelegate::OpenScreenshotInImageEditor(
+    const base::FilePath& file_path) {}
+
 bool TestCaptureModeDelegate::Uses24HourFormat() const {
   return false;
 }

ash/capture_mode/test_capture_mode_delegate.h

@@ -19,6 +19,7 @@ class TestCaptureModeDelegate : public CaptureModeDelegate {
   // CaptureModeDelegate:
   base::FilePath GetActiveUserDownloadsDir() const override;
   void ShowScreenCaptureItemInFolder(const base::FilePath& file_path) override;
+  void OpenScreenshotInImageEditor(const base::FilePath& file_path) override;
   bool Uses24HourFormat() const override;
   bool IsCaptureAllowed(const aura::Window* window,
                         const gfx::Rect& bounds,

ash/public/cpp/capture_mode_delegate.h

@@ -36,6 +36,9 @@ class ASH_PUBLIC_EXPORT CaptureModeDelegate {
   virtual void ShowScreenCaptureItemInFolder(
       const base::FilePath& file_path) = 0;
 
+  // Opens the screenshot item in an image editor.
+  virtual void OpenScreenshotInImageEditor(const base::FilePath& file_path) = 0;
+
   // Returns true if the current user is using the 24-hour format (i.e. 14:00
   // vs. 2:00 PM). This is used to build the file name of the captured image or
   // video.

chrome/browser/ui/ash/chrome_capture_mode_delegate.cc

@@ -6,7 +6,11 @@
 
 #include "base/files/file_path.h"
 #include "base/i18n/time_formatting.h"
+#include "chrome/browser/apps/app_service/app_service_proxy.h"
+#include "chrome/browser/apps/app_service/app_service_proxy_factory.h"
+#include "chrome/browser/apps/app_service/launch_utils.h"
 #include "chrome/browser/chromeos/policy/dlp/dlp_content_manager.h"
+#include "chrome/browser/chromeos/web_applications/default_web_app_ids.h"
 #include "chrome/browser/download/download_prefs.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/profiles/profile_manager.h"
@@ -15,6 +19,7 @@
 #include "chromeos/login/login_state/login_state.h"
 #include "components/prefs/pref_service.h"
 #include "ui/aura/window.h"
+#include "ui/base/window_open_disposition.h"
 
 namespace {
 
@@ -44,6 +49,27 @@ void ChromeCaptureModeDelegate::ShowScreenCaptureItemInFolder(
                                   file_path);
 }
 
+void ChromeCaptureModeDelegate::OpenScreenshotInImageEditor(
+    const base::FilePath& file_path) {
+  Profile* profile = ProfileManager::GetActiveUserProfile();
+  if (!profile)
+    return;
+
+  apps::AppServiceProxy* proxy = apps::AppServiceProxyFactory::GetForProfile(
+      profile->GetOriginalProfile());
+  apps::mojom::FilePathsPtr file_paths_ptr =
+      apps::mojom::FilePaths::New(std::vector<base::FilePath>({file_path}));
+
+  // open the image with Essential App: Backlight.
+  proxy->LaunchAppWithFiles(
+      chromeos::default_web_apps::kMediaAppId,
+      apps::mojom::LaunchContainer::kLaunchContainerWindow,
+      apps::GetEventFlags(apps::mojom::LaunchContainer::kLaunchContainerWindow,
+                          WindowOpenDisposition::NEW_WINDOW,
+                          /*preferred_container=*/true),
+      apps::mojom::LaunchSource::kFromFileManager, std::move(file_paths_ptr));
+}
+
 bool ChromeCaptureModeDelegate::Uses24HourFormat() const {
   Profile* profile = ProfileManager::GetActiveUserProfile();
   // TODO(afakhry): Consider moving |prefs::kUse24HourClock| to ash/public so

chrome/browser/ui/ash/chrome_capture_mode_delegate.h

@@ -20,6 +20,7 @@ class ChromeCaptureModeDelegate : public ash::CaptureModeDelegate {
   // ash::CaptureModeDelegate:
   base::FilePath GetActiveUserDownloadsDir() const override;
   void ShowScreenCaptureItemInFolder(const base::FilePath& file_path) override;
+  void OpenScreenshotInImageEditor(const base::FilePath& file_path) override;
   bool Uses24HourFormat() const override;
   bool IsCaptureAllowed(const aura::Window* window,
                         const gfx::Rect& bounds,