Add verbs API to file handlers. Modify the Chrome OS UI so that it displayes the internationalized …

extensions/common/manifest_handlers/file_handler_info.cc

 // Copyright 2014 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 "extensions/common/manifest_handlers/file_handler_info.h"
 
 #include <stddef.h>
 
 #include <memory>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "extensions/common/error_utils.h"
 #include "extensions/common/manifest.h"
 #include "extensions/common/manifest_constants.h"
 
 namespace extensions {
 
 namespace keys = manifest_keys;
 namespace errors = manifest_errors;
 
+namespace file_handler_verbs {
+
+const char kOpenWith[] = "Open with";

[Devlin, 2016/05/16 16:17:32]

manifest constants are almost always all_lowercase_with_underscores.  Let's try
to be consistent (even though we're not always).

[cmihail, 2016/05/18 02:35:31]

Done.

+const char kAddTo[] = "Add to";
+const char kPackWith[] = "Pack with";
+
+}  // namespace file_handler_verbs
+
 namespace {
+
 const int kMaxTypeAndExtensionHandlers = 200;
 const char kNotRecognized[] = "'%s' is not a recognized file handler property.";
+
+bool IsSupportedVerb(const std::string& verb) {
+  std::set<std::string> supported_verbs;
+  supported_verbs.emplace(file_handler_verbs::kOpenWith);
+  supported_verbs.emplace(file_handler_verbs::kAddTo);
+  supported_verbs.emplace(file_handler_verbs::kPackWith);
+  return supported_verbs.find(verb) != supported_verbs.end();

[Devlin, 2016/05/16 16:17:32]

This results in the construction of a set and three extra strings, but doesn't
save anything over a simpler comparison:
return verb == OpenWith || verb == AddTo || verb == PackWith;

[cmihail, 2016/05/18 02:35:31]

Done.

 }
 
-FileHandlerInfo::FileHandlerInfo() : include_directories(false) {}
+}  // namespace
+
+FileHandlerInfo::FileHandlerInfo()
+    : include_directories(false), verb(file_handler_verbs::kOpenWith) {}
 FileHandlerInfo::FileHandlerInfo(const FileHandlerInfo& other) = default;
 FileHandlerInfo::~FileHandlerInfo() {}
 
 FileHandlers::FileHandlers() {}
 FileHandlers::~FileHandlers() {}
 
 // static
 const FileHandlersInfo* FileHandlers::GetFileHandlers(
     const Extension* extension) {
   FileHandlers* info = static_cast<FileHandlers*>(
@@ skipping 35 matching lines @@
 
   handler.include_directories = false;
   if (handler_info.HasKey("include_directories") &&
       !handler_info.GetBoolean("include_directories",
                                &handler.include_directories)) {
     *error = ErrorUtils::FormatErrorMessageUTF16(
         errors::kInvalidFileHandlerIncludeDirectories, handler_id);
     return false;
   }
 
+  handler.verb = file_handler_verbs::kOpenWith;
+  if (handler_info.HasKey(keys::kFileHandlerVerb) &&
+      !handler_info.GetString(keys::kFileHandlerVerb, &handler.verb) &&
+      !IsSupportedVerb(handler.verb)) {
+    *error = ErrorUtils::FormatErrorMessageUTF16(
+        errors::kInvalidFileHandlerVerb, handler_id);
+    return false;
+  }
+
   if ((!mime_types || mime_types->empty()) &&
       (!file_extensions || file_extensions->empty()) &&
       !handler.include_directories) {
     *error = ErrorUtils::FormatErrorMessageUTF16(
         errors::kInvalidFileHandlerNoTypeOrExtension,
         handler_id);
     return false;
   }
 
   if (mime_types) {
@@ skipping 83 matching lines @@
   extension->SetManifestData(keys::kFileHandlers, info.release());
   extension->AddInstallWarnings(install_warnings);
   return true;
 }
 
 const std::vector<std::string> FileHandlersParser::Keys() const {
   return SingleKey(keys::kFileHandlers);
 }
 
 }  // namespace extensions