New issue
Advanced search Search tips

Issue 903623 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Console.table shows blank after certain rows

Reported by tajrij.k...@gmail.com, Nov 9

Issue description

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

Steps to reproduce the problem:
1. Open this code https://jsfiddle.net/rk4jhw2o/
2. See on dev tools ( console )
3. There are many blank rows

What is the expected behavior?
it should print the text for all rows

What went wrong?
For after a certain row number, it goes blank without any text.

Did this work before? Yes 

Chrome version: 72.0.3605.0  Channel: canary
OS Version: OS X 10.13.6
Flash Version: 

1st screenshot was on canary Version 72.0.3605.0 (Official Build) canary (64-bit)
2nd screenshot was on Chrome Version 70.0.3538.77 (Official Build) (64-bit) , it shows text for all rows
 
Screen Shot 2018-11-09 at 8.16.01 AM.png
50.5 KB View Download
Screen Shot 2018-11-09 at 8.18.03 AM.png
200 KB View Download
Labels: Needs-Triage-M72 Needs-Bisect
Cc: viswa.karala@chromium.org
Labels: -Pri-2 -Needs-Bisect hasbisect-per-revision ReleaseBlock-Stable RegressedIn-72 Triaged-ET Target-72 M-72 FoundIn-72 OS-Linux OS-Windows Pri-1
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on reported version# 72.0.3605.0 using Mac 10.12.6, Ubuntu 14.04 and Windows-10, hence providing Bisect Info

Bisect Info:
================
Good build: 72.0.3598.0
Bad build: 72.0.3599.0

You are probably looking for a change made after 604627 (known good), but no later than 604628 (first known bad).
https://chromium.googlesource.com/chromium/src/+log/d4551e74b3bc17fd56a1d83f40e547fe56984f84..c20a6343f43c1c4df1d829f2d1a6f953f4cfe000

Unable to find the correct suspect from above change log, hence marking this issue as Untriaged and requesting someone from the Dev team help in assigning it to correct owner.
Adding ReleaseBlock-Stable for M-72, feel free to remove it if is not applicable.

Thanks!
Owner: l...@chromium.org
Components: -Platform>DevTools Platform>DevTools>JavaScript
Status: Assigned (was: Untriaged)
luoe@: Could you please take a look at this.
c#4: Yes, I have a CL in review.  If it goes well, we should be able to land it in V8 and have the following V8-autoroll before the 72 branch.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 15

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

commit b11cedb4726a40032ca2c34dc0b232f98d6007b3
Author: Erik Luo <luoe@chromium.org>
Date: Thu Nov 15 00:23:05 2018

[inspector] restore console.table limits

`console.table` used to have a 1k limit on preview properties, which
regressed to 100 during refactoring.

This CL restores the 1k limit, and ensures that `buildEntryPreview`
does not use the 1k limit.

Bug:  chromium:903623 
Change-Id: I8fab3182a79d1bb6b662e2ff9b229db3d3a889ca
Reviewed-on: https://chromium-review.googlesource.com/c/1332127
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Erik Luo <luoe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57518}
[modify] https://crrev.com/b11cedb4726a40032ca2c34dc0b232f98d6007b3/src/inspector/injected-script.cc
[modify] https://crrev.com/b11cedb4726a40032ca2c34dc0b232f98d6007b3/src/inspector/value-mirror.cc
[modify] https://crrev.com/b11cedb4726a40032ca2c34dc0b232f98d6007b3/src/inspector/value-mirror.h
[modify] https://crrev.com/b11cedb4726a40032ca2c34dc0b232f98d6007b3/test/inspector/runtime/console-table-expected.txt
[modify] https://crrev.com/b11cedb4726a40032ca2c34dc0b232f98d6007b3/test/inspector/runtime/console-table.js

Status: Fixed (was: Assigned)

Sign in to add a comment