Remove unused code for old input[type=date] UI
authortkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Oct 2012 12:10:07 +0000 (12:10 +0000)
committertkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Oct 2012 12:10:07 +0000 (12:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=100734

Reviewed by Hajime Morita.

Source/WebCore:

The old input[type=date] UI used in Google Chrome 20-23 (text field with
a fixed placeholder) was replaced with ENABLE_INPUT_MULTIPLE_FIELDS_UI.

No new tests because of no behavior changes.

* html/DateInputType.cpp: Remove ENABLE_INPUT_TYPE_DATE_LEGACY_UI code path.
* html/DateInputType.h: Ditto.

* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::supportsPlaceholder):
Remove fixed-placeholder feature.
* html/HTMLInputElement.h:
(HTMLInputElement): Ditto.
* html/HTMLTextFormControlElement.h:
(HTMLTextFormControlElement): isPlaceholderEmpty() is not needed to be virtual
* html/InputType.cpp: Remove fixed-placeholder feature.
* html/InputType.h: Ditto.
* html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::updatePlaceholderText): Ditto.

* platform/text/PlatformLocale.h:
(Locale): Remove dateFormatText.
* platform/text/LocaleICU.cpp: Ditto.
* platform/text/LocaleICU.h: Ditto.
* platform/text/LocaleNone.cpp: Ditto.
* platform/text/mac/LocaleMac.h: Ditto.
* platform/text/mac/LocaleMac.mm: Ditto.
* platform/text/win/LocaleWin.cpp: Ditto.
* platform/text/win/LocaleWin.h: Ditto.
* platform/LocalizedStrings.h:
(WebCore): Remove unused functions; calendarTodayText,
calendarClearText, dateFormatYearText, dateFormatMonthText,
dateFormatDayInMonthText

Source/WebKit/chromium:

* src/LocalizedStrings.cpp:
Follow the LocalizedStrings.h change.
* tests/LocaleMacTest.cpp:
(LocaleMacTest): Remove tests for Locale::dateFormatText.
* tests/LocaleWinTest.cpp:
(LocaleWinTest): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@132896 268f45cc-cd09-0410-ab3c-d52691b4dbfc

22 files changed:
Source/WebCore/ChangeLog
Source/WebCore/html/DateInputType.cpp
Source/WebCore/html/DateInputType.h
Source/WebCore/html/HTMLInputElement.cpp
Source/WebCore/html/HTMLInputElement.h
Source/WebCore/html/HTMLTextFormControlElement.h
Source/WebCore/html/InputType.cpp
Source/WebCore/html/InputType.h
Source/WebCore/html/TextFieldInputType.cpp
Source/WebCore/platform/LocalizedStrings.h
Source/WebCore/platform/text/LocaleICU.cpp
Source/WebCore/platform/text/LocaleICU.h
Source/WebCore/platform/text/LocaleNone.cpp
Source/WebCore/platform/text/PlatformLocale.h
Source/WebCore/platform/text/mac/LocaleMac.h
Source/WebCore/platform/text/mac/LocaleMac.mm
Source/WebCore/platform/text/win/LocaleWin.cpp
Source/WebCore/platform/text/win/LocaleWin.h
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/LocalizedStrings.cpp
Source/WebKit/chromium/tests/LocaleMacTest.cpp
Source/WebKit/chromium/tests/LocaleWinTest.cpp

index c11ab3e..9d7b599 100644 (file)
@@ -1,5 +1,46 @@
 2012-10-30  Kent Tamura  <tkent@chromium.org>
 
+        Remove unused code for old input[type=date] UI
+        https://bugs.webkit.org/show_bug.cgi?id=100734
+
+        Reviewed by Hajime Morita.
+
+        The old input[type=date] UI used in Google Chrome 20-23 (text field with
+        a fixed placeholder) was replaced with ENABLE_INPUT_MULTIPLE_FIELDS_UI.
+
+        No new tests because of no behavior changes.
+
+        * html/DateInputType.cpp: Remove ENABLE_INPUT_TYPE_DATE_LEGACY_UI code path.
+        * html/DateInputType.h: Ditto.
+
+        * html/HTMLInputElement.cpp:
+        (WebCore::HTMLInputElement::supportsPlaceholder):
+        Remove fixed-placeholder feature.
+        * html/HTMLInputElement.h:
+        (HTMLInputElement): Ditto.
+        * html/HTMLTextFormControlElement.h:
+        (HTMLTextFormControlElement): isPlaceholderEmpty() is not needed to be virtual
+        * html/InputType.cpp: Remove fixed-placeholder feature.
+        * html/InputType.h: Ditto.
+        * html/TextFieldInputType.cpp:
+        (WebCore::TextFieldInputType::updatePlaceholderText): Ditto.
+
+        * platform/text/PlatformLocale.h:
+        (Locale): Remove dateFormatText.
+        * platform/text/LocaleICU.cpp: Ditto.
+        * platform/text/LocaleICU.h: Ditto.
+        * platform/text/LocaleNone.cpp: Ditto.
+        * platform/text/mac/LocaleMac.h: Ditto.
+        * platform/text/mac/LocaleMac.mm: Ditto.
+        * platform/text/win/LocaleWin.cpp: Ditto.
+        * platform/text/win/LocaleWin.h: Ditto.
+        * platform/LocalizedStrings.h:
+        (WebCore): Remove unused functions; calendarTodayText,
+        calendarClearText, dateFormatYearText, dateFormatMonthText,
+        dateFormatDayInMonthText
+
+2012-10-30  Kent Tamura  <tkent@chromium.org>
+
         Fix crash by calendar picker or suggestion picker
         https://bugs.webkit.org/show_bug.cgi?id=100728
 
index 989b30d..ab8ac7c 100644 (file)
@@ -53,9 +53,6 @@ static const int dateStepScaleFactor = 86400000;
 
 inline DateInputType::DateInputType(HTMLInputElement* element)
     : BaseDateInputType(element)
-#if ENABLE(INPUT_TYPE_DATE_LEGACY_UI)
-    , m_pickerElement(0)
-#endif
 {
 }
 
@@ -103,73 +100,7 @@ bool DateInputType::isDateField() const
     return true;
 }
 
-#if ENABLE(INPUT_TYPE_DATE_LEGACY_UI)
-void DateInputType::createShadowSubtree()
-{
-    BaseDateAndTimeInputType::createShadowSubtree();
-    RefPtr<PickerIndicatorElement> pickerElement = PickerIndicatorElement::create(element()->document());
-    m_pickerElement = pickerElement.get();
-    containerElement()->insertBefore(m_pickerElement, innerBlockElement()->nextSibling(), ASSERT_NO_EXCEPTION);
-}
-
-void DateInputType::destroyShadowSubtree()
-{
-    TextFieldInputType::destroyShadowSubtree();
-    m_pickerElement = 0;
-}
-
-bool DateInputType::needsContainer() const
-{
-    return true;
-}
-
-bool DateInputType::shouldHaveSpinButton() const
-{
-    return false;
-}
-
-void DateInputType::handleKeydownEvent(KeyboardEvent* event)
-{
-    if (element()->disabled() || element()->readOnly())
-        return;
-    if (event->keyIdentifier() == "Down") {
-        if (m_pickerElement)
-            m_pickerElement->openPopup();
-        event->setDefaultHandled();
-        return;
-    }
-    BaseDateAndTimeInputType::handleKeydownEvent(event);
-}
-
-void DateInputType::handleBlurEvent()
-{
-    if (m_pickerElement)
-        m_pickerElement->closePopup();
-
-    // Reset the renderer value, which might be unmatched with the element value.
-    element()->setFormControlValueMatchesRenderer(false);
-    // We need to reset the renderer value explicitly because an unacceptable
-    // renderer value should be purged before style calculation.
-    updateInnerTextValue();
-}
-
-bool DateInputType::supportsPlaceholder() const
-{
-    return true;
-}
-
-bool DateInputType::usesFixedPlaceholder() const
-{
-    return true;
-}
-
-String DateInputType::fixedPlaceholder()
-{
-    return element()->locale().dateFormatText();
-}
-#endif // ENABLE(INPUT_TYPE_DATE_LEGACY_UI)
-
-#if ENABLE(INPUT_MULTIPLE_FIELDS_UI) && !ENABLE(INPUT_TYPE_DATE_LEGACY_UI)
+#if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
 String DateInputType::formatDateTimeFieldsState(const DateTimeFieldsState& dateTimeFieldsState) const
 {
     if (!dateTimeFieldsState.hasDayOfMonth() || !dateTimeFieldsState.hasMonth() || !dateTimeFieldsState.hasYear())
index 934ac26..1f8b72e 100644 (file)
@@ -39,7 +39,7 @@ namespace WebCore {
 
 class PickerIndicatorElement;
 
-#if ENABLE(INPUT_MULTIPLE_FIELDS_UI) && !ENABLE(INPUT_TYPE_DATE_LEGACY_UI)
+#if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
 typedef BaseMultipleFieldsDateAndTimeInputType BaseDateInputType;
 #else
 typedef BaseDateAndTimeInputType BaseDateInputType;
@@ -58,27 +58,11 @@ private:
     virtual bool setMillisecondToDateComponents(double, DateComponents*) const OVERRIDE;
     virtual bool isDateField() const OVERRIDE;
 
-#if ENABLE(INPUT_TYPE_DATE_LEGACY_UI)
-    virtual void createShadowSubtree() OVERRIDE;
-    virtual void destroyShadowSubtree() OVERRIDE;
-    virtual void handleKeydownEvent(KeyboardEvent*) OVERRIDE;
-    virtual void handleBlurEvent() OVERRIDE;
-    virtual bool supportsPlaceholder() const OVERRIDE;
-    virtual bool usesFixedPlaceholder() const OVERRIDE;
-    virtual String fixedPlaceholder() OVERRIDE;
-
-    // TextFieldInputType functions
-    virtual bool needsContainer() const OVERRIDE;
-    virtual bool shouldHaveSpinButton() const OVERRIDE;
-
-    PickerIndicatorElement* m_pickerElement;
-#else
 #if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
     // BaseMultipleFieldsDateAndTimeInputType functions
     virtual String formatDateTimeFieldsState(const DateTimeFieldsState&) const OVERRIDE;
     virtual void setupLayoutParameters(DateTimeEditElement::LayoutParameters&, const DateComponents&) const OVERRIDE;
 #endif
-#endif
 };
 
 } // namespace WebCore
index db84892..bda4471 100644 (file)
@@ -1742,13 +1742,6 @@ bool HTMLInputElement::supportsPlaceholder() const
     return m_inputType->supportsPlaceholder();
 }
 
-bool HTMLInputElement::isPlaceholderEmpty() const
-{
-    if (m_inputType->usesFixedPlaceholder())
-        return m_inputType->fixedPlaceholder().isEmpty();
-    return HTMLTextFormControlElement::isPlaceholderEmpty();
-}
-
 void HTMLInputElement::updatePlaceholderText()
 {
     return m_inputType->updatePlaceholderText();
index f6edb5e..ef4d76e 100644 (file)
@@ -369,7 +369,6 @@ private:
     bool tooLong(const String&, NeedsToCheckDirtyFlag) const;
 
     virtual bool supportsPlaceholder() const;
-    virtual bool isPlaceholderEmpty() const OVERRIDE;
     virtual void updatePlaceholderText();
     virtual bool isEmptyValue() const OVERRIDE { return innerTextValue().isEmpty(); }
     virtual bool isEmptySuggestedValue() const { return suggestedValue().isEmpty(); }
index c7e8691..9a5fa61 100644 (file)
@@ -89,7 +89,7 @@ public:
 
 protected:
     HTMLTextFormControlElement(const QualifiedName&, Document*, HTMLFormElement*);
-    virtual bool isPlaceholderEmpty() const;
+    bool isPlaceholderEmpty() const;
     virtual void updatePlaceholderText() = 0;
 
     virtual void parseAttribute(const Attribute&) OVERRIDE;
index 4195e8e..d669cd6 100644 (file)
@@ -871,16 +871,6 @@ bool InputType::supportsPlaceholder() const
     return false;
 }
 
-bool InputType::usesFixedPlaceholder() const
-{
-    return false;
-}
-
-String InputType::fixedPlaceholder()
-{
-    return String();
-}
-
 void InputType::updateInnerTextValue()
 {
 }
index 0c2f010..32a55f2 100644 (file)
@@ -268,14 +268,7 @@ public:
     virtual bool isCheckable();
     virtual bool isSteppable() const;
     virtual bool shouldRespectHeightAndWidthAttributes();
-    // If supportsPlaceholder() && !usesFixedPlaceholder(), it means a type
-    // supports the 'placeholder' attribute.
-    // If supportsPlaceholder() && usesFixedPlaceholder(), it means a type
-    // doesn't support the 'placeholder' attribute, but shows
-    // fixedPlaceholder() string as a placeholder.
     virtual bool supportsPlaceholder() const;
-    virtual bool usesFixedPlaceholder() const;
-    virtual String fixedPlaceholder();
     virtual void updateInnerTextValue();
     virtual void updatePlaceholderText();
     virtual void multipleAttributeChanged();
index 3e6c1ee..a5c2745 100644 (file)
@@ -402,7 +402,7 @@ void TextFieldInputType::updatePlaceholderText()
     if (!supportsPlaceholder())
         return;
     ExceptionCode ec = 0;
-    String placeholderText = usesFixedPlaceholder() ? fixedPlaceholder() : element()->strippedPlaceholder();
+    String placeholderText = element()->strippedPlaceholder();
     if (placeholderText.isEmpty()) {
         if (m_placeholder) {
             m_placeholder->parentNode()->removeChild(m_placeholder.get(), ec);
index 8c56fe1..b4ba219 100644 (file)
@@ -246,13 +246,6 @@ namespace WebCore {
     String validationMessageRangeUnderflowText(const String& minimum);
     String validationMessageRangeOverflowText(const String& maximum);
     String validationMessageStepMismatchText(const String& base, const String& step);
-#if ENABLE(CALENDAR_PICKER)
-    String calendarTodayText();
-    String calendarClearText();
-    String dateFormatYearText();
-    String dateFormatMonthText();
-    String dateFormatDayInMonthText();
-#endif
 #if USE(SOUP)
     String unacceptableTLSCertificate();
 #endif
index 9a4f219..302118e 100644 (file)
@@ -193,76 +193,6 @@ static String getDateFormatPattern(const UDateFormat* dateFormat)
 #endif
 
 #if ENABLE(CALENDAR_PICKER)
-static inline bool isICUYearSymbol(UChar letter)
-{
-    return letter == 'y' || letter == 'Y';
-}
-
-static inline bool isICUMonthSymbol(UChar letter)
-{
-    return letter == 'M';
-}
-
-static inline bool isICUDayInMonthSymbol(UChar letter)
-{
-    return letter == 'd';
-}
-
-// Specification of the input:
-// http://icu-project.org/apiref/icu4c/classSimpleDateFormat.html#details
-static String localizeFormat(const String& buffer)
-{
-    StringBuilder builder;
-    UChar lastChar = 0;
-    bool inQuote = false;
-    for (unsigned i = 0; i < buffer.length(); ++i) {
-        if (inQuote) {
-            if (buffer[i] == '\'') {
-                inQuote = false;
-                lastChar = 0;
-                ASSERT(i);
-                if (buffer[i - 1] == '\'')
-                    builder.append('\'');
-            } else
-                builder.append(buffer[i]);
-        } else {
-            if (isASCIIAlpha(lastChar) && lastChar == buffer[i])
-                continue;
-            lastChar = buffer[i];
-            if (isICUYearSymbol(lastChar)) {
-                String text = dateFormatYearText();
-                builder.append(text.isEmpty() ? "Year" : text);
-            } else if (isICUMonthSymbol(lastChar)) {
-                String text = dateFormatMonthText();
-                builder.append(text.isEmpty() ? "Month" : text);
-            } else if (isICUDayInMonthSymbol(lastChar)) {
-                String text = dateFormatDayInMonthText();
-                builder.append(text.isEmpty() ? "Day" : text);
-            } else if (lastChar == '\'')
-                inQuote = true;
-            else
-                builder.append(lastChar);
-        }
-    }
-    return builder.toString();
-}
-
-void LocaleICU::initializeLocalizedDateFormatText()
-{
-    if (!m_localizedDateFormatText.isNull())
-        return;
-    m_localizedDateFormatText = emptyString();
-    if (!initializeShortDateFormat())
-        return;
-    m_localizedDateFormatText = localizeFormat(getDateFormatPattern(m_shortDateFormat));
-}
-
-String LocaleICU::dateFormatText()
-{
-    initializeLocalizedDateFormatText();
-    return m_localizedDateFormatText;
-}
-
 PassOwnPtr<Vector<String> > LocaleICU::createLabelVector(const UDateFormat* dateFormat, UDateFormatSymbolType type, int32_t startIndex, int32_t size)
 {
     if (!dateFormat)
index a5a4fee..1cd67c8 100644 (file)
@@ -52,8 +52,6 @@ public:
     // For LocalizedDate
     virtual double parseDateTime(const String&, DateComponents::Type) OVERRIDE;
 #if ENABLE(CALENDAR_PICKER)
-    virtual String dateFormatText() OVERRIDE;
-
     virtual const Vector<String>& weekDayShortLabels() OVERRIDE;
     virtual unsigned firstDayOfWeek() OVERRIDE;
     virtual bool isRTL() OVERRIDE;
@@ -85,7 +83,6 @@ private:
     UDateFormat* openDateFormat(UDateFormatStyle timeStyle, UDateFormatStyle dateStyle) const;
 
 #if ENABLE(CALENDAR_PICKER)
-    void initializeLocalizedDateFormatText();
     void initializeCalendar();
 #endif
 
@@ -104,7 +101,6 @@ private:
     bool m_didCreateShortDateFormat;
 
 #if ENABLE(CALENDAR_PICKER)
-    String m_localizedDateFormatText;
     OwnPtr<Vector<String> > m_weekDayShortLabels;
     unsigned m_firstDayOfWeek;
 #endif
index 35afcf7..82858b6 100644 (file)
@@ -38,7 +38,6 @@ private:
     virtual void initializeLocaleData() OVERRIDE FINAL;
     virtual double parseDateTime(const String&, DateComponents::Type) OVERRIDE;
 #if ENABLE(CALENDAR_PICKER)
-    virtual String dateFormatText() OVERRIDE;
     virtual bool isRTL() OVERRIDE;
 #endif
 #if ENABLE(CALENDAR_PICKER) || ENABLE(INPUT_MULTIPLE_FIELDS_UI)
@@ -81,11 +80,6 @@ double LocaleNone::parseDateTime(const String&, DateComponents::Type)
 }
 
 #if ENABLE(CALENDAR_PICKER)
-String LocaleNone::dateFormatText()
-{
-    return ASCIILiteral("Year-Month-Day");
-}
-
 bool LocaleNone::isRTL()
 {
     return false;
index c6ea9bc..efa45f8 100644 (file)
@@ -117,8 +117,6 @@ public:
     // The first day of a week. 0 is Sunday, and 6 is Saturday.
     virtual unsigned firstDayOfWeek() = 0;
 
-    virtual String dateFormatText() = 0;
-
     // Returns true if people use right-to-left writing in the locale for this
     // object.
     virtual bool isRTL() = 0;
index f934ef1..6fea931 100644 (file)
@@ -53,7 +53,6 @@ public:
     virtual double parseDateTime(const String&, DateComponents::Type) OVERRIDE;
 
 #if ENABLE(CALENDAR_PICKER)
-    virtual String dateFormatText() OVERRIDE;
     virtual const Vector<String>& weekDayShortLabels() OVERRIDE;
     virtual unsigned firstDayOfWeek() OVERRIDE;
     virtual bool isRTL() OVERRIDE;
@@ -81,7 +80,6 @@ private:
     RetainPtr<NSLocale> m_locale;
     RetainPtr<NSCalendar> m_gregorianCalendar;
 #if ENABLE(CALENDAR_PICKER)
-    String m_localizedDateFormatText;
     Vector<String> m_weekDayShortLabels;
 #endif
 #if ENABLE(CALENDAR_PICKER) || ENABLE(INPUT_MULTIPLE_FIELDS_UI)
index c95c27c..c06c005 100644 (file)
@@ -124,63 +124,6 @@ double LocaleMac::parseDateTime(const String& input, DateComponents::Type type)
     return [date timeIntervalSince1970] * msPerSecond;
 }
 
-#if ENABLE(CALENDAR_PICKER)
-static bool isYearSymbol(UChar letter) { return letter == 'y' || letter == 'Y' || letter == 'u'; }
-static bool isMonthSymbol(UChar letter) { return letter == 'M' || letter == 'L'; }
-static bool isDaySymbol(UChar letter) { return letter == 'd'; }
-
-// http://unicode.org/reports/tr35/tr35-6.html#Date_Format_Patterns
-static String localizeDateFormat(const String& format)
-{
-    String yearText = dateFormatYearText().isEmpty() ? "Year" : dateFormatYearText();
-    String monthText = dateFormatMonthText().isEmpty() ? "Month" : dateFormatMonthText();
-    String dayText = dateFormatDayInMonthText().isEmpty() ? "Day" : dateFormatDayInMonthText();
-    StringBuilder buffer;
-    bool inQuote = false;
-    for (unsigned i = 0; i < format.length(); ++i) {
-        UChar ch = format[i];
-        if (inQuote) {
-            if (ch == '\'') {
-                inQuote = false;
-                ASSERT(i);
-                if (format[i - 1] == '\'')
-                    buffer.append('\'');
-            } else
-                buffer.append(ch);
-            continue;
-        }
-
-        if (ch == '\'') {
-            inQuote = true;
-            if (i > 0 && format[i - 1] == '\'')
-                buffer.append(ch);
-        } else if (isYearSymbol(ch)) {
-            if (i > 0 && format[i - 1] == ch)
-                continue;
-            buffer.append(yearText);
-        } else if (isMonthSymbol(ch)) {
-            if (i > 0 && format[i - 1] == ch)
-                continue;
-            buffer.append(monthText);
-        } else if (isDaySymbol(ch)) {
-            if (i > 0 && format[i - 1] == ch)
-                continue;
-            buffer.append(dayText);
-        } else
-            buffer.append(ch);
-    }
-    return buffer.toString();
-}
-
-String LocaleMac::dateFormatText()
-{
-    if (!m_localizedDateFormatText.isNull())
-        return m_localizedDateFormatText;
-    m_localizedDateFormatText = localizeDateFormat([shortDateFormatter().get() dateFormat]);
-    return  m_localizedDateFormatText;
-}
-#endif
-
 #if ENABLE(CALENDAR_PICKER) || ENABLE(INPUT_MULTIPLE_FIELDS_UI)
 const Vector<String>& LocaleMac::monthLabels()
 {
index 66efc25..a2840f9 100644 (file)
@@ -450,48 +450,6 @@ void LocaleWin::ensureShortDateTokens()
     m_shortDateTokens = parseDateFormat(getLocaleInfoString(LOCALE_SSHORTDATE));
 }
 
-static String substituteLabelsIntoFormat(const Vector<DateFormatToken>& tokens, const String& yearText, const String& monthText, const String& dayText)
-{
-    String nonEmptyDayText = dayText.isEmpty() ? "Day" : dayText;
-    String nonEmptyMonthText = monthText.isEmpty() ? "Month" : monthText;
-    String nonEmptyYearText = yearText.isEmpty() ? "Year" : yearText;
-    StringBuilder buffer;
-    for (unsigned i = 0; i < tokens.size(); ++i) {
-        switch (tokens[i].type) {
-        case DateFormatToken::Literal:
-            buffer.append(tokens[i].data);
-            break;
-        case DateFormatToken::Day1:
-        case DateFormatToken::Day2:
-            buffer.append(nonEmptyDayText);
-            break;
-        case DateFormatToken::Month1:
-        case DateFormatToken::Month2:
-        case DateFormatToken::Month3:
-        case DateFormatToken::Month4:
-            buffer.append(nonEmptyMonthText);
-            break;
-        case DateFormatToken::Year1:
-        case DateFormatToken::Year2:
-        case DateFormatToken::Year4:
-            buffer.append(nonEmptyYearText);
-            break;
-        }
-    }
-    return buffer.toString();
-}
-
-String LocaleWin::dateFormatText()
-{
-    ensureShortDateTokens();
-    return substituteLabelsIntoFormat(m_shortDateTokens, dateFormatYearText(), dateFormatMonthText(), dateFormatDayInMonthText());
-}
-
-String LocaleWin::dateFormatText(const String& format, const String& yearText, const String& monthText, const String& dayText)
-{
-    return substituteLabelsIntoFormat(parseDateFormat(format), yearText, monthText, dayText);
-}
-
 void LocaleWin::ensureMonthLabels()
 {
     if (!m_monthLabels.isEmpty())
index 4671445..41f4f45 100644 (file)
@@ -48,7 +48,6 @@ public:
     ~LocaleWin();
     virtual double parseDateTime(const String&, DateComponents::Type) OVERRIDE;
 #if ENABLE(CALENDAR_PICKER)
-    virtual String dateFormatText() OVERRIDE;
     virtual const Vector<String>& weekDayShortLabels() OVERRIDE;
     virtual unsigned firstDayOfWeek() OVERRIDE;
     virtual bool isRTL() OVERRIDE;
@@ -69,8 +68,6 @@ public:
 
     // For testing.
     double parseDate(const String& format, int baseYear, const String& input);
-    String formatDate(const String& format, int baseYear, int year, int month, int day);
-    static String dateFormatText(const String& format, const String& yearText, const String& monthText, const String& dayText);
 #if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
     static String dateFormat(const String&);
 #endif
index c12828d..1b5ff3b 100644 (file)
@@ -1,5 +1,19 @@
 2012-10-30  Kent Tamura  <tkent@chromium.org>
 
+        Remove unused code for old input[type=date] UI
+        https://bugs.webkit.org/show_bug.cgi?id=100734
+
+        Reviewed by Hajime Morita.
+
+        * src/LocalizedStrings.cpp:
+        Follow the LocalizedStrings.h change.
+        * tests/LocaleMacTest.cpp:
+        (LocaleMacTest): Remove tests for Locale::dateFormatText.
+        * tests/LocaleWinTest.cpp:
+        (LocaleWinTest): Ditto.
+
+2012-10-30  Kent Tamura  <tkent@chromium.org>
+
         Fix crash by calendar picker or suggestion picker
         https://bugs.webkit.org/show_bug.cgi?id=100728
 
index f5477f3..ac8aae5 100644 (file)
@@ -496,31 +496,4 @@ String validationMessageStepMismatchText(const String& base, const String& step)
     return query(WebLocalizedString::ValidationStepMismatch, base, step);
 }
 
-#if ENABLE(CALENDAR_PICKER)
-String calendarTodayText()
-{
-    return query(WebLocalizedString::CalendarToday);
-}
-
-String calendarClearText()
-{
-    return query(WebLocalizedString::CalendarClear);
-}
-
-String dateFormatYearText()
-{
-    return query(WebLocalizedString::DateFormatYearLabel);
-}
-
-String dateFormatMonthText()
-{
-    return query(WebLocalizedString::DateFormatMonthLabel);
-}
-
-String dateFormatDayInMonthText()
-{
-    return query(WebLocalizedString::DateFormatDayInMonthLabel);
-}
-#endif
-
 } // namespace WebCore
index 17b2e91..0388976 100644 (file)
@@ -88,12 +88,6 @@ protected:
     }
 
 #if ENABLE(CALENDAR_PICKER)
-    String dateFormatText(const String& localeString)
-    {
-        OwnPtr<LocaleMac> locale = LocaleMac::create(localeString);
-        return locale->dateFormatText();
-    }
-
     unsigned firstDayOfWeek(const String& localeString)
     {
         OwnPtr<LocaleMac> locale = LocaleMac::create(localeString);
@@ -206,13 +200,6 @@ TEST_F(LocaleMacTest, parseDate)
 }
 
 #if ENABLE(CALENDAR_PICKER)
-TEST_F(LocaleMacTest, dateFormatText)
-{
-    EXPECT_STREQ("Month/Day/Year", dateFormatText("en_US").utf8().data());
-    EXPECT_STREQ("Day/Month/Year", dateFormatText("fr_FR").utf8().data());
-    EXPECT_STREQ("Year/Month/Day", dateFormatText("ja_JP").utf8().data());
-}
-
 TEST_F(LocaleMacTest, firstDayOfWeek)
 {
     EXPECT_EQ(Sunday, firstDayOfWeek("en_US"));
index 7b2dc9c..e2e03c1 100644 (file)
@@ -97,12 +97,6 @@ protected:
     }
 
 #if ENABLE(CALENDAR_PICKER)
-    String dateFormatText(LCID lcid)
-    {
-        OwnPtr<LocaleWin> locale = LocaleWin::create(lcid);
-        return locale->dateFormatText();
-    }
-
     unsigned firstDayOfWeek(LCID lcid)
     {
         OwnPtr<LocaleWin> locale = LocaleWin::create(lcid);
@@ -167,20 +161,6 @@ protected:
 #endif
 };
 
-TEST_F(LocaleWinTest, TestLocalizedDateFormatText)
-{
-    EXPECT_STREQ("year/month/day", LocaleWin::dateFormatText("y/M/d", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("year/month/day", LocaleWin::dateFormatText("yy/MM/dd", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("year/month/day", LocaleWin::dateFormatText("yyy/MMM/ddd", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("year/month/day", LocaleWin::dateFormatText("yyyy/MMMM/dddd", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("/month/day, year", LocaleWin::dateFormatText("/MM/dd, yyyy", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("month/day, year=year.", LocaleWin::dateFormatText("MM/dd, 'year='yyyy.", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("month-day 'year", LocaleWin::dateFormatText("MM-dd ''yyy", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("month-day aaa'bbb year", LocaleWin::dateFormatText("MM-dd 'aaa''bbb' yyy", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("year/month/day/year/month/day", LocaleWin::dateFormatText("yyyy/MMMM/dddd/yyyy/MMMM/dddd", "year", "month", "day").utf8().data());
-    EXPECT_STREQ("YY/mm/DD", LocaleWin::dateFormatText("YY/mm/DD", "year", "month", "day").utf8().data());
-}
-
 TEST_F(LocaleWinTest, TestParse)
 {
     OwnPtr<LocaleWin> locale = LocaleWin::create(EnglishUS);
@@ -235,13 +215,6 @@ TEST_F(LocaleWinTest, parseDate)
 }
 
 #if ENABLE(CALENDAR_PICKER)
-TEST_F(LocaleWinTest, dateFormatText)
-{
-    EXPECT_STREQ("Month/Day/Year", dateFormatText(EnglishUS).utf8().data());
-    EXPECT_STREQ("Day/Month/Year", dateFormatText(FrenchFR).utf8().data());
-    EXPECT_STREQ("Year/Month/Day", dateFormatText(JapaneseJP).utf8().data());
-}
-
 TEST_F(LocaleWinTest, firstDayOfWeek)
 {
     EXPECT_EQ(Sunday, firstDayOfWeek(EnglishUS));