Bug 52009 – WKView on Windows has no equivalent of Mac's -[WKView setDraws[Transparent]Background:]

• Component: WebKit2
• Status: RESOLVED FIXED

[Adam Roben (:aroben)]

WKView on Mac has the following APIs:

-[WKView setDrawsBackground:]
-[WKView setDrawsTransparentBackground:]

These allow API clients to cause the WKView to be transparent or to have some other custom background. We have no equivalent of this on Windows (but don't need it, at the moment, either).

[Adam Roben (:aroben)]

<rdar://problem/8829746>

[Anders Carlsson]

Created attachment 92319 [details]
Patch

[Adam Roben (:aroben)]

Comment on attachment 92319 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=92319&action=review

> Source/WebKit2/UIProcess/win/WebView.cpp:642
> -static void drawPageBackground(HDC dc, const RECT& rect)
> +static void drawPageBackground(HDC dc, const WebPageProxy* page, const RECT& rect)
>  {
> -    // Mac checks WebPageProxy::drawsBackground and
> -    // WebPageProxy::drawsTransparentBackground here, but those are always false on
> -    // Windows currently (see <http://webkit.org/b/52009>).
> +    if (!page->drawsBackground() || page->drawsTransparentBackground())
> +        return;
> +
>      ::FillRect(dc, &rect, reinterpret_cast<HBRUSH>(COLOR_WINDOW + 1));
>  }

Failing to paint parts of the HWND will lead to pixel smearing when Aero is disabled. That doesn't seem good!

[Adam Roben (:aroben)]

Comment on attachment 92319 [details]
Patch

I guess this will perhaps someday be useful if we add a way to make a WS_EX_LAYERED WKView. Maybe we'll find other uses for it before then.

[Anders Carlsson]

Committed r85794: <http://trac.webkit.org/changeset/85794>