Chromium Code Reviews| Index: chrome/browser/chromeos/enrollment_dialog_view.cc |
| diff --git a/chrome/browser/chromeos/enrollment_dialog_view.cc b/chrome/browser/chromeos/enrollment_dialog_view.cc |
| index 31e3762de60d2589edb619769e7d77428256dd11..d155b98ce04c5c66b8377450fc935bc4bef5f6d8 100644 |
| --- a/chrome/browser/chromeos/enrollment_dialog_view.cc |
| +++ b/chrome/browser/chromeos/enrollment_dialog_view.cc |
| @@ -10,6 +10,7 @@ |
| #include "chrome/browser/extensions/extension_host.h" |
| #include "chrome/browser/profiles/profile.h" |
| #include "chrome/browser/ui/browser_navigator.h" |
| +#include "chromeos/network/network_event_log.h" |
| #include "content/public/common/page_transition_types.h" |
| #include "extensions/common/constants.h" |
| #include "grit/generated_resources.h" |
| @@ -192,7 +193,7 @@ class DialogEnrollmentDelegate : public EnrollmentDelegate { |
| virtual ~DialogEnrollmentDelegate(); |
| // EnrollmentDelegate overrides |
| - virtual void Enroll(const std::vector<std::string>& uri_list, |
| + virtual bool Enroll(const std::vector<std::string>& uri_list, |
| const base::Closure& connect) OVERRIDE; |
| private: |
| @@ -212,8 +213,8 @@ DialogEnrollmentDelegate::DialogEnrollmentDelegate( |
| DialogEnrollmentDelegate::~DialogEnrollmentDelegate() {} |
| -void DialogEnrollmentDelegate::Enroll(const std::vector<std::string>& uri_list, |
| - const base::Closure& connect) { |
| +bool DialogEnrollmentDelegate::Enroll(const std::vector<std::string>& uri_list, |
| + const base::Closure& post_action) { |
| // Keep the closure for later activation if we notice that |
| // a certificate has been added. |
| @@ -229,8 +230,8 @@ void DialogEnrollmentDelegate::Enroll(const std::vector<std::string>& uri_list, |
| EnrollmentDialogView::ShowDialog(owning_window_, |
| network_name_, |
| profile_, |
| - uri, connect); |
| - return; |
| + uri, post_action); |
| + return true; |
| } |
| } |
| @@ -238,7 +239,8 @@ void DialogEnrollmentDelegate::Enroll(const std::vector<std::string>& uri_list, |
| // TODO(gspencer): provide a path to display this failure to the user. (but |
| // for the most part they won't know what it means, since it's probably coming |
|
pneubeck (no reviews)
2013/08/06 15:45:19
TODO comment obsolete now with the return value?
stevenjb
2013/08/06 20:23:55
Done.
|
| // from a policy-pushed ONC file). |
| - VLOG(1) << "Couldn't find usable scheme in enrollment URI(s)"; |
| + NET_LOG_EVENT("No usable enrollment URI", network_name_); |
| + return false; |
| } |
| } // namespace |