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

Unified Diff: chrome/browser/ui/webui/settings/font_handler.cc

Issue 1405453002: [MD settings] adding c++ side of font settings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 'review changes' Created 5 years, 2 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
« no previous file with comments | « chrome/browser/ui/webui/settings/font_handler.h ('k') | chrome/browser/ui/webui/settings/md_settings_ui.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/settings/font_handler.cc
diff --git a/chrome/browser/ui/webui/settings/font_handler.cc b/chrome/browser/ui/webui/settings/font_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..5fb93cc1ff58e3ad13feebecc6f8194757ee3724
--- /dev/null
+++ b/chrome/browser/ui/webui/settings/font_handler.cc
@@ -0,0 +1,88 @@
+// Copyright 2015 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/settings/font_handler.h"
+
+#include "base/bind_helpers.h"
+#include "base/i18n/rtl.h"
+#include "base/prefs/pref_service.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/character_encoding.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/webui/options/font_settings_utils.h"
+#include "chrome/common/pref_names.h"
+#include "content/public/browser/font_list_async.h"
+#include "content/public/browser/web_ui.h"
+
+namespace settings {
+
+FontHandler::FontHandler(content::WebUI* webui)
+ : weak_ptr_factory_(this) {
+ // Perform validation for saved fonts.
+ PrefService* pref_service = Profile::FromWebUI(webui)->GetPrefs();
+ options::FontSettingsUtilities::ValidateSavedFonts(pref_service);
+}
+
+FontHandler::~FontHandler() {}
+
+void FontHandler::RegisterMessages() {
+ web_ui()->RegisterMessageCallback(
+ "fetchFontsData", base::Bind(&FontHandler::HandleFetchFontsData,
+ base::Unretained(this)));
+}
+
+void FontHandler::HandleFetchFontsData(
+ const base::ListValue* /*args*/) {
+ content::GetFontListAsync(base::Bind(&FontHandler::FontListHasLoaded,
+ weak_ptr_factory_.GetWeakPtr()));
+}
+
+void FontHandler::FontListHasLoaded(scoped_ptr<base::ListValue> list) {
+ // Font list. Selects the directionality for the fonts in the given list.
+ for (size_t i = 0; i < list->GetSize(); i++) {
+ base::ListValue* font;
+ bool has_font = list->GetList(i, &font);
+ DCHECK(has_font);
+
+ base::string16 value;
+ bool has_value = font->GetString(1, &value);
+ DCHECK(has_value);
+
+ bool has_rtl_chars = base::i18n::StringContainsStrongRTLChars(value);
+ font->Append(new base::StringValue(has_rtl_chars ? "rtl" : "ltr"));
+ }
+
+ // Character encoding list.
+ const std::vector<CharacterEncoding::EncodingInfo>* encodings;
+ PrefService* pref_service = Profile::FromWebUI(web_ui())->GetPrefs();
+ encodings = CharacterEncoding::GetCurrentDisplayEncodings(
+ g_browser_process->GetApplicationLocale(),
+ pref_service->GetString(prefs::kStaticEncodings),
+ pref_service->GetString(prefs::kRecentlySelectedEncoding));
+ DCHECK(!encodings->empty());
+
+ base::ListValue encoding_list;
+ for (const auto& it : *encodings) {
+ scoped_ptr<base::ListValue> option(new base::ListValue());
+ if (it.encoding_id) {
+ option->AppendString(
+ CharacterEncoding::GetCanonicalEncodingNameByCommandId(
+ it.encoding_id));
+ option->AppendString(it.encoding_display_name);
+ option->AppendString(
+ base::i18n::StringContainsStrongRTLChars(it.encoding_display_name)
+ ? "rtl"
+ : "ltr");
+ } else {
+ // Add empty value to indicate a separator item.
+ option->AppendString(std::string());
+ }
+ encoding_list.Append(option.Pass());
+ }
+
+ web_ui()->CallJavascriptFunction("Settings.setFontsData", *list,
+ encoding_list);
+}
+
+} // namespace settings
« no previous file with comments | « chrome/browser/ui/webui/settings/font_handler.h ('k') | chrome/browser/ui/webui/settings/md_settings_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698