Issue 304513007: Remove PlatformFile from ppapi

Issue 304513007: Remove PlatformFile from ppapi (Closed)

Created:
6 years, 7 months ago by rvargas (doing something else)

Modified:
6 years, 6 months ago

Reviewers:
dmichael (off chromium), teravest

CC:
chromium-reviews

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

Created: 6 years, 6 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+88 lines, -112 lines)			Patch
M	ppapi/nacl_irt/ppapi_dispatcher.h	View	1 chunk	+1 line, -1 line	0 comments	Download
M	ppapi/proxy/file_io_resource.h	View	7 chunks	+28 lines, -27 lines	0 comments	Download
M	ppapi/proxy/file_io_resource.cc	View	17 chunks	+37 lines, -50 lines	0 comments	Download
M	ppapi/proxy/file_mapping_resource.h	View	1 chunk	+1 line, -1 line	0 comments	Download
M	ppapi/proxy/file_mapping_resource.cc	View	2 chunks	+4 lines, -4 lines	0 comments	Download
M	ppapi/proxy/file_mapping_resource_posix.cc	View	2 chunks	+2 lines, -2 lines	0 comments	Download
M	ppapi/proxy/file_mapping_resource_win.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	ppapi/proxy/ppp_content_decryptor_private_proxy.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	ppapi/proxy/proxy_channel.cc	View	1 chunk	+1 line, -5 lines	0 comments	Download
M	ppapi/proxy/serialized_handle.cc	View	2 chunks	+2 lines, -8 lines	0 comments	Download
M	ppapi/shared_impl/file_type_conversion.cc	View	3 chunks	+9 lines, -10 lines	0 comments	Download
M	ppapi/shared_impl/platform_file.h	View	1 chunk	+1 line, -1 line	0 comments	Download
M	ppapi/shared_impl/var.h	View	1 chunk	+0 lines, -1 line	0 comments	Download

Total messages: 6 (0 generated)

dmichael (off chromium)

teravest@, could you take this one please?

6 years, 6 months ago (2014-06-02 15:56:00 UTC) #2

rvargas (doing something else)

The CQ bit was checked by rvargas@chromium.org

6 years, 6 months ago (2014-06-02 23:28:24 UTC) #4

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rvargas@chromium.org/304513007/40001

6 years, 6 months ago (2014-06-02 23:29:11 UTC) #5

Message was sent while issue was closed.

Change committed as 274377