Bug 59826 - Change EventContext logic to use treeScopes
Summary: Change EventContext logic to use treeScopes
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 59805
  Show dependency tree / graph
 
Reported: 2011-04-29 13:53 PDT by Dimitri Glazkov (Google)
Modified: 2011-06-08 15:09 PDT (History)
0 users

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 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.