New issue
Advanced search Search tips

Issue 711350 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Task



Sign in to add a comment

Use __gCrWeb's elementFromPoint_() in Context.contextFromPoint

Project Member Reported by danyao@chromium.org, Apr 13 2017

Issue description

This came up during refactoring of core.js into smaller pieces (including context_menu.js).

Context.contextFromPoint uses document.elementFromPoint(x, y). The elementFromPoint_() method in __gCrWeb implements similar functionality, and can also look into frames and iframes. However, the comment also suggests that the latter was added for iOS4/5, so it's not clear if this still adds value over the latest native implementation.

We should:
- Understand the difference between the latest native implementation and the __gCrWeb version.
- Evaluate if it makes sense to unify the two, and if so:
- What is the best path



 
Cc: kkhorimoto@chromium.org danyao@chromium.org
Owner: eugene...@chromium.org
Status: Assigned (was: Untriaged)
Components: -Mobile>WebView>Glue UI>Browser>Search>ContextualSearch
Owner: olivierrobin@chromium.org
Components: UI>Browser>Mobile>TouchToSearch
Components: -UI>Browser>Search>ContextualSearch

Sign in to add a comment