Datalist UI (WebKit part)

Source/WebKit/chromium/src/AutofillPopupMenuClient.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 34 matching lines @@
 #include "WebViewClient.h"
 #include "WebViewImpl.h"
 #include "platform/WebString.h"
 #include "platform/WebVector.h"
 
 using namespace WebCore;
 
 namespace WebKit {
 
 AutofillPopupMenuClient::AutofillPopupMenuClient()
-    : m_separatorIndex(-1)
-    , m_selectedIndex(-1)

[Ilya Sherman, 2012/04/11 00:27:52]

nit: We should still initialize m_selectedIndex.

[keishi, 2012/04/11 12:13:12]

Done.

-    , m_textField(0)
+    : m_textField(0)
 {
 }
 
 AutofillPopupMenuClient::~AutofillPopupMenuClient()
 {
 }
 
 unsigned AutofillPopupMenuClient::getSuggestionsCount() const
 {
-    return m_names.size() + ((m_separatorIndex == -1) ? 0 : 1);
+    return m_names.size();
 }
 
 WebString AutofillPopupMenuClient::getSuggestion(unsigned listIndex) const
 {
-    int index = convertListIndexToInternalIndex(listIndex);
-    if (index == -1)
-        return WebString();
-
-    ASSERT(index >= 0 && static_cast<size_t>(index) < m_names.size());
-    return m_names[index];
+    ASSERT(listIndex >= 0 && static_cast<size_t>(listIndex) < m_names.size());

[Ilya Sherman, 2012/04/11 00:27:52]

nit: No need to check nonnegativity nor to cast -- the listIndex is already
unsigned.

[keishi, 2012/04/11 12:13:12]

Done.

+    return m_names[listIndex];
 }
 
 WebString AutofillPopupMenuClient::getLabel(unsigned listIndex) const
 {
-    int index = convertListIndexToInternalIndex(listIndex);
-    if (index == -1)
-        return WebString();
-
-    ASSERT(index >= 0 && static_cast<size_t>(index) < m_labels.size());
-    return m_labels[index];
+    ASSERT(listIndex >= 0 && static_cast<size_t>(listIndex) < m_labels.size());

[Ilya Sherman, 2012/04/11 00:27:52]

nit: Ditto

[keishi, 2012/04/11 12:13:12]

Done.

+    return m_labels[listIndex];
 }
 
 WebString AutofillPopupMenuClient::getIcon(unsigned listIndex) const
 {
-    int index = convertListIndexToInternalIndex(listIndex);
-    if (index == -1)
-        return WebString();
-
-    ASSERT(index >= 0 && static_cast<size_t>(index) < m_icons.size());
-    return m_icons[index];
+    ASSERT(listIndex >= 0 && static_cast<size_t>(listIndex) < m_icons.size());

[Ilya Sherman, 2012/04/11 00:27:52]

nit: Ditto

[keishi, 2012/04/11 12:13:12]

Done.

+    return m_icons[listIndex];
 }
 
 void AutofillPopupMenuClient::removeSuggestionAtIndex(unsigned listIndex)
 {
     if (!canRemoveSuggestionAtIndex(listIndex))
         return;
 
-    int index = convertListIndexToInternalIndex(listIndex);
+    ASSERT(static_cast<unsigned>(listIndex) < m_names.size());

[Ilya Sherman, 2012/04/11 00:27:52]

nit: No need to cast, the listIndex is already unsigned.

[keishi, 2012/04/11 12:13:12]

Done.

 
-    ASSERT(static_cast<unsigned>(index) < m_names.size());
-
-    m_names.remove(index);
-    m_labels.remove(index);
-    m_icons.remove(index);
-    m_uniqueIDs.remove(index);
-
-    // Shift the separator index if necessary.
-    if (m_separatorIndex != -1)
-        m_separatorIndex--;
+    m_names.remove(listIndex);
+    m_labels.remove(listIndex);
+    m_icons.remove(listIndex);
+    m_itemIDs.remove(listIndex);
 }
 
 bool AutofillPopupMenuClient::canRemoveSuggestionAtIndex(unsigned listIndex)
 {
-    // Only allow deletion of items before the separator that have unique id 0
-    // (i.e. are autocomplete rather than autofill items).
-    int index = convertListIndexToInternalIndex(listIndex);
-    return !m_uniqueIDs[index] && (m_separatorIndex == -1 || listIndex < static_cast<unsigned>(m_separatorIndex));
+    return m_itemIDs[listIndex] > 0 || // Is autofill entry?

[Ilya Sherman, 2012/04/11 00:27:52]

It's not currently possible to remove Autofill entries.  I'm also not sure about
password suggestions.

[keishi, 2012/04/11 12:13:12]

Done. I'll match the current behavior.

+           m_itemIDs[listIndex] == AutocompleteEntryMenuItemID ||
+           m_itemIDs[listIndex] == PasswordEntryMenuItemID;
 }
 
 void AutofillPopupMenuClient::valueChanged(unsigned listIndex, bool fireEvents)
 {
     WebViewImpl* webView = getWebView();
     if (!webView)
         return;
 
-    if (m_separatorIndex != -1 && listIndex > static_cast<unsigned>(m_separatorIndex))
-        --listIndex;
-
     ASSERT(listIndex < m_names.size());
 
     webView->autofillClient()->didAcceptAutofillSuggestion(WebNode(getTextField()),
                                                            m_names[listIndex],
                                                            m_labels[listIndex],
-                                                           m_uniqueIDs[listIndex],
+                                                           m_itemIDs[listIndex],
                                                            listIndex);
 }
 
 void AutofillPopupMenuClient::selectionChanged(unsigned listIndex, bool fireEvents)
 {
     WebViewImpl* webView = getWebView();
     if (!webView)
         return;
 
-    if (m_separatorIndex != -1 && listIndex > static_cast<unsigned>(m_separatorIndex))
-        --listIndex;
-
     ASSERT(listIndex < m_names.size());
 
     webView->autofillClient()->didSelectAutofillSuggestion(WebNode(getTextField()),
                                                            m_names[listIndex],
                                                            m_labels[listIndex],
-                                                           m_uniqueIDs[listIndex]);
+                                                           m_itemIDs[listIndex]);
 }
 
 void AutofillPopupMenuClient::selectionCleared()
 {
     WebViewImpl* webView = getWebView();
     if (webView)
         webView->autofillClient()->didClearAutofillSelection(WebNode(getTextField()));
 }
 
 String AutofillPopupMenuClient::itemText(unsigned listIndex) const
@@ skipping 51 matching lines @@
     WebViewImpl* webView = getWebView();
     if (!webView)
         return;
 
     webView->autofillPopupDidHide();
     webView->autofillClient()->didClearAutofillSelection(WebNode(getTextField()));
 }
 
 bool AutofillPopupMenuClient::itemIsSeparator(unsigned listIndex) const
 {
-    return (m_separatorIndex != -1 && static_cast<unsigned>(m_separatorIndex) == listIndex);
+    return m_itemIDs[listIndex] == SeparatorMenuItemID;
 }
 
 bool AutofillPopupMenuClient::itemIsWarning(unsigned listIndex) const
 {
-    int index = convertListIndexToInternalIndex(listIndex);
-    if (index == -1)
-        return false;
-
-    ASSERT(index >= 0 && static_cast<size_t>(index) < m_uniqueIDs.size());
-    return m_uniqueIDs[index] < 0;
+    return m_itemIDs[listIndex] == WarningMessageMenuItemID;
 }
 
 void AutofillPopupMenuClient::setTextFromItem(unsigned listIndex)
 {
     m_textField->setValue(getSuggestion(listIndex));
 }
 
 FontSelector* AutofillPopupMenuClient::fontSelector() const
 {
     return m_textField->document()->styleSelector()->fontSelector();
@@ skipping 10 matching lines @@
     ScrollbarControlSize size)
 {
     return Scrollbar::createNativeScrollbar(scrollableArea, orientation, size);
 }
 
 void AutofillPopupMenuClient::initialize(
     HTMLInputElement* textField,
     const WebVector<WebString>& names,
     const WebVector<WebString>& labels,
     const WebVector<WebString>& icons,
-    const WebVector<int>& uniqueIDs,
-    int separatorIndex)
+    const WebVector<int>& itemIDs)
 {
     ASSERT(names.size() == labels.size());
     ASSERT(names.size() == icons.size());
-    ASSERT(names.size() == uniqueIDs.size());
-    ASSERT(separatorIndex < static_cast<int>(names.size()));
+    ASSERT(names.size() == itemIDs.size());
 
     m_selectedIndex = -1;
     m_textField = textField;
 
     // The suggestions must be set before initializing the
     // AutofillPopupMenuClient.
-    setSuggestions(names, labels, icons, uniqueIDs, separatorIndex);
+    setSuggestions(names, labels, icons, itemIDs);
 
     FontDescription regularFontDescription;
     RenderTheme::defaultTheme()->systemFont(CSSValueWebkitControl,
                                             regularFontDescription);
     RenderStyle* style = m_textField->computedStyle();
     regularFontDescription.setComputedSize(style->fontDescription().computedSize());
 
     Font regularFont(regularFontDescription, 0, 0);
     regularFont.update(textField->document()->styleSelector()->fontSelector());
     // The direction of text in popup menu is set the same as the direction of
@@ skipping 10 matching lines @@
     m_warningStyle = adoptPtr(new PopupMenuStyle(Color::darkGray, m_regularStyle->backgroundColor(), warningFont,
                                                  m_regularStyle->isVisible(), m_regularStyle->isDisplayNone(),
                                                  m_regularStyle->textIndent(), m_regularStyle->textDirection(),
                                                  m_regularStyle->hasTextDirectionOverride(),
                                                  PopupMenuStyle::AutofillPopup));
 }
 
 void AutofillPopupMenuClient::setSuggestions(const WebVector<WebString>& names,
                                              const WebVector<WebString>& labels,
                                              const WebVector<WebString>& icons,
-                                             const WebVector<int>& uniqueIDs,
-                                             int separatorIndex)
+                                             const WebVector<int>& itemIDs)
 {
     ASSERT(names.size() == labels.size());
     ASSERT(names.size() == icons.size());
-    ASSERT(names.size() == uniqueIDs.size());
-    ASSERT(separatorIndex < static_cast<int>(names.size()));
+    ASSERT(names.size() == itemIDs.size());
 
     m_names.clear();
     m_labels.clear();
     m_icons.clear();
-    m_uniqueIDs.clear();
+    m_itemIDs.clear();
     for (size_t i = 0; i < names.size(); ++i) {
         m_names.append(names[i]);
         m_labels.append(labels[i]);
         m_icons.append(icons[i]);
-        m_uniqueIDs.append(uniqueIDs[i]);
+        m_itemIDs.append(itemIDs[i]);
     }
 
-    m_separatorIndex = separatorIndex;
-
     // Try to preserve selection if possible.
     if (getSelectedIndex() >= static_cast<int>(names.size()))
         setSelectedIndex(-1);
 }
 
-int AutofillPopupMenuClient::convertListIndexToInternalIndex(unsigned listIndex) const
-{
-    if (listIndex == static_cast<unsigned>(m_separatorIndex))
-        return -1;
-
-    if (m_separatorIndex == -1 || listIndex < static_cast<unsigned>(m_separatorIndex))
-        return listIndex;
-    return listIndex - 1;
-}
-
 WebViewImpl* AutofillPopupMenuClient::getWebView() const
 {
     Frame* frame = m_textField->document()->frame();
     if (!frame)
         return 0;
 
     Page* page = frame->page();
     if (!page)
         return 0;
 
     return static_cast<WebViewImpl*>(page->chrome()->client()->webView());
 }
 
 RenderStyle* AutofillPopupMenuClient::textFieldStyle() const
 {
     RenderStyle* style = m_textField->computedStyle();
     if (!style) {
         // It seems we can only have a 0 style in a TextField if the
         // node is detached, in which case we the popup should not be
         // showing.  Please report this in http://crbug.com/7708 and
         // include the page you were visiting.
         ASSERT_NOT_REACHED();
     }
     return style;
 }
 
 } // namespace WebKit