Remove SupportsUserData from ResourceRequestBodyImpl.

content/browser/loader/resource_request_info_impl.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 CONTENT_BROWSER_LOADER_RESOURCE_REQUEST_INFO_IMPL_H_
 #define CONTENT_BROWSER_LOADER_RESOURCE_REQUEST_INFO_IMPL_H_
 
 #include <memory>
 #include <string>
 #include <utility>
 
 #include "base/callback.h"
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/supports_user_data.h"
+#include "content/browser/blob_storage/chrome_blob_storage_context.h"
 #include "content/browser/loader/resource_requester_info.h"
 #include "content/common/resource_request_body_impl.h"
 #include "content/common/url_loader.mojom.h"
 #include "content/public/browser/navigation_ui_data.h"
 #include "content/public/browser/resource_request_info.h"
 #include "content/public/common/previews_state.h"
 #include "content/public/common/referrer.h"
 #include "content/public/common/resource_type.h"
 #include "net/base/load_states.h"
 
@@ skipping 166 matching lines @@
 
   void set_navigation_ui_data(
       std::unique_ptr<NavigationUIData> navigation_ui_data) {
     navigation_ui_data_ = std::move(navigation_ui_data);
   }
 
   void set_on_transfer(const TransferCallback& on_transfer) {
     on_transfer_ = on_transfer;
   }
 
+  void SetBlobHandles(std::unique_ptr<BlobHandles> blob_handles);
+
  private:
   FRIEND_TEST_ALL_PREFIXES(ResourceDispatcherHostTest,
                            DeletedFilterDetached);
   FRIEND_TEST_ALL_PREFIXES(ResourceDispatcherHostTest,
                            DeletedFilterDetachedRedirect);
   // Non-owning, may be NULL.
   DetachableResourceHandler* detachable_handler_;
 
   scoped_refptr<ResourceRequesterInfo> requester_info_;
   int route_id_;
@@ skipping 19 matching lines @@
   blink::WebReferrerPolicy referrer_policy_;
   blink::WebPageVisibilityState visibility_state_;
   ResourceContext* context_;
   bool report_raw_headers_;
   bool is_async_;
   PreviewsState previews_state_;
   scoped_refptr<ResourceRequestBodyImpl> body_;
   bool initiated_in_secure_context_;
   std::unique_ptr<NavigationUIData> navigation_ui_data_;
 
+  // Keeps blobs alive for the duration of the request.
+  std::unique_ptr<BlobHandles> blob_handles_;

[dmurph, 2017/06/27 21:29:07]

Can you make all of these just BlobHandles instead of unique_ptr? and then
std::move the vector?

+
   // This callback is set by MojoAsyncResourceHandler to update its mojo binding
   // and remote endpoint. This callback will be removed once PlzNavigate is
   // shipped.
   TransferCallback on_transfer_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourceRequestInfoImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_RESOURCE_REQUEST_INFO_IMPL_H_