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

Issue 762385 link

Starred by 4 users
Status: Fixed
Owner:
yukishiino@chromium.org
Closed: Sep 2017
Cc:
█ krajshree@chromium.org
ajha@chromium.org
nyerramilli@chromium.org
rbasuvula@chromium.org
█ brajkumar@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
lfg@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Tab crashes on performing audit in devtools.

Reported by rp...@etouch.net, Sep 6 2017 
Version: 63.0.3207.0 7a1987ec20dbb890a2e57ae33cb5013e5235e468-refs/heads/master@{#499829}
OS: Windows (7,8,8.1,10)

What steps will reproduce the problem?
1.Launch chrome, navigate to NTP and open devtools and run audit,observe
 
Actual: Tab crashes on performing audit
Expected: Tab should not crash

Crash ID : 8cf9e10a90ceb197 (Local Crash ID: 1e847ac0-502e-4842-8d4c-ad7e2d40a7a4)

This is regression issue, broken in ‘M 63’ and will soon update other info :
Good build:63.0.3206.0
Bad build: 63.0.3207.0

Comment 1 by rp...@etouch.net, Sep 6 2017

Labels: hasbisect-per-revision OS-Linux
Owner: yukishiino@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 63.0.3206.0 (Revision: 499528).
Bad build: 63.0.3207.0 (Revision: 499829).

You are probably looking for a change made after 499549 (known good), but no lat
er than 499550 (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/146841b1b3e8c7de3c0f3d7997277c5e7d9e1201..014bbbb65ef9fed41c95613f56225758445d54f8?pretty=fuller&n=100

From the CL above, assigning the issue to the concern owner 

@yukishiino- 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.

Suspect : https://chromium.googlesource.com/chromium/src/+/014bbbb65ef9fed41c95613f56225758445d54f8

Thanks!
Actual_video.mp4
478 KB View Download
Expected_video.mp4
941 KB View Download

Comment 2 by rp...@etouch.net, Sep 6 2017

Labels: HasTestcase

Comment 3 by brajkumar@chromium.org, Sep 6 2017

Cc: krajshree@chromium.org brajkumar@chromium.org ajha@chromium.org
 Issue 762387  has been merged into this issue.

Comment 4 by rp...@etouch.net, Sep 6 2017

Labels: OS-Mac

Comment 5 by yukishiino@chromium.org, Sep 6 2017

Labels: OS-Android OS-Chrome OS-Fuchsia
Status: Started (was: Assigned)

Comment 6 by yukishiino@chromium.org, Sep 6 2017

Components: Blink>Bindings

Comment 7 by rbasuvula@chromium.org, Sep 6 2017

Labels: ReleaseBlock-Dev
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.
Project Member

Comment 8 by bugdroid1@chromium.org, Sep 7 2017 

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

commit eeadb48c284e76dd16effce5c3ccd6b712f674e8
Author: Yuki Shiino <yukishiino@chromium.org>
Date: Thu Sep 07 05:18:38 2017

Supports Object::DefineOwnProperty with ENTER_V8_NO_SCRIPT.

As Blink needs a way to define a property without running a script,
make Object::DefineOwnProperty use ENTER_V8_NO_SCRIPT if the
receiver object is not a JSProxy.  Quite similar to
https://crrev.com/c/v8/v8/+/608348

Bug: chromium:728583,  chromedriver:1978 ,  chromium:762385 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
Change-Id: If358bf0d156139c456de369ac04da2be6e626143
Reviewed-on: https://chromium-review.googlesource.com/651949
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Adam Klein <adamk@chromium.org>
Reviewed-by: Camillo Bruni <cbruni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#47862}
[modify] https://crrev.com/eeadb48c284e76dd16effce5c3ccd6b712f674e8/src/api.cc

Comment 9 by yukishiino@chromium.org, Sep 7 2017

Status: Fixed (was: Started)
►

Sign in to add a comment