Revert of Call to CrosSettings::Set() is replaced by OwnerSettingsService::Set() in VersionUpda...

Revert of Call to CrosSettings::Set() is replaced by OwnerSettingsService::Set() in VersionUpdaterCros. (patchset #7 id:120001 of https://codereview.chromium.org/738843002/)

Reason for revert:
Speculative revert to see if this is the cause of issue 440063.

BUG=440063
TBR=ygorshenin@chromium.org

Original issue's description:
> Call to CrosSettings::Set() is replaced by OwnerSettingsService::Set() in VersionUpdaterCros. Also, fixed permissions checks for channel switcher UI in a multi-profiles sessions.
>
> BUG=433840
> TEST=existing unit_tests for OwnerSettingsService
>
> Committed: https://crrev.com/e0ed5a18bc9e3fa930f1af3b8f9c3cfdb91329f5
> Cr-Commit-Position: refs/heads/master@{#305012}

TBR=nkostylev@chromium.org,ygorshenin@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=433840
Committed: https://crrev.com/ed8e4785e42fdf65d79656fb922f2d1b16e9857b
Cr-Commit-Position: refs/heads/master@{#307710}

[earthdok, 2014-12-10 15:54:14]

Created Revert of Call to CrosSettings::Set() is replaced by
OwnerSettingsService::Set() in VersionUpdaterCros.

[earthdok, 2014-12-10 15:55:33]

The CQ bit was checked by earthdok@chromium.org

[commit-bot: I haz the power, 2014-12-10 15:56:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/790183002/1

[commit-bot: I haz the power, 2014-12-10 15:58:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-10 15:58:02]

Failed to apply patch for chrome/browser/ui/webui/help/help_handler.cc:
While running git apply --index -3 -p1;
  error: patch failed: chrome/browser/ui/webui/help/help_handler.cc:49
  Falling back to three-way merge...
  Applied patch to 'chrome/browser/ui/webui/help/help_handler.cc' with
conflicts.
  U chrome/browser/ui/webui/help/help_handler.cc

Patch:       chrome/browser/ui/webui/help/help_handler.cc
Index: chrome/browser/ui/webui/help/help_handler.cc
diff --git a/chrome/browser/ui/webui/help/help_handler.cc
b/chrome/browser/ui/webui/help/help_handler.cc
index
b88247f3f3061447b0ef0f9e38ef3f2e5bed1c6b..97a636af9a22845ca5068b39bf0421082ba7ea21
100644
--- a/chrome/browser/ui/webui/help/help_handler.cc
+++ b/chrome/browser/ui/webui/help/help_handler.cc
@@ -33,7 +33,6 @@
 #include "components/google/core/browser/google_util.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
-#include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/common/user_agent.h"
 #include "grit/components_strings.h"
@@ -49,10 +48,7 @@
 #include "base/i18n/time_formatting.h"
 #include "base/prefs/pref_service.h"
 #include "base/sys_info.h"
-#include "chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h"
-#include
"chrome/browser/chromeos/ownership/owner_settings_service_chromeos_factory.h"
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
-#include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/chromeos/image_source.h"
@@ -92,7 +88,7 @@
 }
 
 // Returns true if current user can change channel, false otherwise.
-bool CanChangeChannel(Profile* profile) {
+bool CanChangeChannel() {
   bool value = false;
   chromeos::CrosSettings::Get()->GetBoolean(chromeos::kReleaseChannelDelegated,
                                             &value);
@@ -104,24 +100,18 @@
       return false;
     // Get the currently logged in user and strip the domain part only.
     std::string domain = "";
-    const user_manager::User* user =
-        profile ? chromeos::ProfileHelper::Get()->GetUserByProfile(profile)
-                : nullptr;
-    std::string email = user ? user->email() : std::string();
-    size_t at_pos = email.find('@');
-    if (at_pos != std::string::npos && at_pos + 1 < email.length())
-      domain = email.substr(email.find('@') + 1);
+    std::string user =
+        user_manager::UserManager::Get()->GetLoggedInUser()->email();
+    size_t at_pos = user.find('@');
+    if (at_pos != std::string::npos && at_pos + 1 < user.length())
+      domain = user.substr(user.find('@') + 1);
     policy::BrowserPolicyConnectorChromeOS* connector =
        
g_browser_process->platform_part()->browser_policy_connector_chromeos();
     return domain == connector->GetEnterpriseDomain();
-  } else {
-    chromeos::OwnerSettingsServiceChromeOS* service =
-        chromeos::OwnerSettingsServiceChromeOSFactory::GetInstance()
-            ->GetForBrowserContext(profile);
+  } else if (user_manager::UserManager::Get()->IsCurrentUserOwner()) {
     // On non managed machines we have local owner who is the only one to
change
     // anything. Ensure that ReleaseChannelDelegated is false.
-    if (service && service->IsOwner())
-      return !value;
+    return !value;
   }
   return false;
 }
@@ -131,7 +121,8 @@
 }  // namespace
 
 HelpHandler::HelpHandler()
-    : version_updater_(VersionUpdater::Create(nullptr)), weak_factory_(this) {
+    : version_updater_(VersionUpdater::Create()),
+      weak_factory_(this) {
 }
 
 HelpHandler::~HelpHandler() {
@@ -285,10 +276,6 @@
 }
 
 void HelpHandler::RegisterMessages() {
-#if defined(OS_CHROMEOS)
-  version_updater_.reset(
-      VersionUpdater::Create(web_ui()->GetWebContents()->GetBrowserContext()));
-#endif
   registrar_.Add(this, chrome::NOTIFICATION_UPGRADE_RECOMMENDED,
                  content::NotificationService::AllSources());
 
@@ -364,7 +351,7 @@
 
   web_ui()->CallJavascriptFunction(
       "help.HelpPage.updateEnableReleaseChannel",
-      base::FundamentalValue(CanChangeChannel(Profile::FromWebUI(web_ui()))));
+      base::FundamentalValue(CanChangeChannel()));
 
   base::Time build_time = base::SysInfo::GetLsbReleaseTime();
   base::string16 build_date = base::TimeFormatFriendlyDate(build_time);
@@ -439,7 +426,7 @@
 void HelpHandler::SetChannel(const base::ListValue* args) {
   DCHECK(args->GetSize() == 2);
 
-  if (!CanChangeChannel(Profile::FromWebUI(web_ui()))) {
+  if (!CanChangeChannel()) {
     LOG(WARNING) << "Non-owner tried to change release track.";
     return;
   }

[commit-bot: I haz the power, 2014-12-10 15:58:04]

The CQ bit was unchecked by commit-bot@chromium.org

[earthdok, 2014-12-10 16:12:05]

The CQ bit was checked by earthdok@chromium.org

[commit-bot: I haz the power, 2014-12-10 16:12:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/790183002/250001

[commit-bot: I haz the power, 2014-12-10 16:13:39]

Committed patchset #2 (id:250001)

[commit-bot: I haz the power, 2014-12-10 16:14:34]

Patchset 2 (id:??) landed as
https://crrev.com/ed8e4785e42fdf65d79656fb922f2d1b16e9857b
Cr-Commit-Position: refs/heads/master@{#307710}

[earthdok, 2014-12-10 16:15:38]

On 2014/12/10 16:13:39, I haz the power (commit-bot) wrote:
> Committed patchset #2 (id:250001)

The hell? I deleted NOTREECHECKS and NOTRY when I tried to commit PS#1. But this
one just lands without CQ?

Hopefully it doesn't break anything.

[earthdok, 2014-12-10 16:20:14]

On 2014/12/10 16:15:38, earthdok wrote:
> On 2014/12/10 16:13:39, I haz the power (commit-bot) wrote:
> > Committed patchset #2 (id:250001)
> 
> The hell? I deleted NOTREECHECKS and NOTRY when I tried to commit PS#1. But
this
> one just lands without CQ?
> 
> Hopefully it doesn't break anything.

Apparently this does break compilation. Reverting back.

[earthdok, 2014-12-10 16:20:45]

A revert of this CL (patchset #2 id:250001) has been created in
https://codereview.chromium.org/788243002/ by earthdok@chromium.org.

The reason for reverting is: Accidentally committed without CQ, broke build :(.