Regression: On print preview page, 'Copy link' overlay of 'Google+' is blackout.
Reported by
db...@etouch.net,
May 4 2017
|
||||||
Issue descriptionChrome Version: 60.0.3088.3 Revision 49d525d4dafb21903d1cd21025174928a3fdde75-refs/branch-heads/3088@{#4} OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.11.6,10.12.1) What steps will reproduce the problem? (1) Launch chrome, navigate to https://plus.google.com/u/0/ and sign into it. (2) Click on Share icon of any news/post, click on Copy link option. (3) Press 'Ctrl+P', observe on Preview page. Actual: On print preview page, 'Copy link' overlay is blackout. Expected: On print preview page, Copy link overlay should seen properly. This is a regression issue, broken in 'M-59', will soon update the bisect info: Good Build: 59.0.3055.0 Bad Build: 59.0.3056.0
,
May 8 2017
chrishtr@ Gentle Ping! Since this issue is marked as RB-Stable would you mind taking a look in to this issue? Thanks!
,
May 8 2017
Please stop pinging these bugs so aggressively. You filed it only a few days ago, and there has been a weekend.
,
May 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fb001947e4c1149351519bdfa30858843978f048 commit fb001947e4c1149351519bdfa30858843978f048 Author: chrishtr <chrishtr@chromium.org> Date: Fri May 12 03:15:45 2017 Pass isPrinting and other bits from parent context to child during print. BUG= 718362 Review-Url: https://codereview.chromium.org/2872033002 Cr-Commit-Position: refs/heads/master@{#471204} [add] https://crrev.com/fb001947e4c1149351519bdfa30858843978f048/third_party/WebKit/LayoutTests/paint/printing/print-box-shadow-expected.png [add] https://crrev.com/fb001947e4c1149351519bdfa30858843978f048/third_party/WebKit/LayoutTests/paint/printing/print-box-shadow-expected.txt [add] https://crrev.com/fb001947e4c1149351519bdfa30858843978f048/third_party/WebKit/LayoutTests/paint/printing/print-box-shadow.html [modify] https://crrev.com/fb001947e4c1149351519bdfa30858843978f048/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
,
May 12 2017
,
May 13 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f17c74d3b71031903b14a64ad7a3a1d4c6087860 commit f17c74d3b71031903b14a64ad7a3a1d4c6087860 Author: Chris Harrelson <chrishtr@chromium.org> Date: Sat May 13 03:33:34 2017 Pass isPrinting and other bits from parent context to child during print. BUG= 718362 Review-Url: https://codereview.chromium.org/2872033002 Cr-Original-Commit-Position: refs/heads/master@{#471204} Review-Url: https://codereview.chromium.org/2880903002 . Cr-Commit-Position: refs/branch-heads/3071@{#543} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} [add] https://crrev.com/f17c74d3b71031903b14a64ad7a3a1d4c6087860/third_party/WebKit/LayoutTests/paint/printing/print-box-shadow-expected.png [add] https://crrev.com/f17c74d3b71031903b14a64ad7a3a1d4c6087860/third_party/WebKit/LayoutTests/paint/printing/print-box-shadow-expected.txt [add] https://crrev.com/f17c74d3b71031903b14a64ad7a3a1d4c6087860/third_party/WebKit/LayoutTests/paint/printing/print-box-shadow.html [modify] https://crrev.com/f17c74d3b71031903b14a64ad7a3a1d4c6087860/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
,
May 13 2017
,
May 17 2017
Verified this issue on Ubuntu 14.01, Mac OS 10.12 and Windows-10 using chrome latest beta M59-59.0.3071.61 by following steps mentioned in the original comment, Observed 'Copy link' overlay is seen clear with out any black outs in the print-preview page. Hence adding TE-Verified label. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by msrchandra@chromium.org
, May 4 2017Owner: chrishtr@chromium.org
Status: Assigned (was: Unconfirmed)