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

Side by Side Diff: content/browser/fileapi/mock_url_request_delegate.cc

Issue 1062413002: Remove unnecessary instrumentation for several jank bugs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Compile fixes Created 5 years, 8 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 #include "content/browser/fileapi/mock_url_request_delegate.h" 5 #include "content/browser/fileapi/mock_url_request_delegate.h"
6 6
7 #include "base/profiler/scoped_tracker.h"
8 #include "base/run_loop.h" 7 #include "base/run_loop.h"
9 #include "net/base/io_buffer.h" 8 #include "net/base/io_buffer.h"
10 #include "testing/gtest/include/gtest/gtest.h" 9 #include "testing/gtest/include/gtest/gtest.h"
11 10
12 namespace { 11 namespace {
13 const int kBufferSize = 1024; 12 const int kBufferSize = 1024;
14 } 13 }
15 14
16 namespace content { 15 namespace content {
17 16
18 MockURLRequestDelegate::MockURLRequestDelegate() 17 MockURLRequestDelegate::MockURLRequestDelegate()
19 : io_buffer_(new net::IOBuffer(kBufferSize)) { 18 : io_buffer_(new net::IOBuffer(kBufferSize)) {
20 } 19 }
21 20
22 MockURLRequestDelegate::~MockURLRequestDelegate() { 21 MockURLRequestDelegate::~MockURLRequestDelegate() {
23 } 22 }
24 23
25 void MockURLRequestDelegate::OnResponseStarted(net::URLRequest* request) { 24 void MockURLRequestDelegate::OnResponseStarted(net::URLRequest* request) {
26 // TODO(vadimt): Remove ScopedTracker below once crbug.com/422516 is fixed.
27 tracked_objects::ScopedTracker tracking_profile(
28 FROM_HERE_WITH_EXPLICIT_FUNCTION(
29 "422516 MockURLRequestDelegate::OnResponseStarted"));
30
31 if (request->status().is_success()) { 25 if (request->status().is_success()) {
32 EXPECT_TRUE(request->response_headers()); 26 EXPECT_TRUE(request->response_headers());
33 ReadSome(request); 27 ReadSome(request);
34 } else { 28 } else {
35 RequestComplete(); 29 RequestComplete();
36 } 30 }
37 } 31 }
38 32
39 void MockURLRequestDelegate::OnReadCompleted(net::URLRequest* request, 33 void MockURLRequestDelegate::OnReadCompleted(net::URLRequest* request,
40 int bytes_read) { 34 int bytes_read) {
41 // TODO(vadimt): Remove ScopedTracker below once crbug.com/422516 is fixed.
42 tracked_objects::ScopedTracker tracking_profile(
43 FROM_HERE_WITH_EXPLICIT_FUNCTION(
44 "422516 MockURLRequestDelegate::OnReadCompleted"));
45
46 if (bytes_read > 0) 35 if (bytes_read > 0)
47 ReceiveData(request, bytes_read); 36 ReceiveData(request, bytes_read);
48 else 37 else
49 RequestComplete(); 38 RequestComplete();
50 } 39 }
51 40
52 void MockURLRequestDelegate::ReadSome(net::URLRequest* request) { 41 void MockURLRequestDelegate::ReadSome(net::URLRequest* request) {
53 if (!request->is_pending()) { 42 if (!request->is_pending()) {
54 RequestComplete(); 43 RequestComplete();
55 return; 44 return;
(...skipping 18 matching lines...) Expand all
74 } else { 63 } else {
75 RequestComplete(); 64 RequestComplete();
76 } 65 }
77 } 66 }
78 67
79 void MockURLRequestDelegate::RequestComplete() { 68 void MockURLRequestDelegate::RequestComplete() {
80 base::MessageLoop::current()->Quit(); 69 base::MessageLoop::current()->Quit();
81 } 70 }
82 71
83 } // namespace 72 } // namespace
OLDNEW
« no previous file with comments | « content/browser/cache_storage/cache_storage_cache.cc ('k') | content/browser/gpu/shader_disk_cache.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698