Deliver attach intent to packaged apps

chrome/renderer/extensions/file_system_natives.cc

Index: chrome/renderer/extensions/file_system_natives.cc
diff --git a/chrome/renderer/extensions/file_system_natives.cc b/chrome/renderer/extensions/file_system_natives.cc
index 88faeba421084326a86f62e7a306678b0a0621a2..7e1f0dbe9b4bfafe3506b86d74cfb2ec9e902457 100644
--- a/chrome/renderer/extensions/file_system_natives.cc
+++ b/chrome/renderer/extensions/file_system_natives.cc
@@ -21,7 +21,7 @@ namespace {
 
 static v8::Handle<v8::Value> GetIsolatedFileSystem(
     const v8::Arguments& args) {
-  DCHECK(args.Length() == 1);
+  DCHECK(args.Length() == 1 || args.Length() == 2);
   DCHECK(args[0]->IsString());
   std::string file_system_id(*v8::String::Utf8Value(args[0]));
   WebKit::WebFrame* webframe = WebKit::WebFrame::frameForCurrentContext();
@@ -39,10 +39,18 @@ static v8::Handle<v8::Value> GetIsolatedFileSystem(
   root.append(file_system_id);
   root.append("/");
 
+  // Optionally create the DOMFileSystem rooted at a subdirectory.

[James Hawkins, 2012/08/29 22:46:51]

Optionally?

[thorogood, 2012/08/30 01:36:28]

I've expanded this comment slightly.

+  if (args.Length() == 2) {
+    DCHECK(args[1]->IsString());
+    name = *v8::String::Utf8Value(args[1]);
+    root.append(name);
+    root.append("/");
+  }
+
   return webframe->createFileSystem(
       WebKit::WebFileSystem::TypeIsolated,
-      WebKit::WebString::fromUTF8(name.c_str()),
-      WebKit::WebString::fromUTF8(root.c_str()));
+      WebKit::WebString::fromUTF8(name),
+      WebKit::WebString::fromUTF8(root));
 }
 
 }  // namespace