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

Issue 962113003: Adding HD peerconnection WebRTC call 45-second performance test. (Closed)

Created:
5 years, 9 months ago by phoglund_chromium
Modified:
5 years, 9 months ago
Reviewers:
sullivan, qyearsley
CC:
chromium-reviews, telemetry-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Adding HD peerconnection WebRTC call 45-second performance test. The purpose is to test CPU usage and vital WebRTC stats over a longer, more taxing call. Also updates all pages since the GitHub location has moved again. BUG=436361 Committed: https://crrev.com/5f2d347f17b795e99cdfe5602d54812461d883de Cr-Commit-Position: refs/heads/master@{#319623}

Patch Set 1 #

Patch Set 2 : Adding page sets #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -14 lines) Patch
M tools/perf/page_sets/data/webrtc_cases.json View 1 1 chunk +10 lines, -0 lines 0 comments Download
A tools/perf/page_sets/data/webrtc_cases_010.wpr.sha1 View 1 1 chunk +1 line, -0 lines 0 comments Download
M tools/perf/page_sets/webrtc_cases.py View 10 chunks +30 lines, -14 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
phoglund_chromium
On 2015/02/27 12:22:21, phoglund wrote: > mailto:phoglund@chromium.org changed reviewers: > + mailto:qyearsley@chromium.org Friendly ping :)
5 years, 9 months ago (2015-03-03 07:46:59 UTC) #2
qyearsley
On 2015/03/03 07:46:59, phoglund wrote: > On 2015/02/27 12:22:21, phoglund wrote: > > mailto:phoglund@chromium.org changed ...
5 years, 9 months ago (2015-03-03 23:12:40 UTC) #3
phoglund_chromium
> > Oh, thanks for the ping, I didn't see this earlier! I didn't spot ...
5 years, 9 months ago (2015-03-09 12:32:55 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/962113003/20001
5 years, 9 months ago (2015-03-09 12:33:18 UTC) #7
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 9 months ago (2015-03-09 13:42:46 UTC) #8
commit-bot: I haz the power
5 years, 9 months ago (2015-03-09 13:44:10 UTC) #9
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/5f2d347f17b795e99cdfe5602d54812461d883de
Cr-Commit-Position: refs/heads/master@{#319623}

Powered by Google App Engine
This is Rietveld 408576698