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

Issue 614250 link

Starred by 16 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Select menu blinks repeatedly when opening

Project Member Reported by esprehn@chromium.org, May 24 2016

Issue description

Google Chrome	52.0.2743.0 (Official Build) canary (64-bit)
Revision	2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}
OS	Mac OS X 

What steps will reproduce the problem?
(1) Load https://www.comparably.com/
(2) Click the Job Title select boxes. The select menu opens and closes twice.

What is the expected output?

Should not open twice, there seems to be some problem where the select menu list is opening and closing repeatedly.
 

Comment 1 by tkent@chromium.org, May 24 2016

Components: -Blink>Forms Blink>Forms>Select
Labels: -OS-All OS-Android OS-Mac
Status: Available (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, May 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/194ad8e687efc243d3b836a648697aaab394fbf0

commit 194ad8e687efc243d3b836a648697aaab394fbf0
Author: tkent <tkent@chromium.org>
Date: Wed May 25 07:02:13 2016

SELECT popup: Do not update popup content for arbitrary attribute change.

In the site of  crbug.com/615250 , mousedown event handler for a SELECT element sets
and deletes id attribute of the SELECT element. We shouldn't try to update popup
content in such case.

BUG= 614250 

Review-Url: https://codereview.chromium.org/2012653002
Cr-Commit-Position: refs/heads/master@{#395818}

[modify] https://crrev.com/194ad8e687efc243d3b836a648697aaab394fbf0/third_party/WebKit/Source/core/html/HTMLSelectElement.cpp

Comment 3 by tkent@chromium.org, May 25 2016

Labels: Merge-Request-52 M-52
Owner: tkent@chromium.org
Status: Fixed (was: Available)

Comment 4 by tin...@google.com, May 25 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)
Please merge your CL before 5.00 pm PST, if you want to make it for tomorrows Dev release.
Project Member

Comment 6 by bugdroid1@chromium.org, May 25 2016

Labels: -merge-approved-52 merge-merged-2743
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/265054928b149e423dc8950e7222b642a092e0cb

commit 265054928b149e423dc8950e7222b642a092e0cb
Author: Kent Tamura <tkent@chromium.org>
Date: Wed May 25 22:51:58 2016

Merge "SELECT popup: Do not update popup content for arbitrary attribute change." to M52.

In the site of  crbug.com/615250 , mousedown event handler for a SELECT element sets
and deletes id attribute of the SELECT element. We shouldn't try to update popup
content in such case.

BUG= 614250 

Review-Url: https://codereview.chromium.org/2012653002
Cr-Commit-Position: refs/heads/master@{#395818}
(cherry picked from commit 194ad8e687efc243d3b836a648697aaab394fbf0)

Review URL: https://codereview.chromium.org/2012943002 .

Cr-Commit-Position: refs/branch-heads/2743@{#64}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/265054928b149e423dc8950e7222b642a092e0cb/third_party/WebKit/Source/core/html/HTMLSelectElement.cpp

Labels: TE-Verified-M52 TE-Verified-52.0.2743.10
Verified this issue on Mac OS 10.11.5 using chrome latest Dev M52-52.0.2743.10 by following steps mentioned in the comment #0. Observed the select menu in the page opens  without any blinks and able to  select available options as expected. Hence adding TE-Verified label.

614250.mp4
891 KB Download

Comment 8 by tkent@chromium.org, Jun 1 2016

Cc: rnimmagadda@chromium.org
 Issue 614838  has been merged into this issue.
 Issue 616197  has been merged into this issue.

Comment 10 by tkent@chromium.org, Jun 15 2016

Cc: ccameron@chromium.org hodie@chromium.org
 Issue 619865  has been merged into this issue.

Comment 11 by tkent@chromium.org, Jun 22 2016

 Issue 621613  has been merged into this issue.

Sign in to add a comment