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

Issue 761705 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Grey patch is seen while launching PDF.

Reported by aiman.an...@etouch.net, Sep 4 2017

Issue description

Chrome version: 63.0.3204.0 (Official Build)a3c5c018fa27742e47fdf12d1a20ff92962e832d-refs/heads/master@{#499404}(64 Bit).

OS: Mac(10.12.3, 10.12.5), Win(7,8,10)

Test URL: https://www.google.co.in/search?q=pdf+test&oq=pdf+test&aqs=chrome..69i57j0l5.2190j0j8&sourceid=chrome&ie=UTF-8

Steps to reproduce:
1.Launch chrome, open NTP and go to the above URL
2.Open any PDF and observe

Actual Result: Grey patch is seen when PDF opens.
Expected Result: Grey patch should no be seen

This is Regression Issue broken in M-62 and will soon provide other bisect info.

Manual Bisect Info.
Good Build: 62.0.3202.0
Bad Build: 63.0.3203.0

Note: Will soon update linux os info.
 
Actual Result.mov
3.3 MB Download
Labels: hasbisect
Owner: fsam...@chromium.org
Status: Assigned (was: Unconfirmed)
Narrow Bisect:
https://chromium.googlesource.com/chromium/src/+log/97621d9c31b719379ccdabb8220cca66aad58b8e..f3f9b8370b3d4b0526e5f89981a407abc6d74b30?pretty=fuller&n=10000

Suspecting: r499223

@fsamuel: Kindly help to re-assign if your change is not the cause for this issue.
Labels: -hasbisect hasbisect-per-revision
Labels: OS-Linux
Note: Above issue is also seen on Linux (14.04) for Canary #63.0.3205.0
Expected Result.mov
3.1 MB Download
Labels: ReleaseBlock-Stable
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.
Cc: danakj@chromium.org
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 6 2017

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

commit 8b93f9a9a5a619593b1956032332e3a83a252d5d
Author: Fady Samuel <fsamuel@chromium.org>
Date: Wed Sep 06 00:41:01 2017

Browser Plugin: Fix gray box during loading

After 646072, the MimeHandlerViewGuest got the initial default size of 300x300
at creation instead of the size of the BrowserPlugin. This CL fixes the problem
by waiting until the size of the BrowserPlugin is known before creating the
MimeHandlerViewGuest.

Bug:  761705 
Change-Id: I5b913252cf8a82c1f00b1381e9bfbaa698c06533
Reviewed-on: https://chromium-review.googlesource.com/651108
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: Ehsan Karamad <ekaramad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499818}
[modify] https://crrev.com/8b93f9a9a5a619593b1956032332e3a83a252d5d/content/renderer/browser_plugin/browser_plugin.cc
[modify] https://crrev.com/8b93f9a9a5a619593b1956032332e3a83a252d5d/extensions/renderer/guest_view/mime_handler_view/mime_handler_view_container.cc
[modify] https://crrev.com/8b93f9a9a5a619593b1956032332e3a83a252d5d/extensions/renderer/guest_view/mime_handler_view/mime_handler_view_container.h

Labels: TE-Verified-63.0.3207.0 TE-Verified-63
Retested the above issue on latest Canary #63.0.3207.0 on Mac(10.12.3, 10.12.5), Win(7,8,10),Linux (14.04).
Above issue is not reproducible on latest canary and is fixed.

Current Result.mov
3.1 MB Download
fsamuel@,
Could you please confirm this issue for closure as it is working as intended as per C#7 ,if there is no further work to be done here .

Thanks..!!
fsamuel@,
Could you please respond on C#8?
Thanks in advance..!
Gentle ping to get an update on this issue as per C#8.

Status: Verified (was: Assigned)
Tagging as verified since the issue is resolved.

Sign in to add a comment