Bug 157267 - Cleanup: Move policy decision for when to apply app quirks and workarounds into separate file
Summary: Cleanup: Move policy decision for when to apply app quirks and workarounds in...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Local Build
Hardware: Mac All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-02 09:10 PDT by Daniel Bates
Modified: 2016-05-03 18:39 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2016-05-02 09:10:48 PDT
Following up on a suggestion by Darin Adler in bug 157252, comment 2, we should extract the policy logic for when to apply a quirk or workaround from file Source/WebKit/mac/WebView/WebView.mm into a separate header file and implementation file. For convenience, the following is what Darin wrote:

[[
We should ... [group] ... functions that make decisions about ... quirks and workarounds in a header analogous to the RuntimeApplicationChecks one. This header would never mention specific application but would just list all the different quirks. The implementation file would contain all the expressions used to decide when each quirk applies, and comments about why each implements the right policy.

The implementations of the quirks would continue to be distributed throughout the code, but the policy of which quirks apply when would be grouped together.
]]