Bug 59826

Summary: Change EventContext logic to use treeScopes
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: UI EventsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 59805    

Description Dimitri Glazkov (Google) 2011-04-29 13:53:15 PDT
Once treeScopes are used in all SVG instances, we no longer need to build a full vector of EventContexts (http://codesearch.google.com/codesearch/p#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/dom/EventDispatcher.cpp&l=237&exact_package=chromium). Instead, we could rely on tree scopes at the time of dispatch to determine whether to retarget.
Comment 1 Dimitri Glazkov (Google) 2011-06-08 15:09:53 PDT
This is wrong. Since we need to know the _previous_ treeScope, we are still stuck with walking up the tree. Perhaps the data structure could be optimized for non-shadow DOM cases, but this is no critical at all.