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

Issue 23621019: Pepper API for output protection. (Closed)

Created:
7 years, 3 months ago by kcwu
Modified:
7 years, 3 months ago
CC:
chromium-reviews, yusukes+watch_chromium.org, raymes+watch_chromium.org, yzshen+watch_chromium.org, Sam Clegg, piman+watch_chromium.org, binji, ihf+watch_chromium.org, Pawel Osciak, Sean Paul, tinskip1, wuchengli
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Pepper API for output protection. This is only interface part. Implementation will be in another CL. R=binji@chromium.org, dalecurtis@chromium.org, ddorwin@chromium.org, dmichael@chromium.org BUG=256538 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221149

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+428 lines, -2 lines) Patch
M native_client_sdk/src/libraries/ppapi_cpp_private/library.dsc View 3 chunks +3 lines, -0 lines 0 comments Download
A ppapi/api/private/ppb_output_protection_private.idl View 1 chunk +137 lines, -0 lines 0 comments Download
M ppapi/c/pp_macros.h View 1 chunk +2 lines, -2 lines 0 comments Download
A ppapi/c/private/ppb_output_protection_private.h View 1 chunk +168 lines, -0 lines 0 comments Download
M ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c View 6 chunks +40 lines, -0 lines 0 comments Download
A ppapi/thunk/ppb_output_protection_private_thunk.cc View 1 chunk +78 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
kcwu
This was reviewed at https://codereview.chromium.org/21840002/ binji@ for native_client_sdk/ and other reviewers for ppapi/api/private/ppb_output_protection_private.idl. Other files ...
7 years, 3 months ago (2013-09-02 10:54:38 UTC) #1
dmichael (off chromium)
lgtm
7 years, 3 months ago (2013-09-03 16:28:18 UTC) #2
DaleCurtis
lgtm
7 years, 3 months ago (2013-09-03 19:29:58 UTC) #3
binji
lgtm
7 years, 3 months ago (2013-09-04 00:41:17 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kcwu@chromium.org/23621019/1
7 years, 3 months ago (2013-09-04 01:39:14 UTC) #5
tinskip
lgtm
7 years, 3 months ago (2013-09-04 01:44:50 UTC) #6
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) app_list_unittests, ash_unittests, aura_unittests, base_unittests, browser_tests, cacheinvalidation_unittests, ...
7 years, 3 months ago (2013-09-04 02:16:08 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kcwu@chromium.org/23621019/1
7 years, 3 months ago (2013-09-04 02:32:53 UTC) #8
commit-bot: I haz the power
7 years, 3 months ago (2013-09-04 04:40:41 UTC) #9
Message was sent while issue was closed.
Change committed as 221149

Powered by Google App Engine
This is Rietveld 408576698