WebRTC Code Reviews
Help | Chromium Project | Sign in
(8508)

Issue 52469004: Remove leaking aecdump testfiles. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 3 months ago by pbos
Modified:
4 years, 3 months ago
Reviewers:
kjellander, Bjorn
CC:
webrtc-reviews_webrtc.org, tterriberry, Bjorn, kwiberg, aluebs, ajm
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Remove leaking aecdump testfiles. Also removes tracing to file in ApmTest because it leads to remaining files. BUG=4258 R=bjornv@webrtc.org, kjellander@webrtc.org Committed: https://chromium.googlesource.com/external/webrtc/+/fade1790a724ccafd46ce9ccd20235f02d6e3d71

Patch Set 1 #

Messages

Total messages: 5 (0 generated)
pbos
PTAL
4 years, 3 months ago (2015-05-12 07:56:45 UTC) #1
Bjorn
lgtm
4 years, 3 months ago (2015-05-12 08:05:04 UTC) #2
kjellander
lgtm
4 years, 3 months ago (2015-05-12 08:43:24 UTC) #3
pbos
Committed patchset #1 (id:1) manually as fade1790a724ccafd46ce9ccd20235f02d6e3d71 (presubmit successful).
4 years, 3 months ago (2015-05-12 08:44:21 UTC) #4
commit-bot
4 years, 3 months ago (2015-05-12 08:44:21 UTC) #5
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/fade1790a724ccafd46ce9ccd20235f02d6e3d71
Cr-Commit-Position: refs/heads/master@{#9175}
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 245c2c2-tainted