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

Issue 23503055: [Android] Fixed method name in EME-related code after it was changed. (Closed)

Created:
7 years, 3 months ago by whywhat
Modified:
7 years, 3 months ago
CC:
chromium-reviews, feature-media-reviews_chromium.org
Visibility:
Public.

Description

Fixed method name after it was changed (https://chromiumcodereview.appspot.com/23452025/) Once EME stuff has moved from content/ to chrome/ one method was renamed everywhere except for one place that is only compiled on the official release Android bots (with branding=='Chrome' and OS=='android'). BUG=None TBR=ddorwin NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223332

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/renderer/media/chrome_key_systems.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
whywhat
PTaL
7 years, 3 months ago (2013-09-16 12:19:21 UTC) #1
whywhat
+peter for a quick LGTM +digit since it may break the official build once it's ...
7 years, 3 months ago (2013-09-16 12:28:15 UTC) #2
Peter Beverloo
lgtm, thanks for the clarifications
7 years, 3 months ago (2013-09-16 12:31:42 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avayvod@chromium.org/23503055/1
7 years, 3 months ago (2013-09-16 12:32:42 UTC) #4
commit-bot: I haz the power
Change committed as 223332
7 years, 3 months ago (2013-09-16 12:40:06 UTC) #5
ddorwin
7 years, 3 months ago (2013-09-16 18:42:27 UTC) #6
Message was sent while issue was closed.
lgtm

Thank you!

Powered by Google App Engine
This is Rietveld 408576698