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

Issue 620574 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary white flash is seen on hitting full screen for gallery

Project Member Reported by sc00335...@techmahindra.com, Jun 16 2016

Issue description

Version: 53.0.2768.0/8459.0.0 (Official Build) dev-channel peppy
OS: Chrome os

What steps will reproduce the problem?
(1) Sign in to user >> Open any image in gallery >> Hit Full screen and observe

Expected: On Transforming to full screen no flash should be seen.
Actual: Instead unwanted white flash is seen on hitting full screen. 

This is a regression issue as no flash is seen in 50.0.2661.103/7978.74.0 stable channel daisy

@yawano: Please confirm the issue

 
Issue is also seen in 53.0.2768.0/8459.0.0 of daisy,blaze,jerry.
Cc: songsuk@chromium.org pucchakayala@chromium.org
Able to reproduce the issue on Blaze using chrome version 53.0.2768.0/8459.0.0

Comment 3 by yawano@chromium.org, Jun 22 2016

Cc: yawano@chromium.org fukino@chromium.org
Owner: oka@chromium.org
oka@: Could you take a look at this? Thank you!

Comment 4 by oka@chromium.org, Jun 22 2016

This kind of FOUS also happens with Chrome.
1. Show a large image such as https://upload.wikimedia.org/wikipedia/commons/3/3d/LARGE_elevation.jpg on Chrome.
2. Maximize the window
Then white blink is seen.

So I think it's not Files App's problem.

I also found that flickering happens only for the first time after the app is opened. If you opens the app and then maximize, minimize and maximize the window, flickering doesn't happen for the second time.

fukino@, do you know to whom can I assign the issue?

Comment 5 by fukino@chromium.org, Jun 22 2016

I'm not sure which part is suspicious based on the current information.
Did you reproduce it on Linux?
If so, can you do bisection and assign this to the CL owner?

Comment 6 by oka@chromium.org, Jun 27 2016

Discussed offline, the root cause will be on Blink. On our side, let's make
background black to mitigate the distraction of flash.
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 27 2016

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

commit 56f5f878bfdfbdf30036b3d0ae5d03b42ff89048
Author: oka <oka@chromium.org>
Date: Mon Jun 27 04:17:45 2016

Make body background black to mitigate flash on hitting full screen for gallery.

BUG= 620574 
TEST=none

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

[modify] https://crrev.com/56f5f878bfdfbdf30036b3d0ae5d03b42ff89048/ui/file_manager/gallery/css/gallery.css

Comment 8 by oka@chromium.org, Jun 27 2016

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
ChromeOS build 8530.96.0 / 53.0.2785.154

Sign in to add a comment