Jquery mobile popups are not displayed
Reported by
arnaud...@gmail.com,
Oct 27 2017
|
|||||||||||||||||||
Issue descriptionUserAgent: 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.
,
Oct 27 2017
,
Oct 27 2017
,
Oct 27 2017
,
Oct 27 2017
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.
,
Oct 27 2017
FWIW I can reproduce this on M62 stable Mac, but not on M64 Mac canary.
,
Oct 27 2017
,
Oct 27 2017
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.
,
Oct 27 2017
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?
,
Oct 27 2017
Any chance it might need a merge to M63/62?
,
Oct 27 2017
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).
,
Oct 27 2017
hmm strange - I reverted the CL in 64 so you should see the issue in both 62 and 64 but not in 63.
,
Oct 27 2017
Hm, are we sure that CL is the right fix in the fix range? Maybe there's another culprit.
,
Oct 27 2017
That could be the case. FWIW. I cant seem to repro the issue on 61, 63 or 64. Attaching screenshots
,
Oct 30 2017
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...!!
,
Oct 30 2017
Correction: As no issue is observed in latest beta #63.0.3239.18. Hence, requesting the merge to M-62. Thanks...!!
,
Oct 30 2017
,
Oct 30 2017
What is the exact user impact for this bug? Is it blocking any functionality and/or is there a workaround?
,
Oct 30 2017
Can you provide me with a video recording of this issue's repro so that I can better understand priority? Thank you!
,
Oct 30 2017
,
Oct 30 2017
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!
,
Oct 30 2017
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.
,
Oct 30 2017
,
Oct 30 2017
[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.
,
Oct 31 2017
,
Nov 6 2017
|
|||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||
Comment 1 by arnaud...@gmail.com
, Oct 27 2017