Fix threading bugs in product label UI.

chrome/browser/ui/webui/help/help_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/help/help_handler.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 31 matching lines @@
 
 #if defined(OS_MACOSX)
 #include "chrome/browser/mac/obsolete_system.h"
 #endif
 
 #if defined(OS_CHROMEOS)
 #include "base/files/file_util_proxy.h"
 #include "base/i18n/time_formatting.h"
 #include "base/prefs/pref_service.h"
 #include "base/sys_info.h"
+#include "base/task_runner_util.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"
 #include "chrome/browser/ui/webui/help/help_utils_chromeos.h"
 #include "chrome/browser/ui/webui/help/version_updater_chromeos.h"
 #include "chromeos/chromeos_switches.h"
@@ skipping 57 matching lines @@
         chromeos::OwnerSettingsServiceChromeOSFactory::GetInstance()
             ->GetForBrowserContext(profile);
     // 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 false;
 }
 
+// Reads the file containing the FCC label text, if found. Must be called from
+// the blocking pool.
+std::string ReadFCCLabelText() {
+  const base::FilePath asset_dir(FILE_PATH_LITERAL(chrome::kChromeOSAssetPath));
+  const base::FilePath label_file_path =
+      asset_dir.AppendASCII(kFCCLabelTextPath);
+
+  std::string contents;
+  if (base::ReadFileToString(label_file_path, &contents))
+    return contents;
+  return std::string();
+}
+
 #endif  // defined(OS_CHROMEOS)
 
 }  // namespace
 
 HelpHandler::HelpHandler()
     : version_updater_(VersionUpdater::Create(nullptr)), weak_factory_(this) {
 }
 
 HelpHandler::~HelpHandler() {
 }
@@ skipping 257 matching lines @@
       "help.HelpPage.updateIsEnterpriseManaged",
       base::FundamentalValue(IsEnterpriseManaged()));
   // First argument to GetChannel() is a flag that indicates whether
   // current channel should be returned (if true) or target channel
   // (otherwise).
   version_updater_->GetChannel(true,
       base::Bind(&HelpHandler::OnCurrentChannel, weak_factory_.GetWeakPtr()));
   version_updater_->GetChannel(false,
       base::Bind(&HelpHandler::OnTargetChannel, weak_factory_.GetWeakPtr()));
 
-  BrowserThread::PostTask(
-      BrowserThread::FILE, FROM_HERE,
-      base::Bind(&HelpHandler::LoadFCCLabelText, weak_factory_.GetWeakPtr()));
+  base::PostTaskAndReplyWithResult(
+      content::BrowserThread::GetBlockingPool(),
+      FROM_HERE,
+      base::Bind(&ReadFCCLabelText),
+      base::Bind(&HelpHandler::OnFCCLabelTextRead,
+                 weak_factory_.GetWeakPtr()));
 #endif
 }
 
 #if defined(OS_MACOSX)
 void HelpHandler::PromoteUpdater(const base::ListValue* args) {
   version_updater_->PromoteUpdater();
 }
 #endif
 
 void HelpHandler::RelaunchNow(const base::ListValue* args) {
@@ skipping 161 matching lines @@
 void HelpHandler::OnCurrentChannel(const std::string& channel) {
   web_ui()->CallJavascriptFunction(
       "help.HelpPage.updateCurrentChannel", base::StringValue(channel));
 }
 
 void HelpHandler::OnTargetChannel(const std::string& channel) {
   web_ui()->CallJavascriptFunction(
       "help.HelpPage.updateTargetChannel", base::StringValue(channel));
 }
 
-void HelpHandler::LoadFCCLabelText() {
-  base::FilePath path(std::string(chrome::kChromeOSAssetPath) +
-                      kFCCLabelTextPath);
-  std::string contents;
-  if (base::ReadFileToString(path, &contents)) {
-    // Remove unnecessary whitespace.
-    base::StringValue label(base::CollapseWhitespaceASCII(contents, true));
-    web_ui()->CallJavascriptFunction("help.HelpPage.setProductLabelText",
-                                     label);
-  }
+void HelpHandler::OnFCCLabelTextRead(std::string text) {
+  // Remove unnecessary whitespace.
+  text = base::CollapseWhitespaceASCII(text, true);
+  if (text.empty())
+    return;
+  base::StringValue label(text);
+  web_ui()->CallJavascriptFunction("help.HelpPage.setProductLabelText", label);
 }
 
 #endif // defined(OS_CHROMEOS)