Bug 253123 - Small cleanup on BMPImageReader.cpp
Summary: Small cleanup on BMPImageReader.cpp
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-02-28 16:35 PST by Ahmad Saleem
Modified: 2023-07-03 00:09 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-02-28 16:35:54 PST 
Hi Team,

While going through Blink's commit, I came across this clean-up.

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/a057769943417c61acd574458034ac6546ed6f6a

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/platform/image-decoders/bmp/BMPImageReader.cpp#62

I know we don't use these on macOS platform but still might be useful for other platforms.

Just wanted to raise it. Happy to do PR as well.

Thanks!
Comment 1 Radar WebKit Bug Importer 2023-03-07 16:36:19 PST 
<rdar://problem/106387835>
Comment 2 EWS 2023-07-03 00:09:55 PDT 
Committed 265704@main (d156132fce2f): <https://commits.webkit.org/265704@main>

Reviewed commits have been landed. Closing PR #15466 and removing active labels.