New issue
Advanced search Search tips

Issue 635214 link

Starred by 2 users
Status: WontFix
Owner: ----
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Remove OS_ANDROID ifdef around RenderViewImpl::detectContentAndroid

Project Member Reported by dglazkov@chromium.org, Aug 6 2016 
We should not have these in ifdefs in RenderViewImpl (or anywhere). If we want to vary the type of detection (or whether it happens), we should just populate a different list of detectors.

Comment 1 by twelling...@chromium.org, Oct 18 2016 

Fyi, I'm planning to disable/remove content detection on Android in favor of a new touch-to-search "quick actions" UX. I can forward some emails with background if you're interested.

Comment 2 by rsgav...@chromium.org, Jan 25 2017

Status: Assigned (was: Untriaged)

Comment 3 by dglazkov@chromium.org, Jan 27 2017

Components: Blink>Internals
Owner: ----
Status: Available (was: Assigned)

Comment 4 by aelias@chromium.org, Mar 31 2017

Status: WontFix (was: Available)
This code is being deleted (tracked in  http://crbug.com/664307 ) so this issue is moot.
►

Sign in to add a comment