Changeset 56463 in webkit

Timestamp:

Mar 24, 2010 3:45:03 PM (14 years ago)

Author:

eric@webkit.org

Message:

2010-03-24 MORITA Hajime <​morrita@google.com>

Reviewed by Darin Adler.

Refactoring: HTMLFormControlElement should not have redundant null check
​https://bugs.webkit.org/show_bug.cgi?id=36487

No new tests as there is no new functionality.

• html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::dispatchFocusEvent): (WebCore::HTMLFormControlElement::dispatchBlurEvent):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTMLFormControlElement.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-03-24  MORITA Hajime  <morrita@google.com>
+
+        Reviewed by Darin Adler.
+
+        Refactoring: HTMLFormControlElement should not have redundant null check
+        https://bugs.webkit.org/show_bug.cgi?id=36487
+        
+        No new tests as there is no new functionality.        
+
+        * html/HTMLFormControlElement.cpp:
+        (WebCore::HTMLFormControlElement::dispatchFocusEvent):
+        (WebCore::HTMLFormControlElement::dispatchBlurEvent):
+
 2010-03-24  Eric Uhrhane  <ericu@chromium.org>
 

trunk/WebCore/html/HTMLFormControlElement.cpp

@@ -368 +368 @@
 void HTMLFormControlElement::dispatchFocusEvent()
 {
-    if (document()->frame() && document()->frame()->page())
-        document()->frame()->page()->chrome()->client()->formDidFocus(this);
+    if (document()->page())
+        document()->page()->chrome()->client()->formDidFocus(this);
 
     HTMLElement::dispatchFocusEvent();
@@ -376 +376 @@
 void HTMLFormControlElement::dispatchBlurEvent()
 {
-    if (document()->frame() && document()->frame()->page())
-        document()->frame()->page()->chrome()->client()->formDidBlur(this);
+    if (document()->page())
+        document()->page()->chrome()->client()->formDidBlur(this);
 
     HTMLElement::dispatchBlurEvent();