[webkit-changes] [109123] trunk/Source/WebKit/blackberry

Tue, 28 Feb 2012 10:51:51 -0800

Title: [109123] trunk/Source/WebKit/blackberry
Revision
109123
Author
[email protected]
Date
2012-02-28 10:51:47 -0800 (Tue, 28 Feb 2012)

Log Message

[BlackBerry] Remove unused focusedNodeChange notifications from the Chrome client.
https://bugs.webkit.org/show_bug.cgi?id=79794

Patch by Mike Fenton <[email protected]> on 2012-02-28
Reviewed by Rob Buis.

Remove unused notification for focusedNodeChanged.

Internally reviewed by Gen Mak.

* Api/WebPageClient.h:
* WebCoreSupport/ChromeClientBlackBerry.cpp:
(WebCore::ChromeClientBlackBerry::focusedNodeChanged):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/Api/WebPageClient.h (109122 => 109123)


--- trunk/Source/WebKit/blackberry/Api/WebPageClient.h	2012-02-28 18:48:02 UTC (rev 109122)
+++ trunk/Source/WebKit/blackberry/Api/WebPageClient.h	2012-02-28 18:51:47 UTC (rev 109123)
@@ -66,42 +66,6 @@
         FlagWindowDefault = 0xFFFFFFFF,
     };
 
-    enum FocusType {
-        FocusUnknown = 0,
-        FocusNone,
-        FocusCanvas,
-        FocusImage,
-        FocusInputButton,
-        FocusInputCheckBox,
-        FocusInputColor,
-        FocusInputDate,
-        FocusInputDateTime,
-        FocusInputDateTimeLocal,
-        FocusInputEmail,
-        FocusInputFile,
-        FocusInputImage,
-        FocusInputMonth,
-        FocusInputNumber,
-        FocusInputPassword,
-        FocusInputRadio,
-        FocusInputRange,
-        FocusInputReset,
-        FocusInputSearch,
-        FocusInputSubmit,
-        FocusInputTelephone,
-        FocusInputText,
-        FocusInputTime,
-        FocusInputURL,
-        FocusInputWeek,
-        FocusInputUnknown,
-        FocusLink,
-        FocusObject,
-        FocusSelect,
-        FocusSVGElement,
-        FocusTextArea,
-        FocusVideo,
-    };
-
     enum AlertType {
         MediaOK = 0,
         MediaDecodeError,
@@ -165,7 +129,6 @@
     virtual void drawTapHighlight(const Platform::IntRectRegion&, int red, int green, int blue, int alpha, bool hideAfterScroll) = 0;
     virtual void hideTapHighlight() = 0;
 
-    virtual void focusChanged(FocusType, int elementId) = 0;
     virtual void inputFocusGained(Platform::BlackBerryInputType, int inputStyle, bool waitForExplicitKeyboardShowCall) = 0;
     virtual void inputFocusLost() = 0;
     virtual void inputTextChanged() = 0;

Modified: trunk/Source/WebKit/blackberry/ChangeLog (109122 => 109123)


--- trunk/Source/WebKit/blackberry/ChangeLog	2012-02-28 18:48:02 UTC (rev 109122)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2012-02-28 18:51:47 UTC (rev 109123)
@@ -1,3 +1,18 @@
+2012-02-28  Mike Fenton  <[email protected]>
+
+        [BlackBerry] Remove unused focusedNodeChange notifications from the Chrome client.
+        https://bugs.webkit.org/show_bug.cgi?id=79794
+
+        Reviewed by Rob Buis.
+
+        Remove unused notification for focusedNodeChanged.
+
+        Internally reviewed by Gen Mak.
+
+        * Api/WebPageClient.h:
+        * WebCoreSupport/ChromeClientBlackBerry.cpp:
+        (WebCore::ChromeClientBlackBerry::focusedNodeChanged):
+
 2012-02-27  Jacky Jiang  <[email protected]>
 
         [BlackBerry] Upstream BlackBerry API web page related files

Modified: trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp (109122 => 109123)


--- trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp	2012-02-28 18:48:02 UTC (rev 109122)
+++ trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp	2012-02-28 18:51:47 UTC (rev 109123)
@@ -190,83 +190,7 @@
 
 void ChromeClientBlackBerry::focusedNodeChanged(Node* node)
 {
-    WebPageClient::FocusType type = WebPageClient::FocusUnknown;
-
-    if (!node)
-        type = WebPageClient::FocusNone;
-    else if (node->isSVGElement())
-        type = WebPageClient::FocusSVGElement;
-    else if (node->isHTMLElement()) {
-        if (node->hasTagName(HTMLNames::aTag)) {
-            type = WebPageClient::FocusLink;
-        } else if (node->hasTagName(HTMLNames::inputTag)) {
-            HTMLInputElement* inputElement = static_cast<HTMLInputElement*>(node);
-            if (inputElement->hasTagName(HTMLNames::isindexTag) || inputElement->isInputTypeHidden())
-                type = WebPageClient::FocusInputUnknown;
-            else if (inputElement->isTextButton())
-                type = WebPageClient::FocusInputButton;
-            else if (inputElement->isCheckbox())
-                type = WebPageClient::FocusInputCheckBox;
-#if ENABLE(INPUT_COLOR)
-            else if (inputElement->isColorControl())
-                type = WebPageClient::FocusInputColor;
-#endif
-            else if (inputElement->isDateControl())
-                type = WebPageClient::FocusInputDate;
-            else if (inputElement->isDateTimeControl())
-                type = WebPageClient::FocusInputDateTime;
-            else if (inputElement->isDateTimeLocalControl())
-                type = WebPageClient::FocusInputDateTimeLocal;
-            else if (inputElement->isEmailField())
-                type = WebPageClient::FocusInputEmail;
-            else if (inputElement->isFileUpload())
-                type = WebPageClient::FocusInputFile;
-            else if (inputElement->isImageButton())
-                type = WebPageClient::FocusInputImage;
-            else if (inputElement->isMonthControl())
-                type = WebPageClient::FocusInputMonth;
-            else if (inputElement->isNumberField())
-                type = WebPageClient::FocusInputNumber;
-            else if (inputElement->isPasswordField())
-                type = WebPageClient::FocusInputPassword;
-            else if (inputElement->isRadioButton())
-                type = WebPageClient::FocusInputRadio;
-            else if (inputElement->isRangeControl())
-                type = WebPageClient::FocusInputRange;
-            else if (inputElement->isResetButton())
-                type = WebPageClient::FocusInputReset;
-            else if (inputElement->isSearchField())
-                type = WebPageClient::FocusInputSearch;
-            else if (inputElement->isSubmitButton())
-                type = WebPageClient::FocusInputSubmit;
-            else if (inputElement->isTelephoneField())
-                type = WebPageClient::FocusInputTelephone;
-            else if (inputElement->isURLField())
-                type = WebPageClient::FocusInputURL;
-            else if (inputElement->isTextField())
-                type = WebPageClient::FocusInputText;
-            else if (inputElement->isTimeControl())
-                type = WebPageClient::FocusInputTime;
-            // FIXME: missing WEEK popup selector
-            else
-                type = WebPageClient::FocusInputUnknown;
-        } else if (node->hasTagName(HTMLNames::imgTag))
-            type = WebPageClient::FocusImage;
-        else if (node->hasTagName(HTMLNames::objectTag))
-            type = WebPageClient::FocusObject;
-        else if (node->hasTagName(HTMLNames::videoTag))
-            type = WebPageClient::FocusVideo;
-        else if (node->hasTagName(HTMLNames::selectTag))
-            type = WebPageClient::FocusSelect;
-        else if (node->hasTagName(HTMLNames::textareaTag))
-            type = WebPageClient::FocusTextArea;
-        else if (node->hasTagName(HTMLNames::canvasTag))
-            type = WebPageClient::FocusCanvas;
-    }
-
     m_webPagePrivate->m_inputHandler->nodeFocused(node);
-
-    m_webPagePrivate->m_client->focusChanged(type, reinterpret_cast<int>(node));
 }
 
 void ChromeClientBlackBerry::focusedFrameChanged(Frame*)

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to