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

Side by Side Diff: chrome/browser/chromeos/syslogs/syslogs_fetcher.cc

Issue 10827130: Refactoring the SysInfoProvider. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/chromeos/syslogs/syslogs_fetcher.h"
6
7 #include "base/bind.h"
8 #include "base/bind_helpers.h"
9 #include "content/public/browser/browser_thread.h"
10
11
12 using content::BrowserThread;
13
14 namespace chromeos {
15
16
17 SysLogsFetcher::SysLogsFetcher(SysLogsDataSources backends)
18 : data_sources_(backends), response_(new SysLogsResponse),
19 responses_(backends.size()),
20 ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {}
21
22 void SysLogsFetcher::Fetch(const SysLogsFetcherCallback& request) {
23 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
24 request_ = request;
25 SysLogsDataSources::iterator it;
26 responses_ = data_sources_.size();
27 for (it = data_sources_.begin(); it != data_sources_.end(); ++it) {
28 (*it)->Fetch(base::Bind(&SysLogsFetcher::AddData,
29 weak_ptr_factory_.GetWeakPtr()));
30 }
31 }
32
33 void SysLogsFetcher::AddData(SysLogsResponse* data) {
34 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
35 SysLogsResponse::iterator it;
36 for (it = data->begin(); it != data->end(); ++it){
37 (*response_)[it->first] = it->second;
38 }
39 delete data;
40
41 if (--responses_)
42 return;
43
44 SysLogsDataSources::iterator ds_it;
45 for (ds_it = data_sources_.begin(); ds_it != data_sources_.end(); ++ds_it) {
46 delete *ds_it;
47 }
48 request_.Run(response_);
49 BrowserThread::DeleteSoon(BrowserThread::UI, FROM_HERE, this);
50 }
51
52 } // namespace chromeos
53
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698