Changeset 52827 in webkit

Timestamp:

Jan 5, 2010 3:06:57 PM (14 years ago)

Author:

Adam Roben

Message:

Remove dead code in WebViewWndProc

Fixes <​http://webkit.org/b/33218>.

Reviewed by Darin Adler.

• WebView.cpp:

(WebView::WebViewWndProc): Removed a redundant isBeingDestroyed check.
We bail out much earlier in this function if isBeingDestroyed is true.

Location:

trunk/WebKit/win

Files:

• ChangeLog (modified) (1 diff)
• WebView.cpp (modified) (1 diff)

trunk/WebKit/win/ChangeLog

@@ +1 @@
+2010-01-05  Adam Roben  <aroben@apple.com>
+
+        Remove dead code in WebViewWndProc
+
+        Fixes <http://webkit.org/b/33218>.
+
+        Reviewed by Darin Adler.
+
+        * WebView.cpp:
+        (WebView::WebViewWndProc): Removed a redundant isBeingDestroyed check.
+        We bail out much earlier in this function if isBeingDestroyed is true.
+
 2010-01-04  Jon Honeycutt  <jhoneycutt@apple.com>
 

trunk/WebKit/win/WebView.cpp

@@ -1955 +1955 @@
         // FIXME: We need to check WM_UNICHAR to support supplementary characters (that don't fit in 16 bits).
         case WM_SIZE:
-            if (webView->isBeingDestroyed())
-                // If someone has sent us this message while we're being destroyed, we should bail out so we don't crash.
-                break;
-
             if (lParam != 0) {
                 webView->deleteBackingStore();