Pepper: Move FileRef to the "new" resource proxy.

content/renderer/pepper/pepper_file_ref_renderer_host.cc

+// Copyright 2013 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/renderer/pepper/pepper_file_ref_renderer_host.h"
+
+#include <string>
+
+#include "base/files/file_path.h"
+#include "content/public/renderer/renderer_ppapi_host.h"
+#include "net/base/escape.h"
+#include "ppapi/c/pp_errors.h"
+#include "ppapi/c/pp_file_info.h"
+#include "ppapi/c/pp_instance.h"
+#include "ppapi/c/pp_resource.h"
+#include "ppapi/host/ppapi_host.h"
+#include "ppapi/host/resource_host.h"
+
+namespace content {
+
+PepperFileRefRendererHost::PepperFileRefRendererHost(
+    RendererPpapiHost* host,
+    PP_Instance instance,
+    PP_Resource resource,
+    PP_Resource file_system,
+    const std::string& internal_path)
+    : ResourceHost(host->GetPpapiHost(), instance, resource),
+      file_system_type_(PP_FILESYSTEMTYPE_INVALID),
+      internal_path_(internal_path) {
+  ResourceHost* fs_host = host->GetPpapiHost()->GetResourceHost(file_system);
+  if (fs_host && fs_host->IsFileSystemHost()) {
+    fs_host_ = base::AsWeakPtr(static_cast<PepperFileSystemHost*>(fs_host));
+    file_system_type_ = fs_host_->GetType();
+  }
+}
+
+
+PepperFileRefRendererHost::PepperFileRefRendererHost(
+    RendererPpapiHost* host,
+    PP_Instance instance,
+    PP_Resource resource,
+    const base::FilePath& external_path)
+    : ResourceHost(host->GetPpapiHost(), instance, resource),
+      file_system_type_(PP_FILESYSTEMTYPE_EXTERNAL),
+      external_path_(external_path) {
+}
+
+PepperFileRefRendererHost::~PepperFileRefRendererHost() {
+}
+
+PP_FileSystemType PepperFileRefRendererHost::GetFileSystemType() const {
+  return file_system_type_;
+}
+
+std::string PepperFileRefRendererHost::GetFileSystemURLSpec() const {
+  if (fs_host_.get() && fs_host_->IsOpened() &&
+      fs_host_->GetRootUrl().is_valid()) {
+    return fs_host_->GetRootUrl().Resolve(
+        net::EscapePath(internal_path_.substr(1))).spec();

[dmichael (off chromium), 2013/09/03 22:34:50]

the subtsr(1) looks slightly magical. I guess you're peeling off the leading
'/', and we've already made sure there is one, so it's safe to call substr(1)?
It may be good to have a CHECK and a comment.

[teravest, 2013/09/04 14:46:25]

Added a check and a comment. Also added checks for ppapi::IsValidInternalPath()
and ppapi::IsValidExternalPath() in the constructors.

+  }
+  return std::string();
+}
+
+base::FilePath PepperFileRefRendererHost::GetExternalFilePath() const {
+  if (file_system_type_ != PP_FILESYSTEMTYPE_EXTERNAL)
+    return base::FilePath();
+  return external_path_;
+}
+
+int32_t PepperFileRefRendererHost::OnResourceMessageReceived(
+      const IPC::Message& msg,
+      ppapi::host::HostMessageContext* context) {
+  // We don't handle any messages from the plugin in this host.
+  return PP_ERROR_FAILED;

[dmichael (off chromium), 2013/09/03 22:34:50]

NOTREACHED() also? To catch mistakes in debug builds?

[teravest, 2013/09/04 14:46:25]

Done.

+}
+
+bool PepperFileRefRendererHost::IsFileRefHost() {
+  return true;
+}
+
+}  // namespace content