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

Issue 704360 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: 2017-03-23
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

shapedetection/* layout tests fail under MSAN

Project Member Reported by dgozman@chromium.org, Mar 23 2017

Issue description

Comment 1 by mcasas@chromium.org, Mar 23 2017

Cc: mcasas@chromium.org
Components: Blink>ImageCapture
This https://codereview.chromium.org/2771703002/ might help
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 23 2017

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

commit 32e0bf3ee63c3798f51c2ecefb941195326cc0da
Author: dgozman <dgozman@chromium.org>
Date: Thu Mar 23 00:21:34 2017

Disable some shapedetection layout tests under MSAN

BUG= 704360 
NOTRY=true
TBR=junwei,haraken

Review-Url: https://codereview.chromium.org/2774503003
Cr-Commit-Position: refs/heads/master@{#458949}

[modify] https://crrev.com/32e0bf3ee63c3798f51c2ecefb941195326cc0da/third_party/WebKit/LayoutTests/MSANExpectations

Labels: -Pri-1 -Sheriff-Chromium Pri-2
Owner: mcasas@chromium.org
Status: Assigned (was: Untriaged)
P1 -> P2, removing sheriff label as the test has been disabled

mcasas@, could you follow up on this?

Comment 4 by mcasas@chromium.org, Mar 23 2017

NextAction: 2017-03-23
yeah, https://codereview.chromium.org/2771703002/ just landed
and it might address this issue since it basically initializes
otherwise un-initialized fields.  Will check tomorrow

Comment 5 by mcasas@chromium.org, Mar 23 2017

Labels: ShapeDetection OS-Linux
Status: Fixed (was: Assigned)
The CL mentioned in #4 (https://crrev.com/2771703002/,
against https://crrev.com/665488) made the MSAN build
green after landing in :

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20MSAN/builds/882

So I'm going to preventively mark this Issue as Fixed.

Sign in to add a comment