Bug 214537

Summary:	AccessibilityTable::isDataTable() compares the Color::alphaByte() value to 1, but probably wants to be comparing it to 255
Product:	WebKit	Reporter:	Sam Weinig <sam>
Component:	Accessibility	Assignee:	Nobody <webkit-unassigned>
Status:	NEW ---
Severity:	Normal	CC:	cfleizach, jcraig, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Sam Weinig 2020-07-19 11:14:22 PDT

While doing some refactoring, changing Color::alpha() to be named Color::alphaByte() to clarify it returns a 0-255 based value, I spotted a use in AccessibilityTable::isDataTable() that compares the Color::alphaByte() value to 1, but probably wants to be comparing it to 255 (or checking isOpaque()).

We should verify that assumption, fix it and add a test.

Comment 1 Radar WebKit Bug Importer 2020-07-19 11:14:33 PDT

<rdar://problem/65793228>

Comment 2 Sam Weinig 2020-07-19 11:14:56 PDT

Adding James and Chris.

Comment 3 Radar WebKit Bug Importer 2020-07-19 11:15:03 PDT

<rdar://problem/65793254>