chromeos: Add note-taking-app utility functions.

chrome/browser/chromeos/note_taking_app_utils.cc

Index: chrome/browser/chromeos/note_taking_app_utils.cc
diff --git a/chrome/browser/chromeos/note_taking_app_utils.cc b/chrome/browser/chromeos/note_taking_app_utils.cc
new file mode 100644
index 0000000000000000000000000000000000000000..ff06087d71cc1dc27faed4eb54a170020b0bc3c9
--- /dev/null
+++ b/chrome/browser/chromeos/note_taking_app_utils.cc
@@ -0,0 +1,83 @@
+// 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 "chrome/browser/chromeos/note_taking_app_utils.h"
+
+#include <string>
+#include <vector>
+
+#include "apps/launcher.h"
+#include "ash/common/system/chromeos/palette/palette_utils.h"
+#include "base/command_line.h"
+#include "base/files/file_path.h"
+#include "base/strings/string_split.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chromeos/chromeos_switches.h"
+#include "extensions/browser/extension_registry.h"
+#include "extensions/common/extension.h"
+#include "url/gurl.h"
+
+namespace chromeos {
+namespace {
+
+// TODO(derat): Add more IDs.
+const char* const kExtensionIds[] = {
+    "hmjkmjkepdijhoojdojkdfohbdgmmhki",  // Google Keep app (Web Store)
+};
+
+// Returns the first installed and enabled whitelisted note-taking app, or null
+// if none is installed.
+// TODO(derat): Instead of a using hardcoded list of IDs, use an app designated
+// by a pref.
+const extensions::Extension* GetApp(Profile* profile) {
+  std::vector<std::string> ids;
+
+  const std::string switch_value =
+      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+          switches::kNoteTakingAppIds);
+  if (!switch_value.empty()) {
+    ids = base::SplitString(switch_value, ",", base::TRIM_WHITESPACE,
+                            base::SPLIT_WANT_NONEMPTY);
+  } else {
+    ids.assign(kExtensionIds, kExtensionIds + arraysize(kExtensionIds));
+  }
+
+  const extensions::ExtensionRegistry* extension_registry =
+      extensions::ExtensionRegistry::Get(profile);
+  const extensions::ExtensionSet& enabled_extensions =
+      extension_registry->enabled_extensions();
+  for (const auto& id : ids) {
+    if (enabled_extensions.Contains(id)) {
+      return extension_registry->GetExtensionById(
+          id, extensions::ExtensionRegistry::ENABLED);
+    }
+  }
+  return nullptr;
+}
+
+}  // namespace
+
+bool IsNoteTakingAppAvailable(Profile* profile) {
+  DCHECK(profile);
+  return ash::IsPaletteEnabled() && GetApp(profile);
+}
+
+void LaunchNoteTakingAppForNewNote(Profile* profile,
+                                   const base::FilePath& path) {
+  DCHECK(profile);
+  const extensions::Extension* app = GetApp(profile);
+  if (!app) {
+    LOG(ERROR) << "Failed to find note-taking app";
+    return;
+  }
+
+  // TODO: Launch with an "action" launch source and the appropriate action type

[jdufault, 2016/08/05 22:59:38]

FYI, I have a CL that I'll upload shortly that does this.

[Daniel Erat, 2016/08/05 23:19:23]

i'd read the doc and figured you were were working on the api change (hence the
mention of the "action" launch source). :-)

do you mean you have a changelist that does the same thing as this one, too?

+  // to create a new note once that's been added to chrome.appRuntime.
+  if (path.empty())
+    apps::LaunchPlatformApp(profile, app, extensions::SOURCE_UNTRACKED);
+  else
+    apps::LaunchPlatformAppWithPath(profile, app, path);

[jdufault, 2016/08/05 22:59:38]

Did you have a chance to take a look at the API proposal (link below)?

I think we should add a new action type (annotate_screenshot) with a LaunchItem
or just a file path associated with it inside of the ActionData.

https://docs.google.com/document/u/1/d/1f3zpf8RMxavy3oTYHjiWKcU-Luo9EV25RyIpn...

[Daniel Erat, 2016/08/05 23:19:23]

i chatted with tom about this yesterday. i didn't see a compelling reason for an
"annotate_screenshot" action, as the behavior we want for v1 is actually "start
a new note with this image in it" rather than e.g. "open this image for
editing".

the app launcher code for passing file paths seems pretty complicated. i'd
suggest not reimplementing it and instead using the existing file handler code
unless there's a strong reason not to.

(happy to chat over vc or im if that'd help!)

+}
+
+}  // namespace chromeos