Bug 117491 - Consider using element rect instead of selection rect to compute the center of an element to reduce code size
Summary: Consider using element rect instead of selection rect to compute the center o...
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks:
 
Reported: 2013-06-10 22:53 PDT by Ryosuke Niwa
Modified: 2013-06-10 23:40 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 Ryosuke Niwa 2013-06-10 22:53:31 PDT
We might want to merge https://chromium.googlesource.com/chromium/blink/+/a1fc9809d682deb494818a977cd480872f17f94e

Remove unused blink accessibility code.

The only functional change is to clickPoint, where we're now returning
the center of the element rect rather than the selection rect. There's
no difference 99% of the time. If that 1% is needed, we could rewrite
that logic in far less code.

Everything else was literally unused. Much is already implemented in
Chromium's browser process; the rest couldn't be used without refactoring
anyway.


Chris, any opinion on this?
Comment 1 chris fleizach 2013-06-10 23:38:42 PDT
They're pulling out a lot of the code that Mac Accessibility uses to provide a complete API implementation. The comment says that they have implementations of this code in the browser side. 

Also the clickPoint change breaks the ability to bring up the context menu on the selected text range (I think, that code is a few years old)

So we should't take this change
Comment 2 Ryosuke Niwa 2013-06-10 23:40:59 PDT
(In reply to comment #1)
> They're pulling out a lot of the code that Mac Accessibility uses to provide a complete API implementation. The comment says that they have implementations of this code in the browser side. 
> 
> Also the clickPoint change breaks the ability to bring up the context menu on the selected text range (I think, that code is a few years old)
> 
> So we should't take this change

Thanks!