New issue
Advanced search Search tips

Issue 807917 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Missing row after hiding filters in Network table

Reported by nfd...@gmail.com, Feb 1 2018

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.119 Safari/537.36

Steps to reproduce the problem:
1. Record more than a screenful of activity in the Network panel.
2. Show the filter bar.
3. Position a table row so that it just barely extends past the bottom of the viewport.
4. Hide the filter bar.

What is the expected behavior?
The following row is partially revealed.

What went wrong?
There is blank space below the former bottommost row. Scrolling reveals the missing row.

Did this work before? N/A 

Chrome version: 64.0.3282.119  Channel: stable
OS Version: OS X 10.13.3
Flash Version:
 
network table missing row.mov
1.6 MB View Download
Labels: Needs-Triage-M64

Comment 2 by l...@chromium.org, Feb 1 2018

Owner: l...@chromium.org
Status: Assigned (was: Unconfirmed)
Thanks for the report.
Project Member

Comment 3 by bugdroid1@chromium.org, May 31 2018

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

commit 4dcc9eb7b3652120ef1b3bd07bdd9b918c9a7d60
Author: Erik Luo <luoe@chromium.org>
Date: Thu May 31 18:35:49 2018

DevTools: update Network view on filter bar toggle

We used to listen to resize events from the root DevTools window or a
SplitWidget, but not when the viewport grows/shrinks due to toggling
the filter bar.

This CL updates the viewport in that case, too.

Bug:  807917 
Change-Id: I7c5cfbe20f7e293f808e8138b2c7da0ab76df845
Reviewed-on: https://chromium-review.googlesource.com/1077757
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Erik Luo <luoe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563331}
[modify] https://crrev.com/4dcc9eb7b3652120ef1b3bd07bdd9b918c9a7d60/third_party/blink/renderer/devtools/front_end/network/NetworkLogView.js

Comment 4 by l...@chromium.org, May 31 2018

Status: Fixed (was: Assigned)

Sign in to add a comment