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

Issue 1211873010: Roll chromium_revision fbf756f..3ead4bc (336289:336983) (Closed)

Created:
5 years, 5 months ago by phoglund
Modified:
5 years, 5 months ago
CC:
webrtc-reviews_webrtc.org, qiang.lu, niklas.enbom, yujie_mao (webrtc), Andrew MacDonald
Base URL:
https://chromium.googlesource.com/external/webrtc@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Roll chromium_revision fbf756f..3ead4bc (336289:336983) Relevant changes: * src/third_party/boringssl/src: 7e8f2ba..de24aad * src/third_party/icu: c3f7916..7fe225d * src/third_party/libvpx: db3f347..aa9b5f1 * src/tools/grit: 1f898f7..1dac9ae Details: https://chromium.googlesource.com/chromium/src/+/fbf756f..3ead4bc/DEPS Clang version was not updated in this roll. R=pbos@webrtc.org BUG=chromium:506078 Committed: https://crrev.com/8b80fb64e04b7dc713d50cc1dcadfd6017c6331f Cr-Commit-Position: refs/heads/master@{#9524}

Patch Set 1 #

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

Messages

Total messages: 14 (4 generated)
phoglund
5 years, 5 months ago (2015-07-01 09:14:53 UTC) #2
phoglund
5 years, 5 months ago (2015-07-01 09:15:30 UTC) #4
pbos-webrtc
thx, lgtm, but you're listing the same BUG= twice.
5 years, 5 months ago (2015-07-01 09:17:07 UTC) #5
pbos-webrtc
and might want a chromium: in front of it, but I forget about how this ...
5 years, 5 months ago (2015-07-01 09:22:10 UTC) #6
Adrian Kuegel
LGTM Regarding bugdroid: If I read the config correctly, it would default to webrtc issue ...
5 years, 5 months ago (2015-07-01 09:32:53 UTC) #8
pbos-webrtc
On 2015/07/01 09:32:53, Adrian Kuegel wrote: > LGTM > > Regarding bugdroid: If I read ...
5 years, 5 months ago (2015-07-01 09:34:18 UTC) #9
Adrian Kuegel
On 2015/07/01 09:34:18, pbos-webrtc wrote: > On 2015/07/01 09:32:53, Adrian Kuegel wrote: > > LGTM ...
5 years, 5 months ago (2015-07-01 09:35:36 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1211873010/1
5 years, 5 months ago (2015-07-01 09:42:19 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 5 months ago (2015-07-01 10:46:02 UTC) #13
commit-bot: I haz the power
5 years, 5 months ago (2015-07-01 10:46:11 UTC) #14
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/8b80fb64e04b7dc713d50cc1dcadfd6017c6331f
Cr-Commit-Position: refs/heads/master@{#9524}

Powered by Google App Engine
This is Rietveld 408576698