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

Unified Diff: chrome/browser/dom_ui/chrome_url_data_manager.cc

Issue 381009: Increase the width for 'date-container' in download.html to 110px to make roo... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 12 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/app/resources/locale_settings_bn.xtb ('k') | chrome/browser/resources/downloads.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/dom_ui/chrome_url_data_manager.cc
===================================================================
--- chrome/browser/dom_ui/chrome_url_data_manager.cc (revision 35468)
+++ chrome/browser/dom_ui/chrome_url_data_manager.cc (working copy)
@@ -12,6 +12,9 @@
#include "base/string_util.h"
#include "base/thread.h"
#include "base/values.h"
+#if defined(OS_WIN)
+#include "base/win_util.h"
+#endif
#include "chrome/browser/browser_process.h"
#include "chrome/browser/chrome_thread.h"
#include "chrome/browser/net/chrome_url_request_context.h"
@@ -296,8 +299,16 @@
DictionaryValue* localized_strings) {
localized_strings->SetString(L"fontfamily",
l10n_util::GetString(IDS_WEB_FONT_FAMILY));
+
+ int web_font_size_id = IDS_WEB_FONT_SIZE;
+#if defined(OS_WIN)
+ // Some fonts used for some languages changed a lot in terms of the font
+ // metric in Vista. So, we need to use different size before Vista.
+ if (win_util::GetWinVersion() < win_util::WINVERSION_VISTA)
+ web_font_size_id = IDS_WEB_FONT_SIZE_XP;
+#endif
localized_strings->SetString(L"fontsize",
- l10n_util::GetString(IDS_WEB_FONT_SIZE));
+ l10n_util::GetString(web_font_size_id));
localized_strings->SetString(L"textdirection",
(l10n_util::GetTextDirection() == l10n_util::RIGHT_TO_LEFT) ?
« no previous file with comments | « chrome/app/resources/locale_settings_bn.xtb ('k') | chrome/browser/resources/downloads.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698