Changeset 119134 in webkit

Timestamp:

May 31, 2012 12:30:33 PM (12 years ago)

Author:

staikos@webkit.org

Message:

[Blackberry] Initialize the select client and delete the pointer
in the destructor so it doesn't leak. Fixes test crashes.
​https://bugs.webkit.org/show_bug.cgi?id=87992

Reviewed by Rob Buis.

• WebKitSupport/InputHandler.cpp:

(BlackBerry::WebKit::InputHandler::InputHandler):
(BlackBerry::WebKit::InputHandler::~InputHandler):

Location:

trunk/Source/WebKit/blackberry

Files:

• ChangeLog (modified) (1 diff)
• WebKitSupport/InputHandler.cpp (modified) (2 diffs)

trunk/Source/WebKit/blackberry/ChangeLog

@@ +1 @@
+2012-05-31  George Staikos  <staikos@webkit.org>
+
+        [Blackberry] Initialize the select client and delete the pointer
+        in the destructor so it doesn't leak.  Fixes test crashes.
+        https://bugs.webkit.org/show_bug.cgi?id=87992
+
+        Reviewed by Rob Buis.
+
+        * WebKitSupport/InputHandler.cpp:
+        (BlackBerry::WebKit::InputHandler::InputHandler):
+        (BlackBerry::WebKit::InputHandler::~InputHandler):
+
 2012-05-31  Arvid Nilsson  <anilsson@rim.com>
 

trunk/Source/WebKit/blackberry/WebKitSupport/InputHandler.cpp

@@ -121 +121 @@
     , m_pendingKeyboardVisibilityChange(NoChange)
     , m_delayKeyboardVisibilityChange(false)
+    , m_selectClient(0)
 {
 }
@@ -126 +127 @@
 InputHandler::~InputHandler()
 {
+    delete m_selectClient;
 }