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

Issue 768782 link

Starred by 1 user
Status: Fixed
Owner:
█ eostroukhov@chromium.org
Last visit > 30 days ago
Closed: Oct 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary text wrapping is applied for Port forwarding message on chrome://inspect page.

Reported by dchau...@etouch.net, Sep 26 2017 
Chrome Version: 63.0.3223.8 (Official Build)47de836373a20038efd14f8c3a154326f8c9ddda-refs/branch-heads/3223@{#11} 32/64-bit.
OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.12.6).

What steps will reproduce the problem?
1. Launch chrome, go to chrome://inspect/#devices and zoom in the page to 125% or more.
2. Open 'Port forwarding Settings' overlay and check 'Enable port forwarding' checkbox.
3. Completely resize the Chrome to it's extreme left and observe the Port forwarding message at header.

Unnecessary text wrapping is applied for Port forwarding message and it's overlapped to content of page.
Text wrapping should not apply for Port forwarding message at header.

This is a regression issue, broken in M-57 series, below is manual regression range.

Good build: 57.0.2939.0 
Bad build: 57.0.2940.0

You are probably looking for a change made after 436071 (known good), but no later than 436072 (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/3f8acfcbdadd2169715385a4348f53c19dd0942f..203fe850b9f974f324bda2c0a30707f2d4bd77bb

Suspect: https://chromium.googlesource.com/chromium/src/+/203fe850b9f974f324bda2c0a30707f2d4bd77bb

@eostroukhov: Kindly help to reassign, if your changes are not related to this issue.

Kindly review the attached screen-cast for reference.
Actual behavior.mp4
1.6 MB View Download
Expected behavior.mp4
1.8 MB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 19 2017 

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

commit c2aedcf5413fc22c69dc06401e309982f42855f8
Author: Eugene Ostroukhov <eostroukhov@chromium.org>
Date: Thu Oct 19 22:17:12 2017

[DevTools] Make infobar push contents down.

Bug:  768782 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I7fd7a16c3c4493c71662ed5ddc4a7906b6bafeee
Reviewed-on: https://chromium-review.googlesource.com/727188
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Eugene Ostroukhov <eostroukhov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510237}
[modify] https://crrev.com/c2aedcf5413fc22c69dc06401e309982f42855f8/chrome/browser/resources/inspect/inspect.css

Comment 2 by eostroukhov@chromium.org, Oct 20 2017

Status: Fixed (was: Assigned)

Comment 3 by susanjuniab@chromium.org, Oct 23 2017

Labels: TE-Verified-M64 TE-Verified-64.0.3247.0
Re-Tested this issue again on Mac OS 10.12.6, Windows 7 and Ubuntu 14.04 on 64.0.3247.0 Chrome version by following the steps mentioned in the original comment and can observe that unnecessary text wrapping is not applying for Port forwarding message at header, when the window is resized.

Hence adding TE Verified labels .

Thanks...
►

Sign in to add a comment