Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Issue 673093002: LeakExpectations: Remove SVG font leak expectations (Closed)

Created:
6 years, 2 months ago by kouhei (in TOK)
Modified:
6 years, 2 months ago
Reviewers:
pdr.
CC:
blink-reviews
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

LeakExpectations: Remove SVG font leak expectations SVG font implementation is removed from Blink. Remove LeakExpectations entries for SVG font tests as they don't leak Documents anymore. BUG=366455, 242735 TBR=pdr Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184326

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -7 lines) Patch
M LayoutTests/LeakExpectations View 1 chunk +0 lines, -7 lines 0 comments Download

Messages

Total messages: 15 (6 generated)
kouhei (in TOK)
6 years, 2 months ago (2014-10-24 01:46:49 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/673093002/1
6 years, 2 months ago (2014-10-24 01:48:49 UTC) #4
pdr.
On 2014/10/24 at 01:48:49, commit-bot wrote: > CQ is trying da patch. Follow status at ...
6 years, 2 months ago (2014-10-24 01:51:06 UTC) #5
kouhei (in TOK)
On 2014/10/24 01:51:06, pdr wrote: > On 2014/10/24 at 01:48:49, commit-bot wrote: > > CQ ...
6 years, 2 months ago (2014-10-24 01:53:00 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/673093002/1
6 years, 2 months ago (2014-10-24 01:53:49 UTC) #9
pdr.
On 2014/10/24 at 01:53:00, kouhei wrote: > On 2014/10/24 01:51:06, pdr wrote: > > On ...
6 years, 2 months ago (2014-10-24 01:53:56 UTC) #10
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_gn_rel on tryserver.blink (http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/builds/14691)
6 years, 2 months ago (2014-10-24 02:08:24 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/673093002/1
6 years, 2 months ago (2014-10-24 03:07:52 UTC) #14
commit-bot: I haz the power
6 years, 2 months ago (2014-10-24 03:56:37 UTC) #15
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 184326

Powered by Google App Engine
This is Rietveld 408576698