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

Issue 1570063003: clang-format audio_device/mac. (Closed)

Created:
4 years, 11 months ago by Andrew MacDonald
Modified:
4 years, 11 months ago
Reviewers:
henrika_webrtc
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, audio-team_agora.io, sdk-team_agora.io, peah-webrtc
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

clang-format audio_device/mac. NOTRY=true Committed: https://crrev.com/2bc63a1dd3188402f4da1cdb6ed24fc058819304 Cr-Commit-Position: refs/heads/master@{#11212}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3398 lines, -4025 lines) Patch
M webrtc/modules/audio_device/mac/audio_device_mac.h View 3 chunks +284 lines, -288 lines 0 comments Download
M webrtc/modules/audio_device/mac/audio_device_mac.cc View 1 chunk +2184 lines, -2661 lines 0 comments Download
M webrtc/modules/audio_device/mac/audio_mixer_manager_mac.h View 1 chunk +49 lines, -50 lines 0 comments Download
M webrtc/modules/audio_device/mac/audio_mixer_manager_mac.cc View 2 chunks +881 lines, -1026 lines 0 comments Download

Messages

Total messages: 15 (7 generated)
Andrew MacDonald
Assuming you're cool with this, I can do the rest of audio_device/ in a follow-up ...
4 years, 11 months ago (2016-01-08 18:44:02 UTC) #2
henrika_webrtc
Thanks for fixing Andrew. It is tricky to detect any error that might have sneaked ...
4 years, 11 months ago (2016-01-11 08:41:03 UTC) #3
Andrew MacDonald
On 2016/01/11 08:41:03, henrika_webrtc wrote: > Thanks for fixing Andrew. It is tricky to detect ...
4 years, 11 months ago (2016-01-11 19:13:20 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1570063003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1570063003/1
4 years, 11 months ago (2016-01-11 19:13:34 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL) win_baremetal on ...
4 years, 11 months ago (2016-01-11 21:13:53 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1570063003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1570063003/1
4 years, 11 months ago (2016-01-11 23:58:10 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 11 months ago (2016-01-11 23:59:21 UTC) #13
commit-bot: I haz the power
4 years, 11 months ago (2016-01-11 23:59:34 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/2bc63a1dd3188402f4da1cdb6ed24fc058819304
Cr-Commit-Position: refs/heads/master@{#11212}

Powered by Google App Engine
This is Rietveld 408576698