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

Side by Side Diff: chrome/browser/notifications/message_center_notification_manager.cc

Issue 1103713003: win: Move a few WeakPtrFactories to the end of their containing classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 "chrome/browser/notifications/message_center_notification_manager.h" 5 #include "chrome/browser/notifications/message_center_notification_manager.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/prefs/pref_registry_simple.h" 9 #include "base/prefs/pref_registry_simple.h"
10 #include "base/prefs/pref_service.h" 10 #include "base/prefs/pref_service.h"
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 #endif 52 #endif
53 53
54 MessageCenterNotificationManager::MessageCenterNotificationManager( 54 MessageCenterNotificationManager::MessageCenterNotificationManager(
55 message_center::MessageCenter* message_center, 55 message_center::MessageCenter* message_center,
56 PrefService* local_state, 56 PrefService* local_state,
57 scoped_ptr<message_center::NotifierSettingsProvider> settings_provider) 57 scoped_ptr<message_center::NotifierSettingsProvider> settings_provider)
58 : message_center_(message_center), 58 : message_center_(message_center),
59 #if defined(OS_WIN) 59 #if defined(OS_WIN)
60 first_run_idle_timeout_( 60 first_run_idle_timeout_(
61 base::TimeDelta::FromSeconds(kFirstRunIdleDelaySeconds)), 61 base::TimeDelta::FromSeconds(kFirstRunIdleDelaySeconds)),
62 weak_factory_(this),
63 #endif 62 #endif
64 settings_provider_(settings_provider.Pass()), 63 settings_provider_(settings_provider.Pass()),
65 system_observer_(this), 64 system_observer_(this),
66 stats_collector_(message_center), 65 stats_collector_(message_center),
67 google_now_stats_collector_(message_center) { 66 google_now_stats_collector_(message_center)
67 #if defined(OS_WIN)
68 ,
69 weak_factory_(this)
70 #endif
71 {
68 #if defined(OS_WIN) 72 #if defined(OS_WIN)
69 first_run_pref_.Init(prefs::kMessageCenterShowedFirstRunBalloon, local_state); 73 first_run_pref_.Init(prefs::kMessageCenterShowedFirstRunBalloon, local_state);
70 #endif 74 #endif
71 75
72 message_center_->AddObserver(this); 76 message_center_->AddObserver(this);
73 message_center_->SetNotifierSettingsProvider(settings_provider_.get()); 77 message_center_->SetNotifierSettingsProvider(settings_provider_.get());
74 78
75 #if defined(OS_CHROMEOS) 79 #if defined(OS_CHROMEOS)
76 blockers_.push_back( 80 blockers_.push_back(
77 new LoginStateNotificationBlockerChromeOS(message_center)); 81 new LoginStateNotificationBlockerChromeOS(message_center));
(...skipping 324 matching lines...) Expand 10 before | Expand all | Expand 10 after
402 it != registry->enabled_extensions().end(); 406 it != registry->enabled_extensions().end();
403 ++it) { 407 ++it) {
404 if ((*it->get()).permissions_data()->HasAPIPermission( 408 if ((*it->get()).permissions_data()->HasAPIPermission(
405 extensions::APIPermission::ID::kNotificationProvider)) { 409 extensions::APIPermission::ID::kNotificationProvider)) {
406 extension_id = (*it->get()).id(); 410 extension_id = (*it->get()).id();
407 return extension_id; 411 return extension_id;
408 } 412 }
409 } 413 }
410 return extension_id; 414 return extension_id;
411 } 415 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698