NEW236222
[WTF] Retrieve the main thread ID from TEB in Windows 
https://bugs.webkit.org/show_bug.cgi?id=236222
Summary [WTF] Retrieve the main thread ID from TEB in Windows
Yusuke Suzuki
Reported 2022-02-07 01:42:24 PST
[WTF] Retrieve the main thread ID from TEB in Windows
Attachments
Patch (2.30 KB, patch)
2022-02-07 01:42 PST, Yusuke Suzuki 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2022-02-07 01:42:49 PST
Created attachment 451070 [details] Patch
Alexey Proskuryakov
Comment 2 2022-02-07 12:31:50 PST
Comment on attachment 451070 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=451070&action=review > Source/WTF/wtf/win/MainThreadWin.cpp:42 > + // https://en.wikipedia.org/wiki/Win32_Thread_Information_Block Makes me wonder if it is teb or tib.
Yusuke Suzuki
Comment 3 2022-02-07 23:40:08 PST
Comment on attachment 451070 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=451070&action=review >> Source/WTF/wtf/win/MainThreadWin.cpp:42 >> + // https://en.wikipedia.org/wiki/Win32_Thread_Information_Block > > Makes me wonder if it is teb or tib. It is referring to the same thing, but TIB in Windows 9x and TEB in Windows NT~. So, TEB :)
Yusuke Suzuki
Comment 4 2022-02-07 23:40:52 PST
Comment on attachment 451070 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=451070&action=review >>> Source/WTF/wtf/win/MainThreadWin.cpp:42 >>> + // https://en.wikipedia.org/wiki/Win32_Thread_Information_Block >> >> Makes me wonder if it is teb or tib. > > It is referring to the same thing, but TIB in Windows 9x and TEB in Windows NT~. > So, TEB :) Ah, precisely, TEB for Win32.
Radar WebKit Bug Importer
Comment 5 2022-02-14 01:43:18 PST
<rdar://problem/88894012>
Brent Fulgham
Comment 6 2023-02-15 14:50:30 PST
@Yusuke: Is this ready to land? Weird that it never committed when everything was green.
Brent Fulgham
Comment 7 2023-02-15 14:51:07 PST
Note: We only really care about Win10+, since older Windows aren't really supported by MS anymore.
Note You need to log in before you can comment on or make changes to this bug.