Bug 89180

Summary: Event dispatcher should use InsertionPoint::hasDistribution instead of InsertinPoint::isActive in re-targeting.
Product: WebKit Reporter: Hayato Ito <hayato>
Component: UI EventsAssignee: Hayato Ito <hayato>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, dominicc, morrita, shinyak, tasak, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 89073    
Attachments:
Description Flags
fix none

Hayato Ito
Reported 2012-06-15 00:11:07 PDT
Event dispatcher should use InsertionPoint::hasDistribution() instead of InsertionPoint::isActive() in re-targeting. InsertionPoint::isActive() returns true even when the insertion point does not select any nodes and has fallback elements. That causes a wrong re-targeting result.
Attachments
fix (5.36 KB, patch)
2012-06-15 01:00 PDT, Hayato Ito 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Hayato Ito
Comment 1 2012-06-15 01:00:10 PDT
Created attachment 147767 [details] fix
WebKit Review Bot
Comment 2 2012-06-18 12:18:30 PDT
Comment on attachment 147767 [details] fix Clearing flags on attachment: 147767 Committed r120615: <http://trac.webkit.org/changeset/120615>
WebKit Review Bot
Comment 3 2012-06-18 12:18:36 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.