New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 662828 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Android System Webview HTML 5 Datalist display issue

Reported by djsanj00...@gmail.com, Nov 7 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.87 Safari/537.36

Steps to reproduce the problem:
1. DataList is not coming. 

What is the expected behavior?
Data List should be populated

What went wrong?
Filter code is running but not list is showing on the view

Did this work before? Yes Working fine on earlier Android Webview version

Chrome version: 54.0.2840.87  Channel: stable
OS Version: OS X 10.12.1
Flash Version: Shockwave Flash 23.0 r0
 
Cc: jmukthavaram@chromium.org
Labels: Needs-Feedback
Thanks for the Report.

could you please provide us the sample html file / URL to triage the issue further.

Hi,

I am creating a Hybrid Mobile App & i had used simple HTML 5 DataList
Control, which was showing earlier but stop working in recent Android
System Webview Update.
When i  uninstall the Webview it came back.
Following is the HTML Code for same

<input focus type="text" placeholder="City/Municipality" ng-model="dc.model.
address.MunicipalityID.label" list="municipalList" data-list-filter="^"
maxlength="100" />
            <datalist id="municipalList">
                <option ng-repeat="x1 in dc.model.MunicipalList" value="{{x1
.LookupValue}}"></option>
            </datalist>



Thanks & Regards,
Sanjay Nagpal

Comment 3 by l...@chromium.org, Nov 14 2016

Components: -Platform>DevTools Mobile>WebView

Comment 4 by rsesek@chromium.org, Nov 14 2016

Labels: -OS-Mac OS-Android
Any Update on this?

Comment 6 by ondrej.s...@pds.eu, Nov 22 2016

Hi,
I'm having the same issue as described.
You can easily test it on datalist test page - http://www.w3schools.com/tags/tryit.asp?filename=tryhtml5_datalist
(datalist is not showing up). The issue persist also on beta channel.

Regards
Ondřej Svoboda
Alright, this is a regression between versions 54.0.2808.0 and 54.0.2809.0.


https://chromium.googlesource.com/chromium/src/+log/54.0.2808.0..54.0.2809.0?pretty=fuller&n=10000
Cc: wangxianzhu@chromium.org
wangxianzhu@ can this be related to any of your changes in the range of CLs in comment #7?
Cc: gsennton@chromium.org
Cc: tkent@chromium.org
Ah, thanks  wangxianzhu@ I just took a quick look at the CLs and the closest (language-wise) to data list I could find was your data item list stuff (but I guess that could be completely unrelated ;))
Labels: -Needs-Feedback
Labels: -Pri-2 M-55 ReleaseBlock-Stable Pri-1
Tagging this as RB-Stable given that it's a regression.
hmmm, tkent@ is ooo this week :/

Comment 15 by aluo@chromium.org, Nov 23 2016

Cc: aluo@chromium.org
Labels: Needs-Bisect
manually reverted those 2 changes (1 at a time), still see the datalist not showing up.  I'm having trouble building with the old tree, will work on more proper bisect Wednesday.
Alright, thanks aluo@
Owner: aluo@chromium.org

Comment 18 by aluo@chromium.org, Nov 23 2016

Cc: leon....@intel.com
Labels: -Needs-Bisect
Owner: gsennton@chromium.org
Found breaking cl: https://codereview.chromium.org/2007473004  This issue happens only in webview, not chrome.  Assigning to Gustav for now but cc'ed original committer as well.

Comment 19 by leon....@intel.com, Nov 24 2016

OK I suppose this has the same root cause with  issue 662040 , which has been solved by https://chromium.googlesource.com/chromium/src/+/b0152a63b6a341a79949f97b8d77ea876d73727c on M55 branch, but seems it's too late to merge into M54.
For sure would you please help to confirm whether this issue still exists on latest M55? Thanks.

M55 should have been fixed, but Master is still not OK, https://codereview.chromium.org/2523193002/ is trying to do so for Master.

So sorry for the regression...

Comment 20 by aluo@chromium.org, Nov 24 2016

Confirmed fixed on M55 55.0.2883.64 on Nexus 6 device but not on master.

Comment 21 by leon....@intel.com, Nov 24 2016

Labels: M-56
Waiting for landing of https://codereview.chromium.org/2523193002/, after that we need to merge both https://codereview.chromium.org/2491103003 and https://codereview.chromium.org/2523193002/ into M56(branch 2919).

#Note: M55 has already been fixed by https://chromium.googlesource.com/chromium/src.git/+/b0152a63b6a341a79949f97b8d77ea876d73727c, no further work needed.

Comment 22 by leon....@intel.com, Nov 24 2016

Cc: -leon....@intel.com
Owner: leon....@intel.com
Status: Started (was: Unconfirmed)
Labels: -M-55
Removing M55 given c#21, we'll keep the bug open to track the fix for M56.

Comment 24 by leon....@intel.com, Nov 29 2016

Locally confirmed on Android device that datalist can show well on the latest M56(branch 2924), because the root cause has been solved at  issue 662040 .

Comment 25 by aluo@chromium.org, Jan 4 2017

Confirmed fixed on M56 56.0.2924.51 on Galaxy A5.
Status: Fixed (was: Started)
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-56; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-56 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD

Sign in to add a comment