Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Side by Side Diff: chrome/browser/devtools/devtools_network_transaction.h

Issue 2298823002: Resetting the HttpRequestInfo pointers in HttpNetworkTransaction and streams (Closed)
Patch Set: Rebased, removed upload progress plumbing, feedback. (Rebased till refs/heads/master@{#417381}) Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_TRANSACTION_H_ 5 #ifndef CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_TRANSACTION_H_
6 #define CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_TRANSACTION_H_ 6 #define CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_TRANSACTION_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 14 matching lines...) Expand all
25 25
26 namespace net { 26 namespace net {
27 class AuthCredentials; 27 class AuthCredentials;
28 class BoundNetLog; 28 class BoundNetLog;
29 class HttpRequestHeaders; 29 class HttpRequestHeaders;
30 struct HttpRequestInfo; 30 struct HttpRequestInfo;
31 class HttpResponseInfo; 31 class HttpResponseInfo;
32 class HttpNetworkSession; 32 class HttpNetworkSession;
33 class IOBuffer; 33 class IOBuffer;
34 struct LoadTimingInfo; 34 struct LoadTimingInfo;
35 class UploadProgress;
36 class X509Certificate; 35 class X509Certificate;
37 } // namespace net 36 } // namespace net
38 37
39 namespace test { 38 namespace test {
40 class DevToolsNetworkControllerHelper; 39 class DevToolsNetworkControllerHelper;
41 } 40 }
42 41
43 // DevToolsNetworkTransaction is a wrapper for network transaction. All 42 // DevToolsNetworkTransaction is a wrapper for network transaction. All
44 // HttpTransaction methods are proxied to real transaction, but |callback| 43 // HttpTransaction methods are proxied to real transaction, but |callback|
45 // parameter is saved and replaced with proxy callback. Fail method should be 44 // parameter is saved and replaced with proxy callback. Fail method should be
(...skipping 26 matching lines...) Expand all
72 int Read(net::IOBuffer* buf, 71 int Read(net::IOBuffer* buf,
73 int buf_len, 72 int buf_len,
74 const net::CompletionCallback& callback) override; 73 const net::CompletionCallback& callback) override;
75 void StopCaching() override; 74 void StopCaching() override;
76 bool GetFullRequestHeaders(net::HttpRequestHeaders* headers) const override; 75 bool GetFullRequestHeaders(net::HttpRequestHeaders* headers) const override;
77 int64_t GetTotalReceivedBytes() const override; 76 int64_t GetTotalReceivedBytes() const override;
78 int64_t GetTotalSentBytes() const override; 77 int64_t GetTotalSentBytes() const override;
79 void DoneReading() override; 78 void DoneReading() override;
80 const net::HttpResponseInfo* GetResponseInfo() const override; 79 const net::HttpResponseInfo* GetResponseInfo() const override;
81 net::LoadState GetLoadState() const override; 80 net::LoadState GetLoadState() const override;
82 net::UploadProgress GetUploadProgress() const override;
83 void SetQuicServerInfo(net::QuicServerInfo* quic_server_info) override; 81 void SetQuicServerInfo(net::QuicServerInfo* quic_server_info) override;
84 bool GetLoadTimingInfo(net::LoadTimingInfo* load_timing_info) const override; 82 bool GetLoadTimingInfo(net::LoadTimingInfo* load_timing_info) const override;
85 bool GetRemoteEndpoint(net::IPEndPoint* endpoint) const override; 83 bool GetRemoteEndpoint(net::IPEndPoint* endpoint) const override;
86 void PopulateNetErrorDetails(net::NetErrorDetails* details) const override; 84 void PopulateNetErrorDetails(net::NetErrorDetails* details) const override;
87 void SetPriority(net::RequestPriority priority) override; 85 void SetPriority(net::RequestPriority priority) override;
88 void SetWebSocketHandshakeStreamCreateHelper( 86 void SetWebSocketHandshakeStreamCreateHelper(
89 net::WebSocketHandshakeStreamBase::CreateHelper* create_helper) override; 87 net::WebSocketHandshakeStreamBase::CreateHelper* create_helper) override;
90 void SetBeforeNetworkStartCallback( 88 void SetBeforeNetworkStartCallback(
91 const BeforeNetworkStartCallback& callback) override; 89 const BeforeNetworkStartCallback& callback) override;
92 void SetBeforeHeadersSentCallback( 90 void SetBeforeHeadersSentCallback(
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 126
129 const net::HttpRequestInfo* request_; 127 const net::HttpRequestInfo* request_;
130 128
131 // True if Fail was already invoked. 129 // True if Fail was already invoked.
132 bool failed_; 130 bool failed_;
133 131
134 DISALLOW_COPY_AND_ASSIGN(DevToolsNetworkTransaction); 132 DISALLOW_COPY_AND_ASSIGN(DevToolsNetworkTransaction);
135 }; 133 };
136 134
137 #endif // CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_TRANSACTION_H_ 135 #endif // CHROME_BROWSER_DEVTOOLS_DEVTOOLS_NETWORK_TRANSACTION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698