Pass command line arguments onto platform apps which provide the right intent.

chrome/browser/extensions/platform_app_intent_dispatcher.cc

+// Copyright (c) 2012 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/extensions/platform_app_intent_dispatcher.h"
+
+#include "base/bind.h"
+#include "base/command_line.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/string16.h"
+#include "base/threading/thread_restrictions.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/browser/extensions/extension_host.h"
+#include "chrome/common/extensions/extension_messages.h"
+#include "chrome/common/extensions/url_pattern.h"
+#include "chrome/common/extensions/url_pattern_set.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/child_process_security_policy.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/browser/web_intents_dispatcher.h"
+#include "content/public/common/url_constants.h"
+#include "googleurl/src/gurl.h"
+#include "net/base/mime_util.h"
+#include "net/base/net_util.h"
+#include "webkit/glue/web_intent_data.h"
+#include "webkit/glue/web_intent_service_data.h"
+
+namespace extensions {
+
+void DispatchPlatformAppIntentData(ExtensionHost* host) {
+  // See if there is a command line argument.
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  CommandLine::StringVector args = command_line->GetArgs();
+  if (!args.size())
+    return;
+
+  FilePath file_path(args[0]);
+  std::string mime_type;
+  {
+    // TODO(benwells): Remove this ScopedAllowIO. This should be done by
+    // delaying the creation of the render view until after the MIME type has
+    // been retrieved on the FILE thread. This should be done once the platform
+    // app startup and windowing model has settled down.
+    base::ThreadRestrictions::ScopedAllowIO allow;
+
+    // Get the MIME type of the file.
+    if (!net::GetMimeTypeFromFile(file_path, &mime_type))
+      return;
+  }
+
+  // Find the intent service from the platform app for the file being opened.
+  const string16 kOpenAction = ASCIIToUTF16("http://webintents.org/view");

[James Hawkins, 2012/03/28 00:59:52]

Can we find a better variable name than kOpenAction?  I forgot this string ==
'view' later on in the code.

kViewAction may be better.

[benwells, 2012/03/28 04:50:27]

Done.

[Greg Billock, 2012/03/28 14:21:02]

Can you put this constant in webkit/glue/web_intent_data? There'll be at least
two uses of it already shortly.

On 2012/03/28 04:50:27, benwells wrote:

+  webkit_glue::WebIntentServiceData service;
+  bool found_service = false;
+
+  std::vector<webkit_glue::WebIntentServiceData> services =
+      host->extension()->intents_services();
+  for (size_t i = 0; i < services.size(); i++) {
+    std::string service_type_ascii = UTF16ToASCII(services[i].type);
+    if (services[i].action == kOpenAction &&
+        net::MatchesMimeType(service_type_ascii, mime_type)) {
+      service = services[i];
+      found_service = true;
+    }
+  }
+  if (!found_service)
+    return;
+
+  // Give the app permission to use the file:// scheme.

[James Hawkins, 2012/03/28 00:59:52]

Just to make sure I'm reading this correctly, the app only has access to the
particular file:// URL passed in |data|, right?

[benwells, 2012/03/28 04:50:27]

Yes that's right. I've tried to make it more obvious from the comments, let me
know if its still unclear.

[Greg Billock, 2012/03/28 14:21:02]

Don't you just want GrantReadFile?

On 2012/03/28 04:50:27, benwells wrote:

[benwells, 2012/03/29 03:29:18]

GrantReadFile doesn't do what I want, as WebKit (or wherever the cross origin
policy stuff lives) will filter out any XHR reads before it gets to the browser
process.

It might be correct to also use GrantReadFile, however I found it not necessary
for XHR requests (but back when I was playing with Blob stuff I think I remember
it was needed).

+  using content::ChildProcessSecurityPolicy;
+  ChildProcessSecurityPolicy* policy =
+      ChildProcessSecurityPolicy::GetInstance();
+  DCHECK(policy);
+  int child_id = host->render_process_host()->GetID();
+  policy->GrantScheme(child_id, chrome::kFileScheme);
+
+  // Give the renderer permission to the given URL.
+  ExtensionAPIPermissionSet apis;
+  URLPatternSet script_hosts;
+  GURL file_url(net::FilePathToFileURL(file_path));
+  URLPattern file_url_pattern(URLPattern::SCHEME_FILE, file_url.spec());
+  URLPatternSet explicit_hosts;
+  explicit_hosts.AddPattern(file_url_pattern);
+  host->render_process_host()->Send(new ExtensionMsg_UpdatePermissions(
+      static_cast<int>(UpdatedExtensionPermissionsInfo::ADDED),
+      host->extension()->id(), apis, explicit_hosts, script_hosts));
+
+  // Dispatch the intent data to the renderer.
+  string16 mime_type_utf16 = ASCIIToUTF16(mime_type);
+  webkit_glue::WebIntentData intent_data(kOpenAction, mime_type_utf16,
+      ASCIIToUTF16(file_url.spec()));
+  DirectDispatchIntent(intent_data, host->render_view_host());
+}
+
+}  // namespace extensions