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

Side by Side Diff: base/trace_event/trace_log.cc

Issue 1942053002: Deletes base::MessageLoop::set_thread_name(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: uses PlatformThread::GetName Created 4 years, 6 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/threading/thread.cc ('k') | chrome/app_shim/chrome_main_app_mode_mac.mm » ('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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/trace_event/trace_log.h" 5 #include "base/trace_event/trace_log.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 1026 matching lines...) Expand 10 before | Expand all | Expand 10 after
1037 } 1037 }
1038 1038
1039 LOG(WARNING) 1039 LOG(WARNING)
1040 << "The following threads haven't finished flush in time. " 1040 << "The following threads haven't finished flush in time. "
1041 "If this happens stably for some thread, please call " 1041 "If this happens stably for some thread, please call "
1042 "TraceLog::GetInstance()->SetCurrentThreadBlocksMessageLoop() from " 1042 "TraceLog::GetInstance()->SetCurrentThreadBlocksMessageLoop() from "
1043 "the thread to avoid its trace events from being lost."; 1043 "the thread to avoid its trace events from being lost.";
1044 for (hash_set<MessageLoop*>::const_iterator it = 1044 for (hash_set<MessageLoop*>::const_iterator it =
1045 thread_message_loops_.begin(); 1045 thread_message_loops_.begin();
1046 it != thread_message_loops_.end(); ++it) { 1046 it != thread_message_loops_.end(); ++it) {
1047 LOG(WARNING) << "Thread: " << (*it)->thread_name(); 1047 LOG(WARNING) << "Thread: " << (*it)->GetThreadName();
1048 } 1048 }
1049 } 1049 }
1050 FinishFlush(generation, discard_events); 1050 FinishFlush(generation, discard_events);
1051 } 1051 }
1052 1052
1053 void TraceLog::UseNextTraceBuffer() { 1053 void TraceLog::UseNextTraceBuffer() {
1054 logged_events_.reset(CreateTraceBuffer()); 1054 logged_events_.reset(CreateTraceBuffer());
1055 subtle::NoBarrier_AtomicIncrement(&generation_, 1); 1055 subtle::NoBarrier_AtomicIncrement(&generation_, 1);
1056 thread_shared_chunk_.reset(); 1056 thread_shared_chunk_.reset();
1057 thread_shared_chunk_index_ = 0; 1057 thread_shared_chunk_index_ = 0;
(...skipping 734 matching lines...) Expand 10 before | Expand all | Expand 10 after
1792 } 1792 }
1793 1793
1794 ScopedTraceBinaryEfficient::~ScopedTraceBinaryEfficient() { 1794 ScopedTraceBinaryEfficient::~ScopedTraceBinaryEfficient() {
1795 if (*category_group_enabled_) { 1795 if (*category_group_enabled_) {
1796 TRACE_EVENT_API_UPDATE_TRACE_EVENT_DURATION(category_group_enabled_, name_, 1796 TRACE_EVENT_API_UPDATE_TRACE_EVENT_DURATION(category_group_enabled_, name_,
1797 event_handle_); 1797 event_handle_);
1798 } 1798 }
1799 } 1799 }
1800 1800
1801 } // namespace trace_event_internal 1801 } // namespace trace_event_internal
OLDNEW
« no previous file with comments | « base/threading/thread.cc ('k') | chrome/app_shim/chrome_main_app_mode_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698