Summary: | [Meta] Track all usage of HitTestRequest::DisallowShadowContent in HitTests. | ||
---|---|---|---|
Product: | WebKit | Reporter: | Hayato Ito <hayato> |
Component: | UI Events | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED CONFIGURATION CHANGED | ||
Severity: | Normal | CC: | allan.jensen, cbiesinger, dglazkov, rniwa, tonikitoo, webcomponents-bugzilla, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 112073, 113171 | ||
Bug Blocks: | 59805 |
Description
Hayato Ito
2013-03-11 19:34:34 PDT
We should fix this flag to be on by default. It's very rare that AllowShadowContent == false. Yeah, that's our goal. (In reply to comment #1) > We should fix this flag to be on by default. It's very rare that AllowShadowContent == false. HitTests are used from a lot of places in WebCore. Turning the flag on the default requirers some braveness. I am pretty sure that existing Layout tests can not catch the regression. My current plan is conservative. Use this flag for each cases step by step. Tough, we need to figure out the usage in each places, which might take much time. I feel that we must change the title of this bug after https://bugs.webkit.org/show_bug.cgi?id=113171 is landed, like: 'Track all usage of HitTestRequest::DisallowShadowContent' in WebKit. (In reply to comment #3) > HitTests are used from a lot of places in WebCore. > > Turning the flag on the default requirers some braveness. I am pretty sure that existing Layout tests can not catch the regression. > > My current plan is conservative. Use this flag for each cases step by step. > Tough, we need to figure out the usage in each places, which might take much time. https://bugs.webkit.org/show_bug.cgi?id=113171 was landed. I've updated the title of this bug. Allan, I think we are good to close this bug now, right? HitTestRequest::DisallowShadowContent is not a thing anymore. Oh, actually, this got renamed to DisallowUserAgentShadowContent. But I don't think we need this meta bug to track this. |