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

Side by Side Diff: trunk/src/chrome/browser/notifications/sync_notifier/synced_notification_app_info.h

Issue 269893004: Revert 268203 "There are many ways to get it wrong." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 7 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 // This class represents the metadata for a service sending synced 5 // This class represents the metadata for a service sending synced
6 // notifications. 6 // notifications.
7 7
8 #ifndef CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_APP_INFO_ H_ 8 #ifndef CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_APP_INFO_ H_
9 #define CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_APP_INFO_ H_ 9 #define CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_APP_INFO_ H_
10 10
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 // If an app info is updated, keep track of the newly added app ids so we can 65 // If an app info is updated, keep track of the newly added app ids so we can
66 // later inform the chrome_notifier_service to show any newly enabled 66 // later inform the chrome_notifier_service to show any newly enabled
67 // notifications. 67 // notifications.
68 void set_added_app_ids(std::vector<std::string> added_app_ids) { 68 void set_added_app_ids(std::vector<std::string> added_app_ids) {
69 added_app_ids_ = added_app_ids; 69 added_app_ids_ = added_app_ids;
70 } 70 }
71 71
72 std::vector<std::string> added_app_ids() { return added_app_ids_; } 72 std::vector<std::string> added_app_ids() { return added_app_ids_; }
73 73
74 // If an app info is updated removing app ids, keep track of the removed app 74 // If an app info is updated removing app ids, keep track of the removed app
75 // ids so we can later remove any affected notifications. 75 // ids so we can later remove any affected notfications.
76 void set_removed_app_ids(std::vector<std::string> removed_app_ids) { 76 void set_removed_app_ids(std::vector<std::string> removed_app_ids) {
77 removed_app_ids_ = removed_app_ids; 77 removed_app_ids_ = removed_app_ids;
78 } 78 }
79 79
80 std::vector<std::string> removed_app_ids() { return removed_app_ids_; } 80 std::vector<std::string> removed_app_ids() { return removed_app_ids_; }
81 81
82 // TODO(petewil): Check resolution of system and return the right icon. 82 // TODO(petewil): Check resolution of system and return the right icon.
83 gfx::Image icon(); 83 gfx::Image icon();
84 84
85 // Build a vector of app_ids that this app_info contains. 85 // Build a vector of app_ids that this app_info contains.
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 QueueBitmapFetchJobsTest); 131 QueueBitmapFetchJobsTest);
132 FRIEND_TEST_ALL_PREFIXES(SyncedNotificationAppInfoTest, 132 FRIEND_TEST_ALL_PREFIXES(SyncedNotificationAppInfoTest,
133 AreAllBitmapsFetchedTest); 133 AreAllBitmapsFetchedTest);
134 134
135 DISALLOW_COPY_AND_ASSIGN(SyncedNotificationAppInfo); 135 DISALLOW_COPY_AND_ASSIGN(SyncedNotificationAppInfo);
136 }; 136 };
137 137
138 } // namespace notifier 138 } // namespace notifier
139 139
140 #endif // CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_APP_IN FO_H_ 140 #endif // CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_APP_IN FO_H_
OLDNEW
« no previous file with comments | « trunk/src/base/observer_list_unittest.cc ('k') | trunk/src/chrome/browser/sync/test/integration/sync_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698