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

Unified Diff: src/extensions/experimental/i18n-utils.h

Issue 7014019: Adding DateTimeFormat class to i18n API. (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Narrow->Abbreviated. Added __ prefix to some private methods. Created 9 years, 7 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: src/extensions/experimental/i18n-utils.h
===================================================================
--- src/extensions/experimental/i18n-utils.h (revision 7878)
+++ src/extensions/experimental/i18n-utils.h (working copy)
@@ -28,6 +28,15 @@
#ifndef V8_EXTENSIONS_EXPERIMENTAL_I18N_UTILS_H_
#define V8_EXTENSIONS_EXPERIMENTAL_I18N_UTILS_H_
+#include <v8.h>
+
+#include "unicode/uversion.h"
+
+namespace U_ICU_NAMESPACE {
+class SimpleDateFormat;
jungshik at Google 2011/05/19 18:24:05 You don't need to forward-declare SimpleDateFormat
Nebojša Ćirić 2011/05/19 20:44:41 Done.
+class UnicodeString;
+}
+
namespace v8 {
namespace internal {
@@ -40,6 +49,12 @@
// TODO(cira): Find a way to use OS::SNPrintF instead.
static void StrNCopy(char* dest, int length, const char* src);
+ // Extract a string setting named in |settings| and set it to |result|.
+ // Return true if it's specified. Otherwise, return false.
+ static bool ExtractStringSetting(const v8::Handle<v8::Object>& settings,
+ const char* setting,
+ icu::UnicodeString* result);
+
private:
I18NUtils() {}
};

Powered by Google App Engine
This is Rietveld 408576698