RESOLVED FIXED 186378
[Win][MiniBrowser] Remove gMainWindow global variable 
https://bugs.webkit.org/show_bug.cgi?id=186378
Summary [Win][MiniBrowser] Remove gMainWindow global variable
Fujii Hironori
Reported 2018-06-06 19:27:17 PDT
[Win][MiniBrowser] Remove gMainWindow global variable
Attachments
Patch (9.45 KB, patch)
2018-06-06 19:47 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (9.52 KB, patch)
2018-06-06 23:19 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 2018-06-06 19:47:06 PDT
Created attachment 342107 [details] Patch
Ryosuke Niwa
Comment 2 2018-06-06 22:33:31 PDT
Comment on attachment 342107 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=342107&action=review > Tools/MiniBrowser/win/Common.cpp:139 > + int strLen = GetWindowText(GetDlgItem(hDlg, IDC_AUTH_USER), str, 256); Can we use WTF_ARRAY_LENGTH or sizeof(str) / sizeof(*str) or std::size instead of hard-coding 256 twice? > Tools/MiniBrowser/win/Common.cpp:143 > + strLen = GetWindowText(GetDlgItem(hDlg, IDC_AUTH_PASSWORD), str, 256); Ditto.
Fujii Hironori
Comment 3 2018-06-06 23:09:38 PDT
Thank you very much! I will fix them.
Fujii Hironori
Comment 4 2018-06-06 23:19:48 PDT
Created attachment 342124 [details] Patch
Fujii Hironori
Comment 5 2018-06-06 23:26:08 PDT
Committed r232574: <https://trac.webkit.org/changeset/232574>
Radar WebKit Bug Importer
Comment 6 2018-06-06 23:27:16 PDT
<rdar://problem/40882765>
Note You need to log in before you can comment on or make changes to this bug.