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

Side by Side Diff: content/browser/histogram_synchronizer.cc

Issue 1170623003: Revert "content: Remove use of MessageLoopProxy and deprecated MessageLoop APIs" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/histogram_synchronizer.h" 5 #include "content/browser/histogram_synchronizer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/lazy_instance.h" 8 #include "base/lazy_instance.h"
9 #include "base/location.h"
10 #include "base/logging.h" 9 #include "base/logging.h"
11 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
12 #include "base/metrics/histogram_delta_serialization.h" 11 #include "base/metrics/histogram_delta_serialization.h"
13 #include "base/pickle.h" 12 #include "base/pickle.h"
14 #include "base/single_thread_task_runner.h"
15 #include "base/threading/thread.h" 13 #include "base/threading/thread.h"
16 #include "base/threading/thread_restrictions.h" 14 #include "base/threading/thread_restrictions.h"
17 #include "content/browser/histogram_controller.h" 15 #include "content/browser/histogram_controller.h"
18 #include "content/public/browser/browser_thread.h" 16 #include "content/public/browser/browser_thread.h"
19 #include "content/public/browser/histogram_fetcher.h" 17 #include "content/public/browser/histogram_fetcher.h"
20 #include "content/public/common/content_constants.h" 18 #include "content/public/common/content_constants.h"
21 19
22 using base::Time; 20 using base::Time;
23 using base::TimeDelta; 21 using base::TimeDelta;
24 using base::TimeTicks; 22 using base::TimeTicks;
(...skipping 289 matching lines...) Expand 10 before | Expand all | Expand 10 after
314 callback_.Reset(); 312 callback_.Reset();
315 callback_thread_ = NULL; 313 callback_thread_ = NULL;
316 } 314 }
317 InternalPostTask(thread, callback); 315 InternalPostTask(thread, callback);
318 } 316 }
319 317
320 void HistogramSynchronizer::InternalPostTask(base::MessageLoop* thread, 318 void HistogramSynchronizer::InternalPostTask(base::MessageLoop* thread,
321 const base::Closure& callback) { 319 const base::Closure& callback) {
322 if (callback.is_null() || !thread) 320 if (callback.is_null() || !thread)
323 return; 321 return;
324 thread->task_runner()->PostTask(FROM_HERE, callback); 322 thread->PostTask(FROM_HERE, callback);
325 } 323 }
326 324
327 int HistogramSynchronizer::GetNextAvailableSequenceNumber( 325 int HistogramSynchronizer::GetNextAvailableSequenceNumber(
328 ProcessHistogramRequester requester) { 326 ProcessHistogramRequester requester) {
329 base::AutoLock auto_lock(lock_); 327 base::AutoLock auto_lock(lock_);
330 ++last_used_sequence_number_; 328 ++last_used_sequence_number_;
331 // Watch out for wrapping to a negative number. 329 // Watch out for wrapping to a negative number.
332 if (last_used_sequence_number_ < 0) { 330 if (last_used_sequence_number_ < 0) {
333 // Bypass the reserved number, which is used when a renderer spontaneously 331 // Bypass the reserved number, which is used when a renderer spontaneously
334 // decides to send some histogram data. 332 // decides to send some histogram data.
335 last_used_sequence_number_ = 333 last_used_sequence_number_ =
336 kHistogramSynchronizerReservedSequenceNumber + 1; 334 kHistogramSynchronizerReservedSequenceNumber + 1;
337 } 335 }
338 DCHECK_NE(last_used_sequence_number_, 336 DCHECK_NE(last_used_sequence_number_,
339 kHistogramSynchronizerReservedSequenceNumber); 337 kHistogramSynchronizerReservedSequenceNumber);
340 if (requester == ASYNC_HISTOGRAMS) 338 if (requester == ASYNC_HISTOGRAMS)
341 async_sequence_number_ = last_used_sequence_number_; 339 async_sequence_number_ = last_used_sequence_number_;
342 return last_used_sequence_number_; 340 return last_used_sequence_number_;
343 } 341 }
344 342
345 } // namespace content 343 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/gpu/browser_gpu_channel_host_factory.cc ('k') | content/browser/indexed_db/indexed_db_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698