New issue
Advanced search Search tips

Issue 640886 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Layout Test http/tests/images/png-partial-load-as-document.html is leaking

Project Member Reported by haraken@chromium.org, Aug 25 2016

Issue description

The following layout test is failing on the leak detector

http/tests/images/png-partial-load-as-document.html

Probable cause:

https://codereview.chromium.org/2208073004/

 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 25 2016

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

commit 832983d0321d6d9e821b71e12ca1585354ece470
Author: haraken <haraken@chromium.org>
Date: Thu Aug 25 06:28:28 2016

Rebaseline: Mark png-partial-load-as-document.html as leaky

TBR=hiroshige@chromium.org
BUG= 640886 
NOTRY=true

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

[modify] https://crrev.com/832983d0321d6d9e821b71e12ca1585354ece470/third_party/WebKit/LayoutTests/LeakExpectations

Labels: -Pri-3 Pri-1
Raising the priority to P1. Memory leaks must be fixed asap. If it's not easy to fix, we should revert the change.


Status: Assigned (was: Untriaged)
The CL is being reverted: https://codereview.chromium.org/2278953002/
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 25 2016

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

commit c54188d25542cc58adaf938e75d10dafc1fc0176
Author: hiroshige <hiroshige@chromium.org>
Date: Thu Aug 25 13:51:09 2016

Revert of Rebaseline: Mark png-partial-load-as-document.html as leaky (patchset #1 id:1 of https://codereview.chromium.org/2275213002/ )

Reason for revert:
The CL that introduced the test is to be reverted in
https://codereview.chromium.org/2278953002/.

Original issue's description:
> Rebaseline: Mark png-partial-load-as-document.html as leaky
>
> TBR=hiroshige@chromium.org
> BUG= 640886 
> NOTRY=true
>
> Committed: https://crrev.com/832983d0321d6d9e821b71e12ca1585354ece470
> Cr-Commit-Position: refs/heads/master@{#414338}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 640886 

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

[modify] https://crrev.com/c54188d25542cc58adaf938e75d10dafc1fc0176/third_party/WebKit/LayoutTests/LeakExpectations

Status: Fixed (was: Assigned)
The original CL was reverted.

Sign in to add a comment