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

Unified Diff: chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc

Issue 23588009: Parse /etc/lsb-release only once on ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 3 months 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/extensions/file_manager/private_api_strings.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc b/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
index fb1d24e696ece0a2e9a54a0e4fd5a443e6d76da3..71fcde51c0c706e43d6e67fe288579c6ca4a0dd3 100644
--- a/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
@@ -4,8 +4,8 @@
#include "chrome/browser/chromeos/extensions/file_manager/private_api_strings.h"
+#include "base/sys_info.h"
#include "chrome/browser/chromeos/file_manager/open_with_browser.h"
-#include "chrome/browser/chromeos/system/statistics_provider.h"
#include "chrome/common/extensions/extension_l10n_util.h"
#include "grit/app_locale_settings.h"
#include "grit/generated_resources.h"
@@ -516,14 +516,8 @@ bool FileBrowserPrivateGetStringsFunction::RunImpl() {
webui::SetFontAndTextDirection(dict);
- std::string board;
- chromeos::system::StatisticsProvider* provider =
- chromeos::system::StatisticsProvider::GetInstance();
- if (!provider->GetMachineStatistic(chromeos::system::kMachineInfoBoard,
- &board)) {
- board = "unknown";
- }
- dict->SetString(chromeos::system::kMachineInfoBoard, board);
+ dict->SetString("CHROMEOS_RELEASE_BOARD",
+ base::SysInfo::GetLsbReleaseBoard());
dict->SetString("UI_LOCALE", extension_l10n_util::CurrentLocaleOrDefault());
« no previous file with comments | « chrome/browser/chromeos/chrome_browser_main_chromeos.cc ('k') | chrome/browser/chromeos/extensions/info_private_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698