Fix chrome upload with content uri

content/browser/renderer_host/render_view_host_impl.cc

Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index 8018d57b197574486dc6e2f9e61efd156257650c..fbb8334ecf408ef89b06909518cadb0803d98501 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -1095,8 +1095,18 @@ void RenderViewHostImpl::FilesSelectedInChooser(
       ChildProcessSecurityPolicyImpl::GetInstance()->GrantCreateReadWriteFile(
           GetProcess()->GetID(), file.local_path);
     } else {
+#if defined(OS_ANDROID)
+      GURL url(file.local_path.value());
+      if (url.SchemeIsContent()) {
+        ChildProcessSecurityPolicyImpl::GetInstance()->GrantReadContentUrl(
+            GetProcess()->GetID(), url);

[kinuko, 2013/10/29 02:52:31]

We receive this as file path and then at some point convert it to GURL if the
path can be interpret as content: URL.

What resources do content: URL refer? Can we always assume it points to a local
file resource? It looks if we delay this conversion point later we can remove
more OS_ANDROID code (e.g. we can change some of local file streams to
understand content: URL if it makes sense). Should we do so to limit the os
specific code or not?

[qinmin, 2013/10/29 19:14:21]

For upload case, the content url should point to a local file.
Yes, I think we can delay some of the conversion until we reach the
reading/getFileInfo stage. Also, some of the FilePath member functions are not
applicable to content urls, so I need to bypass those function.

I will do that in the next PS.

On 2013/10/29 02:52:31, kinuko wrote:

+      } else {
+#endif
       ChildProcessSecurityPolicyImpl::GetInstance()->GrantReadFile(
           GetProcess()->GetID(), file.local_path);
+#if defined(OS_ANDROID)
+      }
+#endif
     }
   }
   Send(new ViewMsg_RunFileChooserResponse(GetRoutingID(), files));