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

Issue 8890037: Revert 113656 - Implement a proxy for Pepper FileIO. (Closed)

Created:
9 years ago by brettw
Modified:
9 years ago
Reviewers:
brettw
CC:
chromium-reviews, piman+watch_chromium.org, Paweł Hajdan Jr., darin-cc_chromium.org, yzshen+watch_chromium.org, ihf+watch_chromium.org
Visibility:
Public.

Description

Revert 113656 - Implement a proxy for Pepper FileIO. This splits apart the old in-process implementation into a new object in shared_impl that does most of the general tracking. This alllows that code to be shared by the proxy. BUG=http://crbug.com/101154 Review URL: http://codereview.chromium.org/8764003 TBR=brettw@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113659

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+415 lines, -1362 lines) Patch
M chrome/test/ui/ppapi_uitest.cc View 1 chunk +3 lines, -1 line 0 comments Download
M ppapi/api/ppb_file_io.idl View 9 chunks +8 lines, -24 lines 0 comments Download
M ppapi/c/ppb_file_io.h View 9 chunks +9 lines, -25 lines 0 comments Download
MM ppapi/ppapi_proxy.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M ppapi/ppapi_shared.gypi View 2 chunks +0 lines, -4 lines 0 comments Download
M ppapi/proxy/interface_list.cc View 3 chunks +0 lines, -3 lines 0 comments Download
M ppapi/proxy/ppapi_messages.h View 2 chunks +0 lines, -53 lines 0 comments Download
M ppapi/proxy/ppapi_param_traits.cc View 1 chunk +1 line, -2 lines 0 comments Download
D ppapi/proxy/ppb_file_io_proxy.h View 1 chunk +0 lines, -93 lines 0 comments Download
D ppapi/proxy/ppb_file_io_proxy.cc View 1 chunk +0 lines, -442 lines 0 comments Download
M ppapi/proxy/ppb_file_ref_proxy.h View 2 chunks +1 line, -3 lines 0 comments Download
M ppapi/proxy/ppb_file_ref_proxy.cc View 3 chunks +0 lines, -9 lines 0 comments Download
M ppapi/proxy/resource_creation_proxy.cc View 2 chunks +2 lines, -2 lines 0 comments Download
MM ppapi/shared_impl/api_id.h View 1 chunk +0 lines, -1 line 0 comments Download
D ppapi/shared_impl/file_type_conversion.h View 1 chunk +0 lines, -23 lines 0 comments Download
D ppapi/shared_impl/file_type_conversion.cc View 1 chunk +0 lines, -74 lines 0 comments Download
D ppapi/shared_impl/ppb_file_io_shared.h View 1 chunk +0 lines, -157 lines 0 comments Download
D ppapi/shared_impl/ppb_file_io_shared.cc View 1 chunk +0 lines, -237 lines 0 comments Download
M ppapi/tests/test_file_io.cc View 2 chunks +4 lines, -33 lines 0 comments Download
M ppapi/tests/test_url_loader.cc View 2 chunks +35 lines, -25 lines 0 comments Download
M ppapi/thunk/interfaces_ppb_private.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ppapi/thunk/interfaces_ppb_public_stable.h View 2 chunks +2 lines, -2 lines 0 comments Download
MM ppapi/thunk/thunk.h View 2 chunks +2 lines, -0 lines 0 comments Download
M webkit/glue/webkit_glue.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M webkit/plugins/ppapi/file_callbacks.cc View 2 chunks +2 lines, -2 lines 0 comments Download
A + webkit/plugins/ppapi/file_type_conversions.h View 0 chunks +-1 lines, --1 lines 0 comments Download
A + webkit/plugins/ppapi/file_type_conversions.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
M webkit/plugins/ppapi/plugin_module.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M webkit/plugins/ppapi/ppb_file_io_impl.h View 3 chunks +94 lines, -42 lines 0 comments Download
M webkit/plugins/ppapi/ppb_file_io_impl.cc View 9 chunks +239 lines, -90 lines 0 comments Download
M webkit/plugins/ppapi/ppb_flash_file_impl.cc View 11 chunks +11 lines, -13 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
brettw
9 years ago (2011-12-08 21:13:46 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698