Bug 196391

Summary:

[Win][WebKit] MiniBrowser should support high DPI display

Product:

WebKit

Reporter:

Fujii Hironori <Hironori.Fujii>

Component:

Tools / Tests

Assignee:

Fujii Hironori <Hironori.Fujii>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, bfulgham, don.olmstead, ews-watchlist, lforschler, pvollan, ross.kirsling, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none

Description Fujii Hironori 2019-03-28 23:45:11 PDT

[Win][WebKit] MiniBrowser should support high DPI display

MiniBrowser shows tiny texts in high DPI display at the moment.

In WebKitLegacy Windows port and WebKit other ports, high DPI is
supported by WebView, not by applications. I tried this approach
in Bug 196339. It seems it makes thinks complicated for Windows
because Windows window coordinated system differs from other
ports.

In contrast, it seems easy to implement in the application side just
by changing the default page zoom factor.

Comment 1 Fujii Hironori 2019-03-28 23:58:36 PDT

Created attachment 366253 [details]
Patch

Comment 2 EWS Watchlist 2019-03-29 00:01:32 PDT Comment hidden (obsolete)

Attachment 366253 [details] did not pass style-queue:


ERROR: Tools/MiniBrowser/win/WebKitBrowserWindow.cpp:40:  Use the class HWndDC instead of calling GetDC to avoid potential memory leaks.  [runtime/leaky_pattern] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

Comment 3 Fujii Hironori 2019-03-29 00:10:55 PDT

Created attachment 366255 [details]
Patch

Comment 4 Fujii Hironori 2019-03-31 18:38:36 PDT

Comment on attachment 366255 [details]
Patch

Clearing flags on attachment: 366255

Committed r243685: <https://trac.webkit.org/changeset/243685>

Comment 5 Fujii Hironori 2019-03-31 18:38:39 PDT

All reviewed patches have been landed.  Closing bug.

Comment 6 Radar WebKit Bug Importer 2019-03-31 18:39:23 PDT

<rdar://problem/49464798>