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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 123933006: Pepper: Remove ResourceArray_Dev. (Closed)

Created:
6 years, 11 months ago by teravest
Modified:
6 years, 11 months ago
Reviewers:
raymes1, yzshen1, binji
CC:
chromium-reviews, yusukes+watch_chromium.org, raymes+watch_chromium.org, jam, yzshen+watch_chromium.org, joi+watch-content_chromium.org, teravest+watch_chromium.org, darin-cc_chromium.org, Sam Clegg, nfullagar1, piman+watch_chromium.org, noelallen1, binji, ihf+watch_chromium.org
Visibility:
Public.

Description

Pepper: Remove ResourceArray_Dev. It's no longer used. BUG= TBR=binji Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243429

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -723 lines) Patch
M chrome/browser/component_updater/ppapi_utils.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/test/ppapi/ppapi_browsertest.cc View 1 chunk +0 lines, -13 lines 0 comments Download
M content/renderer/pepper/plugin_module.cc View 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/pepper/resource_creation_impl.h View 1 chunk +0 lines, -3 lines 0 comments Download
M content/renderer/pepper/resource_creation_impl.cc View 2 chunks +0 lines, -11 lines 0 comments Download
M native_client_sdk/src/libraries/ppapi/library.dsc View 1 chunk +0 lines, -1 line 0 comments Download
M native_client_sdk/src/libraries/ppapi_cpp/library.dsc View 2 chunks +0 lines, -2 lines 0 comments Download
D ppapi/api/dev/ppb_resource_array_dev.idl View 1 chunk +0 lines, -68 lines 0 comments Download
D ppapi/c/dev/ppb_resource_array_dev.h View 1 chunk +0 lines, -85 lines 0 comments Download
M ppapi/cpp/dev/audio_input_dev.cc View 1 chunk +0 lines, -1 line 0 comments Download
D ppapi/cpp/dev/resource_array_dev.h View 1 chunk +0 lines, -71 lines 0 comments Download
D ppapi/cpp/dev/resource_array_dev.cc View 1 chunk +0 lines, -92 lines 0 comments Download
M ppapi/cpp/dev/video_capture_dev.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ppapi/cpp/file_io.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c View 4 chunks +1 line, -6 lines 0 comments Download
M ppapi/ppapi_shared.gypi View 2 chunks +0 lines, -4 lines 0 comments Download
M ppapi/ppapi_sources.gypi View 3 chunks +0 lines, -5 lines 0 comments Download
M ppapi/proxy/interface_list.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ppapi/proxy/resource_creation_proxy.h View 1 chunk +0 lines, -3 lines 0 comments Download
M ppapi/proxy/resource_creation_proxy.cc View 2 chunks +0 lines, -10 lines 0 comments Download
D ppapi/shared_impl/ppb_resource_array_shared.h View 1 chunk +0 lines, -42 lines 0 comments Download
D ppapi/shared_impl/ppb_resource_array_shared.cc View 1 chunk +0 lines, -51 lines 0 comments Download
M ppapi/shared_impl/resource.h View 1 chunk +0 lines, -1 line 0 comments Download
M ppapi/tests/all_c_includes.h View 1 chunk +0 lines, -1 line 0 comments Download
M ppapi/tests/all_cpp_includes.h View 1 chunk +0 lines, -1 line 0 comments Download
D ppapi/tests/test_resource_array.h View 1 chunk +0 lines, -28 lines 0 comments Download
D ppapi/tests/test_resource_array.cc View 1 chunk +0 lines, -123 lines 0 comments Download
M ppapi/thunk/interfaces_ppb_public_dev.h View 1 chunk +0 lines, -1 line 0 comments Download
D ppapi/thunk/ppb_resource_array_api.h View 1 chunk +0 lines, -25 lines 0 comments Download
D ppapi/thunk/ppb_resource_array_dev_thunk.cc View 1 chunk +0 lines, -67 lines 0 comments Download
M ppapi/thunk/resource_creation_api.h View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
teravest
6 years, 11 months ago (2014-01-07 18:51:14 UTC) #1
yzshen1
LGTM Thanks!
6 years, 11 months ago (2014-01-07 18:59:43 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/teravest@chromium.org/123933006/1
6 years, 11 months ago (2014-01-07 19:25:57 UTC) #3
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=43583
6 years, 11 months ago (2014-01-07 19:53:06 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/teravest@chromium.org/123933006/1
6 years, 11 months ago (2014-01-07 19:57:29 UTC) #5
commit-bot: I haz the power
Change committed as 243429
6 years, 11 months ago (2014-01-07 23:27:58 UTC) #6
raymes1
6 years, 11 months ago (2014-01-07 23:30:49 UTC) #7 
Hooray!

On Wed, Jan 8, 2014 at 10:27 AM,  <commit-bot@chromium.org> wrote:
> Change committed as 243429
>
> https://codereview.chromium.org/123933006/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698