Refactor the URLResponseInfo to use new design

ppapi/proxy/url_response_info_resource.h

+// 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.
+
+#ifndef PPAPI_PROXY_URL_RESPONSE_INFO_RESOURCE_H_
+#define PPAPI_PROXY_URL_RESPONSE_INFO_RESOURCE_H_
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "ppapi/proxy/plugin_resource.h"
+#include "ppapi/proxy/ppapi_proxy_export.h"
+#include "ppapi/shared_impl/scoped_pp_resource.h"
+#include "ppapi/shared_impl/url_response_info_data.h"
+#include "ppapi/thunk/ppb_url_response_info_api.h"
+
+namespace ppapi {
+namespace proxy {
+
+class PPAPI_PROXY_EXPORT URLResponseInfoResource
+    : public PluginResource,
+      public NON_EXPORTED_BASE(thunk::PPB_URLResponseInfo_API) {
+ public:
+  // Will take an additional reference to the file ref data if there is one.

[bbudge, 2012/10/12 01:27:06]

This is repeated below.

[brettw, 2012/10/12 19:34:53]

Actually not. It's completely contradicted below! :(

+  //
+  // The file_ref_resource should be the body_as_file_ref host resource in the
+  // |data| converted to a resource valid in the current process. A reference
+  // is passed from the caller and is taken over by this object.

[bbudge, 2012/10/12 01:27:06]

It might be helpful to state that file_ref_resource could be null, if the
request didn't have the stream-as-file property.

+  URLResponseInfoResource(Connection connection,
+                          PP_Instance instance,
+                          const URLResponseInfoData& data,
+                          PP_Resource file_ref_resource);
+  virtual ~URLResponseInfoResource();
+
+  // Resource override.
+  virtual PPB_URLResponseInfo_API* AsPPB_URLResponseInfo_API() OVERRIDE;
+
+  // PPB_URLResponseInfo_API implementation.
+  virtual PP_Var GetProperty(PP_URLResponseProperty property) OVERRIDE;
+  virtual PP_Resource GetBodyAsFileRef() OVERRIDE;
+  virtual URLResponseInfoData GetData() OVERRIDE;
+
+ private:
+  URLResponseInfoData data_;
+
+  // Non-zero when the load is being streamed to a file.
+  ScopedPPResource body_as_file_ref_;
+
+  DISALLOW_COPY_AND_ASSIGN(URLResponseInfoResource);
+};
+
+}  // namespace proxy
+}  // namespace ppapi
+
+#endif  // PPAPI_PROXY_URL_RESPONSE_INFO_RESOURCE_H_