New issue
Advanced search Search tips

Issue 864435 link

Starred by 2 users
Status: Fixed
Owner:
beccahughes@chromium.org
Closed: Jul 18
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Videos/Photos get overlapped with main menu icons in Google Photos.

Reported by vineetha...@etouch.net, Jul 17 
Chrome Version: 69.0.3494.0 (Official Build) Revision e91414c45bcdc6397f1a38faa0c826fbc47cd772-refs/branch-heads/3494@{#1}(32/64 bit)
OS: Mac(10.12.6, 10.13.1, 10.13.6, 10.14), Windows(7,8,8.1,10) and Linux(14.04) OS

What steps will reproduce the problem?
(1) Launch Chrome, navigate to https://www.google.com/photos/ , click on 'Go to Google Photos'.
(2) Sign in with valid credentials and observe the main menu icons on the LHS of the page.

Actual Result  : Videos/Photos are seen overlapped with main menu icons.
Expected Result: Videos/Photos should not be overlapped with main menu icons.

This is regression issue broken in ‘M-69’ and providing the bisect using per-revision bisect,
Good build: 69.0.3479.0(Revision: 571804)
Bad build : 69.0.3480.0(Revision: 572081)

You are probably looking for a change made after 572040 (known good), but no later than 572041 (first known bad).

CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  
https://chromium.googlesource.com/chromium/src/+log/228ebb2d5fd733b30d87e90a278836154cc0a2fe..fc8924e1488fb7b15eb5d54206a51621ac8bae0a

Suspect: https://chromium.googlesource.com/chromium/src/+/fc8924e1488fb7b15eb5d54206a51621ac8bae0a

@beccahughes: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Thank You!
ActualVideo.mp4
300 KB View Download
ExpectedVideo.mp4
339 KB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 18 

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

commit 113cc2d54f97a1384ecc5c3a0b8bcddbd0c3765f
Author: Becca Hughes <beccahughes@chromium.org>
Date: Wed Jul 18 00:28:55 2018

[CSS Env Vars] Throw parse error when used with min/max

min() and max() are not supported and should throw a parse
error when used with env() and var().

BUG= 864435 

Change-Id: I7e89607cc6bb41910131b35a72dabb979f10b7dc
Reviewed-on: https://chromium-review.googlesource.com/1140746
Commit-Queue: Emil A Eklund <eae@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#575871}
[add] https://crrev.com/113cc2d54f97a1384ecc5c3a0b8bcddbd0c3765f/third_party/WebKit/LayoutTests/css-parser/variables-invalid-functions.html
[modify] https://crrev.com/113cc2d54f97a1384ecc5c3a0b8bcddbd0c3765f/third_party/blink/renderer/core/css/parser/css_variable_parser.cc

Comment 2 by beccahughes@chromium.org, Jul 18

Status: Fixed (was: Assigned)

Comment 3 by beccahughes@chromium.org, Jul 19 

Issue 865131 has been merged into this issue.
►

Sign in to add a comment