Changeset 159436 in webkit

Timestamp:

Nov 18, 2013 10:46:11 AM (10 years ago)

Author:

commit-queue@webkit.org

Message:

[Win] Optimization, no need to erase background before paint.
​https://bugs.webkit.org/show_bug.cgi?id=124453

Patch by ​peavo@outlook.com <​peavo@outlook.com> on 2013-11-18
Reviewed by Brent Fulgham.

As the paint will blit a bitmap onto the invalid area (no alpha),
there is no need to erase the background first, as this will be overwritten by the blit.

• WebView.cpp:

(WebView::WebViewWndProc): Avoid erasing background before paint.

Location:

trunk/Source/WebKit/win

Files:

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

trunk/Source/WebKit/win/ChangeLog

@@ +1 @@
+2013-11-18  peavo@outlook.com  <peavo@outlook.com>
+
+        [Win] Optimization, no need to erase background before paint.
+        https://bugs.webkit.org/show_bug.cgi?id=124453
+
+        Reviewed by Brent Fulgham.
+
+        As the paint will blit a bitmap onto the invalid area (no alpha),
+        there is no need to erase the background first, as this will be overwritten by the blit.
+
+        * WebView.cpp:
+        (WebView::WebViewWndProc): Avoid erasing background before paint.
+
 2013-11-18  peavo@outlook.com  <peavo@outlook.com>
 

trunk/Source/WebKit/win/WebView.cpp

@@ -2230 +2230 @@
         }
         case WM_ERASEBKGND:
-            if (webView->usesLayeredWindow()) {
-                // Don't perform a background erase for transparent views.
-                handled = true;
-                lResult = 1;
-            }
+            // Don't perform a background erase.
+            handled = true;
+            lResult = 1;
             break;
         case WM_PRINTCLIENT: