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

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

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

Powered by Google App Engine
This is Rietveld 408576698