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

Side by Side Diff: chrome/browser/web_resource/notification_promo_mobile_ntp.cc

Issue 11689004: Move PromoResourceService from Profile to BrowserProcessImpl/local_state(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: android x 4 Created 7 years, 11 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 | Annotate | Revision Log
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/web_resource/notification_promo_mobile_ntp.h" 5 #include "chrome/browser/web_resource/notification_promo_mobile_ntp.h"
6 6
7 #include "base/values.h" 7 #include "base/values.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/prefs/pref_service_simple.h"
9 #include "chrome/browser/web_resource/notification_promo.h" 9 #include "chrome/browser/web_resource/notification_promo.h"
10 10
11 11 NotificationPromoMobileNtp::NotificationPromoMobileNtp(PrefServiceSimple* prefs)
12 NotificationPromoMobileNtp::NotificationPromoMobileNtp(Profile* profile)
13 : valid_(false), 12 : valid_(false),
14 action_args_(NULL), 13 action_args_(NULL),
15 payload_(NULL), 14 payload_(NULL),
16 notification_promo_(profile) { 15 notification_promo_(prefs) {
17 } 16 }
18 17
19 NotificationPromoMobileNtp::~NotificationPromoMobileNtp() { 18 NotificationPromoMobileNtp::~NotificationPromoMobileNtp() {
20 } 19 }
21 20
22 bool NotificationPromoMobileNtp::InitFromPrefs() { 21 bool NotificationPromoMobileNtp::InitFromPrefs() {
23 notification_promo_.InitFromPrefs(NotificationPromo::MOBILE_NTP_SYNC_PROMO); 22 notification_promo_.InitFromPrefs(NotificationPromo::MOBILE_NTP_SYNC_PROMO);
24 return InitFromNotificationPromo(); 23 return InitFromNotificationPromo();
25 } 24 }
26 25
(...skipping 21 matching lines...) Expand all
48 payload_ = notification_promo_.promo_payload(); 47 payload_ = notification_promo_.promo_payload();
49 if (!payload_ || 48 if (!payload_ ||
50 !payload_->GetString("promo_message_short", &text_) || 49 !payload_->GetString("promo_message_short", &text_) ||
51 !payload_->GetString("promo_message_long", &text_long_) || 50 !payload_->GetString("promo_message_long", &text_long_) ||
52 !payload_->GetString("promo_action_type", &action_type_) || 51 !payload_->GetString("promo_action_type", &action_type_) ||
53 !payload_->GetList("promo_action_args", &action_args_) || 52 !payload_->GetList("promo_action_args", &action_args_) ||
54 !action_args_) { 53 !action_args_) {
55 return false; 54 return false;
56 } 55 }
57 56
58 // The rest of the fields is optional. 57 // The rest of the fields are optional.
59 valid_ = true; 58 valid_ = true;
60 payload_->GetBoolean("promo_requires_mobile_only_sync", 59 payload_->GetBoolean("promo_requires_mobile_only_sync",
61 &requires_mobile_only_sync_); 60 &requires_mobile_only_sync_);
62 payload_->GetBoolean("promo_requires_sync", &requires_sync_); 61 payload_->GetBoolean("promo_requires_sync", &requires_sync_);
63 payload_->GetBoolean("promo_show_on_most_visited", &show_on_most_visited_); 62 payload_->GetBoolean("promo_show_on_most_visited", &show_on_most_visited_);
64 payload_->GetBoolean("promo_show_on_open_tabs", &show_on_open_tabs_); 63 payload_->GetBoolean("promo_show_on_open_tabs", &show_on_open_tabs_);
65 payload_->GetBoolean("promo_show_as_virtual_computer", 64 payload_->GetBoolean("promo_show_as_virtual_computer",
66 &show_as_virtual_computer_); 65 &show_as_virtual_computer_);
67 payload_->GetString("promo_virtual_computer_title", &virtual_computer_title_); 66 payload_->GetString("promo_virtual_computer_title", &virtual_computer_title_);
68 payload_->GetString("promo_virtual_computer_lastsync", 67 payload_->GetString("promo_virtual_computer_lastsync",
69 &virtual_computer_lastsync_); 68 &virtual_computer_lastsync_);
70 69
71 return valid_; 70 return valid_;
72 } 71 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698