Add API to construct new vector interchange MIME data type.

content/browser/intents/intent_injector.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 "content/browser/intents/intent_injector.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/logging.h"
@@ skipping 64 matching lines @@
   if (source_intent_.get() == NULL || !web_contents()->GetRenderViewHost())
     return;
 
   // Only deliver the intent to the renderer if it has the same origin
   // as the initial delivery target.
   if (initial_url_.GetOrigin() !=
       render_view_host->GetSiteInstance()->GetSiteURL().GetOrigin()) {
     return;
   }
 
-  if (source_intent_->data_type == webkit_glue::WebIntentData::BLOB) {
+  if (source_intent_->data_type == webkit_glue::WebIntentData::BLOB ||

[Steve McKay, 2012/10/15 19:31:10]

The checks are crystal clear, why you are doing them is not. Can you add a code
comment?

Also, is there a test that should be updated to verify the new rules?

[Greg Billock, 2012/10/15 21:01:43]

The new WebIntentsHost will grow to accomodate this on the encoding side. I plan
to make a content unit test for this file, as well.

+      (source_intent_->data_type == webkit_glue::WebIntentData::MIME_TYPE &&
+       !source_intent_->blob_file.empty())) {
     // Grant read permission on the blob file to the delivered context.
     const int child_id = render_view_host->GetProcess()->GetID();
     ChildProcessSecurityPolicy* policy =
          ChildProcessSecurityPolicy::GetInstance();
     if (!policy->CanReadFile(child_id, source_intent_->blob_file))
       policy->GrantReadFile(child_id, source_intent_->blob_file);
   } else if (source_intent_->data_type ==
              webkit_glue::WebIntentData::FILESYSTEM) {
     const int child_id = render_view_host->GetProcess()->GetID();
     FilePath path;
@@ skipping 24 matching lines @@
 void IntentInjector::OnReply(webkit_glue::WebIntentReplyType reply_type,
                              const string16& data) {
   if (!intents_dispatcher_)
     return;
 
   // Ensure SendReplyMessage is only called once.
   WebIntentsDispatcher* intents_dispatcher = intents_dispatcher_;
   intents_dispatcher_ = NULL;
   intents_dispatcher->SendReplyMessage(reply_type, data);
 }