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

Unified Diff: chrome/browser/chromeos/policy/consumer_management_notifier.cc

Issue 760373002: Fixed some styling issues and added missing header files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/policy/consumer_management_notifier.cc
diff --git a/chrome/browser/chromeos/policy/consumer_management_notifier.cc b/chrome/browser/chromeos/policy/consumer_management_notifier.cc
index 8e96666b4b265b550c8428e4c57395d1aa797603..4e0d96ff96e7b4dfa950a1d57c4078ae23ad5118 100644
--- a/chrome/browser/chromeos/policy/consumer_management_notifier.cc
+++ b/chrome/browser/chromeos/policy/consumer_management_notifier.cc
@@ -4,8 +4,11 @@
#include "chrome/browser/chromeos/policy/consumer_management_notifier.h"
+#include <string>
+
#include "base/bind.h"
#include "base/callback.h"
+#include "base/strings/string16.h"
#include "base/strings/utf_string_conversions.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/notifications/notification.h"
@@ -20,6 +23,7 @@
#include "ui/base/page_transition_types.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/base/window_open_disposition.h"
+#include "ui/message_center/notification_types.h"
#include "ui/message_center/notifier_settings.h"
#include "url/gurl.h"
@@ -47,8 +51,8 @@ class DesktopNotificationDelegate : public NotificationDelegate {
private:
~DesktopNotificationDelegate() override;
- std::string id_;
- base::Closure button_click_callback_;
+ const std::string id_;
+ const base::Closure button_click_callback_;
DISALLOW_COPY_AND_ASSIGN(DesktopNotificationDelegate);
};
@@ -93,7 +97,7 @@ void ConsumerManagementNotifier::Shutdown() {
void ConsumerManagementNotifier::OnConsumerManagementStatusChanged() {
if (consumer_management_service_->HasPendingEnrollmentNotification()) {
- ConsumerManagementService::EnrollmentStage stage =
+ const ConsumerManagementService::EnrollmentStage stage =
consumer_management_service_->GetEnrollmentStage();
// Reset the enrollment stage so that we won't show the same notification,
// again.

Powered by Google App Engine
This is Rietveld 408576698