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

Issue 779051 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Not on Chrome anymore
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Jquery mobile popups are not displayed

Reported by arnaud...@gmail.com, Oct 27 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.75 Safari/537.36

Steps to reproduce the problem:
1. Go to  http://api.jquerymobile.com/popup/
2. Click to open a popup

What is the expected behavior?
Popup has to be displayed on a click or programmatically

What went wrong?
Popup is not displayed

Did this work before? N/A 

Chrome version: 62.0.3202.75  Channel: stable
OS Version: Ubuntu 17.10
Flash Version: 

It occurs since Chrome 61 to 62 upgrade.
 

Comment 1 by arnaud...@gmail.com, Oct 27 2017

It works on Chrome 63.0.3239.18 (Official Build) beta (64-bit)
Components: -Blink UI>Browser>PopupBlocker Blink>WindowDialog
Labels: -Type-Bug M-62 Type-Bug-Regression
Labels: -Pri-2 Pri-1
Forgive me if I'm mistaken, but it seems like this API is not about opening "popups" in the same sense as the popup blocker. These popups seem like lightbox-style widgets on the original page.
Labels: OS-Mac
FWIW I can reproduce this on M62 stable Mac, but not on M64 Mac canary.
Labels: Needs-Bisect Needs-Triage-M62
Cc: nainar@chromium.org
Components: -UI>Browser>PopupBlocker -Blink>WindowDialog Blink>CSS
I bisected the fix to:
https://chromium.googlesource.com/chromium/src/+log/2fa59ed505a54ea19c425854c70eeae0590145ff..115ac3528d0a279b55f5ec5ae82a872db9da3911

I haven't bisected the regression though. Perhaps someone in the fix range can comment.

Maybe https://chromium.googlesource.com/chromium/src/+/9f0c480da9cb188c82e16e6c6751875a60a114cf? Not sure how to best contact Rune since it seems like their opera acct is not monitored. cc naina instead.

Comment 9 by nainar@chromium.org, Oct 27 2017

Cc: rune.lil...@gmail.com
Status: Fixed (was: Unconfirmed)
The CL in the bisect range was reverted in 64, so that would explain why canary doesn't have this. 

Marking the bug as Fixed in that case?
Cc: abdulsyed@chromium.org
Any chance it might need a merge to M63/62?
The CL in the bisect range *is* the fix, since I bisected the fix range. I looks like that landed in M63, so it would only need merge to M62 (if it is necessary).
Status: Unconfirmed (was: Fixed)
hmm strange - I reverted the CL in 64 so you should see the issue in both 62 and 64 but not in 63.
Hm, are we sure that CL is the right fix in the fix range? Maybe there's another culprit.
That could be the case. FWIW. I cant seem to repro the issue on 61, 63 or 64. Attaching screenshots
m61.png
44.0 KB View Download
m63.png
34.5 KB View Download
m64.png
34.7 KB View Download
Screen Shot 2017-10-27 at 10.28.38 AM.png
17.1 KB View Download
Cc: treib@chromium.org
Labels: -Needs-Bisect -M-62 hasbisect-per-revision ReleaseBlock-Stable M-63 OS-Windows
Owner: nainar@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Windows 10, Mac 10.12.6 and Ubuntu 14.04 using reported version #62.0.3202.75 but the same is not reproducible in the latest canary #64.0.3253.0 and latest beta #63.0.3239.18.

Reverse Bisect Information:
=====================
Good build: 63.0.3226.0  Revision(504841)
Bad Build : 63.0.3225.0  Revision(504540)

Change Log URL: 
https://chromium.googlesource.com/chromium/src/+log/a9ba55c8571ab0d2624c83d413ccc7215a185f51..9f0c480da9cb188c82e16e6c6751875a60a114cf

From the above change log possible CL that fixed this issue:
Change-Id: Iee509e43e5fcc29d04944655d3d966ea61cc2adf
Reviewed-on: https://chromium-review.googlesource.com/684186 

nainar@ - Could you please check and merge the fix to M63 if it is a valid candidate.
Note: Assigning to the reviewer of the issue as the author's id i.e rune@opera.com doesn't seems to be valid anymore.
Adding label ReleaseBlock-Stable as it seems to be a recent regression.

Thanks...!!
Labels: -M-63 M-62
Correction:
As no issue is observed in latest beta #63.0.3239.18. Hence, requesting the merge to M-62.
Thanks...!!

Comment 17 by treib@chromium.org, Oct 30 2017

Cc: -treib@chromium.org
What is the exact user impact for this bug? Is it blocking any functionality and/or is there a workaround?
Cc: krajshree@chromium.org
Labels: Needs-TestConfirmation
Can you provide me with a video recording of this issue's repro so that I can better understand priority? Thank you!
Labels: Update-Weekly
Labels: -Needs-TestConfirmation
nainar@, here is the expected behavior. As per c#15, this doesn't repro on M63 & M64 because of this CL: https://chromium-review.googlesource.com/684186.

Seems like the root cause is same for both crbug/778631 & current issue (crbug/779051) ?

Thank you!
779051.mp4
1.0 MB View Download
I am hesitant to merge this to 62, since I didn't write the original code. Would feel more comfortable keeping this in 63 only and letting the change move to stable with 63. 
Labels: M-63
[Bulk Edit]
URGENT - PTAL.
M63 Stable promotion is coming soon and your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP. Thank you.

Labels: -ReleaseBlock-Stable
Status: Fixed (was: Assigned)

Sign in to add a comment