RESOLVED FIXED 186387
[Win][MiniBrowser] MiniBrowser::updateDeviceScaleFactor should be a MainWindow's method 
https://bugs.webkit.org/show_bug.cgi?id=186387
Summary [Win][MiniBrowser] MiniBrowser::updateDeviceScaleFactor should be a MainWindo...
Fujii Hironori
Reported 2018-06-07 01:20:33 PDT
[Win][MiniBrowser] Convert MiniBrowser::updateDeviceScaleFactor to a MainWindow's method MiniBrowser::updateDeviceScaleFactor does nothing for MiniBrowser. It should be a MainWindow's method.
Attachments
Patch (6.59 KB, patch)
2018-06-07 01:26 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (7.07 KB, patch)
2018-06-07 19:13 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 2018-06-07 01:26:46 PDT
Created attachment 342139 [details] Patch
Ryosuke Niwa
Comment 2 2018-06-07 13:15:14 PDT
Comment on attachment 342139 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=342139&action=review > Tools/MiniBrowser/win/MainWindow.cpp:426 > + m_hURLBarFont = ::CreateFont(scaleFactor * 18, 0, 0, 0, FW_NORMAL, FALSE, FALSE, FALSE, DEFAULT_CHARSET, Where did 18 come from!?
Fujii Hironori
Comment 3 2018-06-07 18:54:23 PDT
Comment on attachment 342139 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=342139&action=review Thank you very much for the review. >> Tools/MiniBrowser/win/MainWindow.cpp:426 >> + m_hURLBarFont = ::CreateFont(scaleFactor * 18, 0, 0, 0, FW_NORMAL, FALSE, FALSE, FALSE, DEFAULT_CHARSET, > > Where did 18 come from!? Good question! urlBarHeight is 24. It should be urlBarHeight * 3.0 / 4.
Fujii Hironori
Comment 4 2018-06-07 19:13:21 PDT
Created attachment 342230 [details] Patch
Fujii Hironori
Comment 5 2018-06-07 19:20:00 PDT
Committed r232609: <https://trac.webkit.org/changeset/232609>
Radar WebKit Bug Importer
Comment 6 2018-06-07 19:20:21 PDT
<rdar://problem/40920937>
Note You need to log in before you can comment on or make changes to this bug.