[Cleanup] Clean up Easy Unlock code a bit.

chrome/browser/ui/webui/options/browser_options_handler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/options/browser_options_handler.h"
 
 #include <set>
 #include <string>
 #include <vector>
 
@@ skipping 2017 matching lines @@
 void BrowserOptionsHandler::SetupManagingSupervisedUsers() {
   bool has_users = !Profile::FromWebUI(web_ui())->
       GetPrefs()->GetDictionary(prefs::kSupervisedUsers)->empty();
   base::FundamentalValue has_users_value(has_users);
   web_ui()->CallJavascriptFunction(
       "BrowserOptions.updateManagesSupervisedUsers",
       has_users_value);
 }
 
 void BrowserOptionsHandler::SetupEasyUnlock() {
+  // TODO(isherman): Is the below TODO still relevant?
   // TODO(xiyuan): Update when pairing data is really availble.

[Ilya Sherman, 2015/02/05 02:27:50]

Xiyuan, could you please clarify what you meant by this TODO?  Thanks :)

[xiyuan, 2015/02/05 03:30:09]

The TODO could be removed. It is done. We were checking prefs directly in
initial version when the TODO is added. It should be removed when we switch to
use EasyUnlockService API.

-  const base::ListValue* devices =
-      EasyUnlockService::Get(Profile::FromWebUI(web_ui()))->GetRemoteDevices();
-  bool has_pairing = devices && !devices->empty();
-  base::FundamentalValue has_pairing_value(has_pairing);
+  base::FundamentalValue is_enabled(
+      EasyUnlockService::Get(Profile::FromWebUI(web_ui()))->IsEnabled());
   web_ui()->CallJavascriptFunction(
       "BrowserOptions.updateEasyUnlock",
-      has_pairing_value);
+      is_enabled);
 }
 
 void BrowserOptionsHandler::SetupExtensionControlledIndicators() {
 #if defined(OS_WIN)
   base::DictionaryValue extension_controlled;
 
   // Check if an extension is overriding the Search Engine.
   const extensions::Extension* extension =
       extensions::GetExtensionOverridingSearchEngine(
           Profile::FromWebUI(web_ui()));
@@ skipping 73 matching lines @@
 void BrowserOptionsHandler::OnPolicyUpdated(const policy::PolicyNamespace& ns,
                                             const policy::PolicyMap& previous,
                                             const policy::PolicyMap& current) {
   std::set<std::string> different_keys;
   current.GetDifferingKeys(previous, &different_keys);
   if (ContainsKey(different_keys, policy::key::kMetricsReportingEnabled))
     SetupMetricsReportingCheckbox();
 }
 
 }  // namespace options