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

Side by Side Diff: chrome/renderer/renderer_histogram_snapshots.cc

Issue 3032027: Header cleanup in chrome/renderer/. (Closed) Base URL: http://src.chromium.org/git/chromium.git
Patch Set: change away from lazy initialization Created 10 years, 4 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) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 "chrome/renderer/renderer_histogram_snapshots.h" 5 #include "chrome/renderer/renderer_histogram_snapshots.h"
6 6
7 #include <ctype.h> 7 #include <ctype.h>
8 8
9 #include "base/histogram.h" 9 #include "base/histogram.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
11 #include "base/message_loop.h" 11 #include "base/message_loop.h"
12 #include "base/pickle.h" 12 #include "base/pickle.h"
13 #include "chrome/common/render_messages.h" 13 #include "chrome/common/render_messages.h"
14 #include "chrome/renderer/render_process.h" 14 #include "chrome/renderer/render_process.h"
15 #include "chrome/renderer/render_thread.h" 15 #include "chrome/renderer/render_thread.h"
16 16
17 // TODO(raman): Before renderer shuts down send final snapshot lists. 17 // TODO(raman): Before renderer shuts down send final snapshot lists.
18 18
19 RendererHistogramSnapshots::RendererHistogramSnapshots() 19 RendererHistogramSnapshots::RendererHistogramSnapshots()
20 : ALLOW_THIS_IN_INITIALIZER_LIST( 20 : ALLOW_THIS_IN_INITIALIZER_LIST(
21 renderer_histogram_snapshots_factory_(this)) { 21 renderer_histogram_snapshots_factory_(this)) {
22 } 22 }
23 23
24 RendererHistogramSnapshots::~RendererHistogramSnapshots() {
25 }
26
24 // Send data quickly! 27 // Send data quickly!
25 void RendererHistogramSnapshots::SendHistograms(int sequence_number) { 28 void RendererHistogramSnapshots::SendHistograms(int sequence_number) {
26 RenderThread::current()->message_loop()->PostTask(FROM_HERE, 29 RenderThread::current()->message_loop()->PostTask(FROM_HERE,
27 renderer_histogram_snapshots_factory_.NewRunnableMethod( 30 renderer_histogram_snapshots_factory_.NewRunnableMethod(
28 &RendererHistogramSnapshots::UploadAllHistrograms, sequence_number)); 31 &RendererHistogramSnapshots::UploadAllHistrograms, sequence_number));
29 } 32 }
30 33
31 void RendererHistogramSnapshots::UploadAllHistrograms(int sequence_number) { 34 void RendererHistogramSnapshots::UploadAllHistrograms(int sequence_number) {
32 StatisticsRecorder::Histograms histograms; 35 StatisticsRecorder::Histograms histograms;
33 StatisticsRecorder::GetHistograms(&histograms); 36 StatisticsRecorder::GetHistograms(&histograms);
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 const Histogram::SampleSet& snapshot, 88 const Histogram::SampleSet& snapshot,
86 HistogramPickledList* pickled_histograms) { 89 HistogramPickledList* pickled_histograms) {
87 90
88 DCHECK(0 != snapshot.TotalCount()); 91 DCHECK(0 != snapshot.TotalCount());
89 snapshot.CheckSize(histogram); 92 snapshot.CheckSize(histogram);
90 93
91 std::string histogram_info = 94 std::string histogram_info =
92 Histogram::SerializeHistogramInfo(histogram, snapshot); 95 Histogram::SerializeHistogramInfo(histogram, snapshot);
93 pickled_histograms->push_back(histogram_info); 96 pickled_histograms->push_back(histogram_info);
94 } 97 }
OLDNEW
« no previous file with comments | « chrome/renderer/renderer_histogram_snapshots.h ('k') | chrome/renderer/renderer_webkitclient_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698