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

Side by Side Diff: third_party/WebKit/Source/platform/testing/weburl_loader_mock.cc

Issue 2835123005: Send the decoded size when response completed and stop summing in ResourceLoader::DidReceiveData() (Closed)
Patch Set: Created 3 years, 7 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #include "platform/testing/weburl_loader_mock.h" 5 #include "platform/testing/weburl_loader_mock.h"
6 6
7 #include "platform/testing/weburl_loader_mock_factory_impl.h" 7 #include "platform/testing/weburl_loader_mock_factory_impl.h"
8 #include "public/platform/URLConversion.h" 8 #include "public/platform/URLConversion.h"
9 #include "public/platform/WebData.h" 9 #include "public/platform/WebData.h"
10 #include "public/platform/WebURLError.h" 10 #include "public/platform/WebURLError.h"
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 53
54 // didReceiveResponse() and didReceiveData() might end up getting ::cancel() 54 // didReceiveResponse() and didReceiveData() might end up getting ::cancel()
55 // to be called which will make the ResourceLoader to delete |this|. 55 // to be called which will make the ResourceLoader to delete |this|.
56 WeakPtr<WebURLLoaderMock> self = weak_factory_.CreateWeakPtr(); 56 WeakPtr<WebURLLoaderMock> self = weak_factory_.CreateWeakPtr();
57 57
58 delegate->DidReceiveResponse(client_, response); 58 delegate->DidReceiveResponse(client_, response);
59 if (!self) 59 if (!self)
60 return; 60 return;
61 61
62 if (error.reason) { 62 if (error.reason) {
63 delegate->DidFail(client_, error, data.size(), 0); 63 delegate->DidFail(client_, error, data.size(), 0, 0);
64 return; 64 return;
65 } 65 }
66 delegate->DidReceiveData(client_, data.Data(), data.size()); 66 delegate->DidReceiveData(client_, data.Data(), data.size());
67 if (!self) 67 if (!self)
68 return; 68 return;
69 69
70 delegate->DidFinishLoading(client_, 0, data.size(), data.size()); 70 delegate->DidFinishLoading(client_, 0, data.size(), data.size(), data.size());
71 } 71 }
72 72
73 WebURLRequest WebURLLoaderMock::ServeRedirect( 73 WebURLRequest WebURLLoaderMock::ServeRedirect(
74 const WebURLRequest& request, 74 const WebURLRequest& request,
75 const WebURLResponse& redirect_response) { 75 const WebURLResponse& redirect_response) {
76 KURL redirect_url(kParsedURLString, 76 KURL redirect_url(kParsedURLString,
77 redirect_response.HttpHeaderField("Location")); 77 redirect_response.HttpHeaderField("Location"));
78 78
79 WebURLRequest new_request(redirect_url); 79 WebURLRequest new_request(redirect_url);
80 new_request.SetFirstPartyForCookies(redirect_url); 80 new_request.SetFirstPartyForCookies(redirect_url);
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 // In principle this is NOTIMPLEMENTED(), but if we put that here it floods 164 // In principle this is NOTIMPLEMENTED(), but if we put that here it floods
165 // the console during webkit unit tests, so we leave the function empty. 165 // the console during webkit unit tests, so we leave the function empty.
166 DCHECK(runner); 166 DCHECK(runner);
167 } 167 }
168 168
169 WeakPtr<WebURLLoaderMock> WebURLLoaderMock::GetWeakPtr() { 169 WeakPtr<WebURLLoaderMock> WebURLLoaderMock::GetWeakPtr() {
170 return weak_factory_.CreateWeakPtr(); 170 return weak_factory_.CreateWeakPtr();
171 } 171 }
172 172
173 } // namespace blink 173 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698