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

Issue 2566603003: Convert seconds_since_epoch to integer. (Closed)

Created:
4 years ago by ehmaldonado_webrtc
Modified:
4 years ago
CC:
webrtc-reviews_webrtc.org
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Convert seconds_since_epoch to integer. It looks like the 400 errors when trying to upload to test-results.appspot.com is because the server expects an integer instead of a float. BUG=chromium:325726 TBR=kjellander@webrtc.org, kwiberg@webrtc.org, pbos@webrtc.org NOTRY=True Committed: https://crrev.com/0e738d4fb2f51cc82a90d1cd43de46de4b300abe Cr-Commit-Position: refs/heads/master@{#15504}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M third_party/gtest-parallel/gtest-parallel View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 15 (10 generated)
ehmaldonado_webrtc
I want to try this before submitting a pull request in GitHub, so I know ...
4 years ago (2016-12-09 11:44:21 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2566603003/1
4 years ago (2016-12-09 11:45:44 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-09 11:47:12 UTC) #12
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/0e738d4fb2f51cc82a90d1cd43de46de4b300abe Cr-Commit-Position: refs/heads/master@{#15504}
4 years ago (2016-12-09 11:47:24 UTC) #14
kjellander_webrtc
4 years ago (2016-12-09 11:55:49 UTC) #15
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698