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

Side by Side Diff: base/debug/task_annotator.cc

Issue 1641513004: Update //base to chromium 9659b08ea5a34f889dc4166217f438095ddc10d2 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 4 years, 10 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
« no previous file with comments | « base/debug/task_annotator.h ('k') | base/debug/task_annotator_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "base/debug/task_annotator.h" 5 #include "base/debug/task_annotator.h"
6 6
7 #include "base/debug/alias.h" 7 #include "base/debug/alias.h"
8 #include "base/pending_task.h" 8 #include "base/pending_task.h"
9 #include "base/trace_event/trace_event.h" 9 #include "base/trace_event/trace_event.h"
10 #include "base/tracked_objects.h" 10 #include "base/tracked_objects.h"
11 11
12 namespace base { 12 namespace base {
13 namespace debug { 13 namespace debug {
14 14
15 TaskAnnotator::TaskAnnotator() { 15 TaskAnnotator::TaskAnnotator() {
16 } 16 }
17 17
18 TaskAnnotator::~TaskAnnotator() { 18 TaskAnnotator::~TaskAnnotator() {
19 } 19 }
20 20
21 void TaskAnnotator::DidQueueTask(const char* queue_function, 21 void TaskAnnotator::DidQueueTask(const char* queue_function,
22 const PendingTask& pending_task) { 22 const PendingTask& pending_task) {
23 TRACE_EVENT_FLOW_BEGIN0(TRACE_DISABLED_BY_DEFAULT("toplevel.flow"), 23 TRACE_EVENT_FLOW_BEGIN0(TRACE_DISABLED_BY_DEFAULT("toplevel.flow"),
24 queue_function, 24 queue_function,
25 TRACE_ID_MANGLE(GetTaskTraceID(pending_task))); 25 TRACE_ID_MANGLE(GetTaskTraceID(pending_task)));
26 } 26 }
27 27
28 void TaskAnnotator::RunTask(const char* queue_function, 28 void TaskAnnotator::RunTask(const char* queue_function,
29 const char* run_function,
30 const PendingTask& pending_task) { 29 const PendingTask& pending_task) {
31 tracked_objects::TaskStopwatch stopwatch; 30 tracked_objects::TaskStopwatch stopwatch;
32 stopwatch.Start(); 31 stopwatch.Start();
33 tracked_objects::Duration queue_duration = 32 tracked_objects::Duration queue_duration =
34 stopwatch.StartTime() - pending_task.EffectiveTimePosted(); 33 stopwatch.StartTime() - pending_task.EffectiveTimePosted();
35 34
36 TRACE_EVENT_FLOW_END1(TRACE_DISABLED_BY_DEFAULT("toplevel.flow"), 35 TRACE_EVENT_FLOW_END1(TRACE_DISABLED_BY_DEFAULT("toplevel.flow"),
37 queue_function, 36 queue_function,
38 TRACE_ID_MANGLE(GetTaskTraceID(pending_task)), 37 TRACE_ID_MANGLE(GetTaskTraceID(pending_task)),
39 "queue_duration", 38 "queue_duration",
40 queue_duration.InMilliseconds()); 39 queue_duration.InMilliseconds());
41 40
42 // When tracing memory for posted tasks it's more valuable to attribute the
43 // memory allocations to the source function than generically to the task
44 // runner.
45 TRACE_EVENT_WITH_MEMORY_TAG2(
46 "toplevel",
47 run_function,
48 pending_task.posted_from.function_name(), // Name for memory tracking.
49 "src_file",
50 pending_task.posted_from.file_name(),
51 "src_func",
52 pending_task.posted_from.function_name());
53
54 // Before running the task, store the program counter where it was posted 41 // Before running the task, store the program counter where it was posted
55 // and deliberately alias it to ensure it is on the stack if the task 42 // and deliberately alias it to ensure it is on the stack if the task
56 // crashes. Be careful not to assume that the variable itself will have the 43 // crashes. Be careful not to assume that the variable itself will have the
57 // expected value when displayed by the optimizer in an optimized build. 44 // expected value when displayed by the optimizer in an optimized build.
58 // Look at a memory dump of the stack. 45 // Look at a memory dump of the stack.
59 const void* program_counter = pending_task.posted_from.program_counter(); 46 const void* program_counter = pending_task.posted_from.program_counter();
60 debug::Alias(&program_counter); 47 debug::Alias(&program_counter);
61 48
62 pending_task.task.Run(); 49 pending_task.task.Run();
63 50
64 stopwatch.Stop(); 51 stopwatch.Stop();
65 tracked_objects::ThreadData::TallyRunOnNamedThreadIfTracking( 52 tracked_objects::ThreadData::TallyRunOnNamedThreadIfTracking(
66 pending_task, stopwatch); 53 pending_task, stopwatch);
67 } 54 }
68 55
69 uint64 TaskAnnotator::GetTaskTraceID(const PendingTask& task) const { 56 uint64 TaskAnnotator::GetTaskTraceID(const PendingTask& task) const {
70 return (static_cast<uint64>(task.sequence_num) << 32) | 57 return (static_cast<uint64>(task.sequence_num) << 32) |
71 ((static_cast<uint64>(reinterpret_cast<intptr_t>(this)) << 32) >> 32); 58 ((static_cast<uint64>(reinterpret_cast<intptr_t>(this)) << 32) >> 32);
72 } 59 }
73 60
74 } // namespace debug 61 } // namespace debug
75 } // namespace base 62 } // namespace base
OLDNEW
« no previous file with comments | « base/debug/task_annotator.h ('k') | base/debug/task_annotator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698