summaryrefslogtreecommitdiffstats
path: root/ppapi/proxy
Commit message (Collapse)AuthorAgeFilesLines
* Switched proxy for TCPServerSocketPrivate.ygorshenin@chromium.org2013-08-057-298/+227
| | | | | | | | | BUG=230784 TEST=browser_tests:*TCPServerSocketPrivate Review URL: https://chromiumcodereview.appspot.com/19005006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215597 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: GetInfoForRenderer for N FileRefs.teravest@chromium.org2013-08-021-6/+8
| | | | | | | | | | | | | | | | Currently, operations to support AppendFileRefToBody in the renderer send a sync message to the browser for each file ref. This change alters GetInfoForRenderer (for the FileRef change) to take a vector of FileRefs and return a vector of results back, reducing the number of IPC round trips. TBR=jschuh BUG=225441 Review URL: https://chromiumcodereview.appspot.com/21891002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215356 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 214420 "Do PPB_FileIO Query and Read in the plugin process."bbudge@chromium.org2013-08-013-152/+78
| | | | | | | | | | | | | | | | | | | | | | | | This causes a hang if the plugin passes a blocking callback. We should figure out why the tests don't exercise this case. > Do PPB_FileIO Query and Read in the plugin process. > This changes Open to send a file descriptor to the plugin process on Open. > The file descriptor is used on the plugin side to implement Query and Read. > The implementation reuses base::FileUtilProxy to use a file thread on the > plugin side to call potentially blocking file functions. > > TBR=dmichael@chromium.org, jschuh@chromium.org > > BUG=194304 > TEST=browser_tests, PPAPITest.FileIO > > Review URL: https://codereview.chromium.org/21016004 TBR=bbudge@chromium.org Review URL: https://codereview.chromium.org/20820003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214995 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Use browser and renderer for FileSystem.teravest@chromium.org2013-08-012-2/+17
| | | | | | | | | | | | FileSystem resources have hosts checked in for the browser and renderer. Today, we only use the renderer resource host, but to support FileRef in the browser, we need to use the resource host in the browser as well. BUG=225441 Review URL: https://chromiumcodereview.appspot.com/18031018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214946 0039d316-1c4b-4281-b951-d872f2087c98
* PPAPI: Move IMEInputEvent and TextInput to stable.nona@chromium.org2013-08-011-0/+1
| | | | | | | | | | | | To be able to work with current ppapi Flash, this CL keeps ABI of "PPB_IMEInputEvent_Dev_0_2", "PPB_TextInput_Dev_0_2" and "PPP_TextInput_Dev_0_2". BUG=None TEST=Manually checked that ppapi_example_ime works and also works with current Flash. NOTRY=True Review URL: https://chromiumcodereview.appspot.com/18671004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214878 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Hook up the in-process router for browser.teravest@chromium.org2013-07-312-2/+21
| | | | | | | | | | | | | | | | This is mostly stolen from raymes' change at: https://codereview.chromium.org/15774020/ but cleaned up to apply to master. It also adds in-process variants of the ResourceCall and ResourceReply messages, which are needed because the reply message from the browser to the renderer must be properly routed. This change has been tested (along with a series of other changes) and was shown to work for in-process plugins in the "new" resource proxy. TBR=jochen BUG=225441 Review URL: https://chromiumcodereview.appspot.com/19238007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214828 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper Message Filters: Port to use explicit permission grants in ↵tommycli@chromium.org2013-07-312-4/+3
| | | | | | | | | | | | ChildProcessSecurityPolicy. This ports the Pepper message filters to use explicit permission grants instead of the base::PlatformFile bitmasks which are now deprecated. In the case that we have to deal with Pepper open flags, we now directly translate Pepper open flags to a series of explicit permissions checks instead of converting to base::PlatformFile bitmasks. BUG=262142 Review URL: https://chromiumcodereview.appspot.com/19723010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214581 0039d316-1c4b-4281-b951-d872f2087c98
* Do PPB_FileIO Query and Read in the plugin process.bbudge@chromium.org2013-07-303-78/+152
| | | | | | | | | | | | | | | | This changes Open to send a file descriptor to the plugin process on Open. The file descriptor is used on the plugin side to implement Query and Read. The implementation reuses base::FileUtilProxy to use a file thread on the plugin side to call potentially blocking file functions. TBR=dmichael@chromium.org, jschuh@chromium.org BUG=194304 TEST=browser_tests, PPAPITest.FileIO Review URL: https://codereview.chromium.org/21016004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214420 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 213933 "Move PPB_FileIO Query and Read implementation fro..."dewittj@chromium.org2013-07-263-152/+78
| | | | | | | | | | | | | | | | | | | Failures of PPAPITest.FileIO on Linux Tests (dbg) > Move PPB_FileIO Query and Read implementation from the renderer to the plugin process. > This sends the file descriptor to the plugin side and moves some FileIO > implementation from the renderer to the plugin, using that file descriptor. > > BUG=194304 > R=dmichael@chromium.org, sky@chromium.org, teravest@chromium.org > > Review URL: https://codereview.chromium.org/18063005 TBR=bbudge@chromium.org Review URL: https://codereview.chromium.org/20816002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213976 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_FileIO Query and Read implementation from the renderer to the ↵bbudge@chromium.org2013-07-263-78/+152
| | | | | | | | | | | | | | plugin process. This sends the file descriptor to the plugin side and moves some FileIO implementation from the renderer to the plugin, using that file descriptor. BUG=194304 R=dmichael@chromium.org, sky@chromium.org, teravest@chromium.org Review URL: https://codereview.chromium.org/18063005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213933 0039d316-1c4b-4281-b951-d872f2087c98
* NaCl CompletionCallback is called twice in WebSocket::ReceiveMessage and ↵yhirano@chromium.org2013-07-261-6/+17
| | | | | | | | | | | | | | | | | | crashes the NaCl plugin Some async APIs such as ReceiveMessage takes a completion callback which is called when the operation completes. A completion callback must be called exactly once. If a user calls an async API from non-main message loop, the completion callback will be post to the message loop from the main message loop. But there was no way to detect whether PostRun of the completion callback was called and it caused repetitive calls of PostRun, which leads to an assertion failure and process crash. We fix the issue in this CL. BUG=258018 R=dmichael Review URL: https://chromiumcodereview.appspot.com/19463005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213804 0039d316-1c4b-4281-b951-d872f2087c98
* PAPI: Fix bug in RunWhileLocked, add support for paramsdmichael@chromium.org2013-07-251-0/+1
| | | | | | | | | | | | | | | This CL: - Fixes a bug whereby a Callback bound by RunWhileLocked would release the lock before releasing bound values. This could lead to de-refing and possibly destroying objects (like Resources) that should only be manipulated with the ProxyLock held. - Adds support for up to 3 parameters to RunWhileLocked. Should be easy to extend to more if/when we want. - Adds a unit test that checks the bug and tests other utilities from ProxyLock. There is still possibly a rare issue with the Callback being destroyed without being run, e.g. at process shutdown. I'm going to clean that up in a separate CL: https://codereview.chromium.org/19492014/ BUG= R=bbudge@chromium.org, teravest@chromium.org, yzshen@chromium.org Review URL: https://codereview.chromium.org/19678028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213693 0039d316-1c4b-4281-b951-d872f2087c98
* move BrowserFontResource_Trusted to content/childscottmg@chromium.org2013-07-259-519/+58
| | | | | | | | | | | | | | | Needed to break dependency of ppapi_proxy on webkit code. resource_creation_proxy now creates BrowserFontResource_Trusted by going through PluginGlobals -> PluginProxyDelegate implemented in content by PpapiThread. R=jam@chromium.org, yzshen@chromium.org BUG=237249 Review URL: https://codereview.chromium.org/20214004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213684 0039d316-1c4b-4281-b951-d872f2087c98
* Update include paths in ppapi/proxy for base/process changes.rsesek@chromium.org2013-07-258-8/+5
| | | | | | | | | BUG=242290 R=jschuh@chromium.org, piman@chromium.org Review URL: https://codereview.chromium.org/20058002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213627 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Add routing ID information to Connection.teravest@chromium.org2013-07-231-2/+20
| | | | | | | | | | | | | To support in-process plugins, ResourceCall messages from the renderer to the browser will need to provide routing ID information for receiving a reply. We add a boolean for "in_process" as well so the plugin need only send the extra information when the plugin is running in-process. BUG= Review URL: https://chromiumcodereview.appspot.com/19704009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213223 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: FileRef_CreateExternal from renderer.teravest@chromium.org2013-07-221-0/+7
| | | | | | | | | | | | | | | This change defines FileRef_CreateExternal and allows FileRefs for paths on external file systems to be created by the renderer (but not the plugin). We can't use the permissions checking in PpapiHost because there's only one instance of those per plugin process (and the incoming renderer message would only see the plugin's permissions). BUG=225441 Review URL: https://chromiumcodereview.appspot.com/19356006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212963 0039d316-1c4b-4281-b951-d872f2087c98
* Fix bug caught by clang's -Wlogical-not-parenthesesthakis@chromium.org2013-07-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | ../../ppapi/proxy/ppb_image_data_proxy.cc:153:9: error: logical not is only applied to the left hand side of this comparison [-Werror,-Wlogical-not-parentheses] if (!images_[i].image->type() == type) ^ ~~ ../../ppapi/proxy/ppb_image_data_proxy.cc:153:9: note: add parentheses after the '!' to evaluate the comparison first if (!images_[i].image->type() == type) ^ ( ) ../../ppapi/proxy/ppb_image_data_proxy.cc:153:9: note: add parentheses around left hand side expression to silence this warning if (!images_[i].image->type() == type) ^ ( ) 1 error generated. (Since PPB_ImageData_Shared::ImageDataType currently has only two values, this is no behavior change.) BUG=262248 R=yzshen@chromium.org Review URL: https://codereview.chromium.org/19954006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212908 0039d316-1c4b-4281-b951-d872f2087c98
* NaCl IRT: Remove now-unneeded IrtInit() definition and callmseaborn@chromium.org2013-07-211-16/+0
| | | | | | | | | | | | | | | | | | | | | It used to be that this Chrome-provided function was called from the NaCl side, but reportedly only in the x86 build and not the ARM build. It is now consistently defined on the NaCl side and called there, so we can remove the Chrome side's duplicated definition and call. Also remove the unnecessary NaClSrpcModuleFini() call: We never really need to finalise modules like this when exiting. NaClSrpcModuleFini() might only have the effect of breaking get_random_bytes() if it's used after the PPAPI event loop exits (though I don't think we ever really expect the PPAPI event loop to exit). BUG=https://code.google.com/p/nativeclient/issues/detail?id=3186 TEST=browser_tests, nacl_integration Review URL: https://codereview.chromium.org/19479005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212824 0039d316-1c4b-4281-b951-d872f2087c98
* Replace third_party/icu/public with third_party/icu/source in the include ↵jshin@chromium.org2013-07-182-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | directives. This is to move back icu headers in third_party/icu/public/{i18n,common}/unicode back to their upstream locations in third_party/icu/source/{i18n,common}/unicode. http://codereview.chromium.org/18836004 has is a CL to move ICU header files. Roll ICU to the version with the above ICU CL (http://crrev.com/211851 ). In addition to the global replacement of third_party/icu/public with third_party/icu/source, the top-level DEPS, DEPS in printing and chrome/ are tightened up. (the latter two were too permissive (it used to allow any header from third_party/icu). Besides, sync '-foo' list in ios/public/DEPS with '+foo' in the top-level DEPS and build/linux/unbundled/icu.gyp is updated. BUG=251433 TEST=Compile succeeds on all bots. checkdeps.py does not find any error. TBR=brettw,sky,wtc Review URL: https://chromiumcodereview.appspot.com/18252003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212324 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of the message_loop header in ppapi/, printing/, rlz/, ↵avi@chromium.org2013-07-1812-18/+18
| | | | | | | | | | | | sync/. BUG=260807 TEST=none TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/19627002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212193 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of the shared_memory header in media/, ppapi/, webkit/.avi@chromium.org2013-07-1714-15/+15
| | | | | | | | | | BUG=260807 TEST=none TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/19284015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212116 0039d316-1c4b-4281-b951-d872f2087c98
* Add explicit "ppapi/" to #include in ppapi/proxymseaborn@chromium.org2013-07-171-1/+1
| | | | | | | | | | | | | | The Chrome style is to use the full path of the header file. This cleanup should let us remove "ppapi" from the header file search path. Having "ppapi/" be implicit is left over from when these source files were moved from the NaCl repo into the Chrome repo. BUG=https://code.google.com/p/nativeclient/issues/detail?id=2500 TEST=build Review URL: https://codereview.chromium.org/19601004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212108 0039d316-1c4b-4281-b951-d872f2087c98
* Remove myself from various OWNERS files.viettrungluu@chromium.org2013-07-161-1/+0
| | | | | | | | | | NOTRY=true BUG=none R=brettw@chromium.org Review URL: https://codereview.chromium.org/19218003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211841 0039d316-1c4b-4281-b951-d872f2087c98
* add missing headers in ppapi/*mostynb@opera.com2013-07-155-0/+5
| | | | | | | | | | Include a few headers that are required but not already included directly. BUG=259043 Review URL: https://chromiumcodereview.appspot.com/18209022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211724 0039d316-1c4b-4281-b951-d872f2087c98
* Do not forget to detach message loop from thread in ppapi_unittests.earthdok@chromium.org2013-07-151-0/+1
| | | | | | | | | | | Fixes a test-only memory leak. BUG=258132 R=viettrungluu@chromium.org, dmichael@chromium.org Review URL: https://chromiumcodereview.appspot.com/19197003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211687 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Fix typo in FileRefResource.teravest@chromium.org2013-07-111-2/+2
| | | | | | | | | | | | A logic check in FileRefResource checks the wrong field when determining if the file system information in FileRef_CreateInfo is correct. I came across this issue when putting the FileRef changes back together after much refactoring. BUG= Review URL: https://chromiumcodereview.appspot.com/18093006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211241 0039d316-1c4b-4281-b951-d872f2087c98
* Add GetFileThreadMessageLoop method to PpapiGlobals.bbudge@chromium.org2013-07-112-0/+20
| | | | | | | | | | | This is needed to implement performance critical FileIO functions on the plugin side. BUG=194304 Review URL: https://chromiumcodereview.appspot.com/18144004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211019 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Route CreateResourceHostFromHostReply.teravest@chromium.org2013-07-091-8/+8
| | | | | | | | | | | | | | | Reply messages from the browser to the renderer must be routed so that they end up in the correct RenderView (and eventually the PepperBrowserConnection object). This change makes PpapiHostMsg_CreateResourceHostFromHostReply consistent with PpapiHostMsg_FileRef_GetInfoForRendererReply. BUG=225441 Review URL: https://chromiumcodereview.appspot.com/18826011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210665 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Add FileRef messages for path informationteravest@chromium.org2013-07-081-0/+23
| | | | | | | | | | | | Both FileIO and URLRequestInfo need to get path information for a FileRef resource (while in the renderer). This will introduce an extra round-trip to the browser in both paths. It's possible that we'll do something more clever for FileIO (to go back to only one round trip), but I think we will need this for URLRequestInfo regardless. BUG=225441 Review URL: https://chromiumcodereview.appspot.com/16975012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210394 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Make FileRefResource take a FileSystem refteravest@chromium.org2013-07-032-1/+8
| | | | | | | | | | | | FileRefResource needs to hold a reference on the FileSystem resource that it's passed at creation. This change ensures that happens. R=yzshen1 BUG=225441 Review URL: https://chromiumcodereview.appspot.com/18389003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210067 0039d316-1c4b-4281-b951-d872f2087c98
* Add a new parameter |latency| to PPB_AudioInput_Callback.yzshen@chromium.org2013-07-032-41/+93
| | | | | | | | | | | This CL updates the version of PPB_AudioInput_Dev and PPB_AudioInput_Callback; it also adds a new type PP_TimeDelta. BUG=240900 TEST=None Review URL: https://chromiumcodereview.appspot.com/14619033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209965 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Fix display name for internal FileRefs.teravest@chromium.org2013-07-012-8/+9
| | | | | | | | | | | | | There was a bug where internal FileRefs would use the given display name instead of determining it from the internal filesystem. This fixes that problem. R=yzshen BUG= Review URL: https://chromiumcodereview.appspot.com/18211002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209451 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of time headers in ppapi/, printing/, remoting/.avi@chromium.org2013-06-283-3/+3
| | | | | | | | | | BUG=254986 TEST=none TBR=ben@chromium.org Review URL: https://codereview.chromium.org/18052008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209151 0039d316-1c4b-4281-b951-d872f2087c98
* Update Linux to use scoped_refptr<T>::get() rather than implicit "operator T*"rsleevi@chromium.org2013-06-272-2/+3
| | | | | | | | | | | | Fixes a number of cases either not caught by the automated tool or that have recently regressed. BUG=206189 TBR=darin Review URL: https://chromiumcodereview.appspot.com/17948002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208882 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Initial proxy support for FileRef refactor.teravest@chromium.org2013-06-263-1/+379
| | | | | | | | | | | | | | The proxy doesn't currently extend PPB_FileRef_API because the this class and the current implementation return different types for GetCreateInfo(). This change was tested (as part of a larger local change) and passed tests in FileRef. BUG=225441 Review URL: https://chromiumcodereview.appspot.com/16225007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208613 0039d316-1c4b-4281-b951-d872f2087c98
* PPAPI: Move PPB_NetworkProxy out of Devdmichael@chromium.org2013-06-251-1/+1
| | | | | | | | | | | | This depends on the following CLs: https://codereview.chromium.org/17094022/ https://codereview.chromium.org/16819002/ BUG=247225 Review URL: https://chromiumcodereview.appspot.com/17004010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208513 0039d316-1c4b-4281-b951-d872f2087c98
* Add "PRIVATE" to the enum names of some private Pepper networking APIs:yzshen@chromium.org2013-06-243-9/+10
| | | | | | | | | | | | | | | | | | | | | | | | - ppb_net_address_private.idl - ppb_host_resolver_private.idl - ppb_tcp_socket_private.idl - ppb_udp_socket_private.idl Currently, we have very similar enum names in the private and public Pepper networking APIs (for example, PP_NETADDRESSFAMILY_IPV4 and PP_NETADDRESS_FAMILY_IPV4). The purpose of this CL is to make the distinction more clear. This CL doesn't break existing plugins compiled with old private networking enums (without "PRIVATE" in the names). They will still run on newer versions of Chrome. The only consequence is that they will get compile errors when they compile with new SDK (with new enum names). But the errors should be very straightforward to fix. BUG=253583 TEST=None R=bbudge@chromium.org, tsepez@chromium.org, wez@chromium.org Review URL: https://codereview.chromium.org/17615004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208291 0039d316-1c4b-4281-b951-d872f2087c98
* PPAPI: Introduce PPB_NetworkProxy_Devdmichael@chromium.org2013-06-245-3/+130
| | | | | | | | | | | | | | | This adds support for PPB_NetworkProxy_Dev for out-of-process and NaCl (when using --enable-nacl). Work still to do in future CLs: - Add permissions checking - Translate appropriate error codes from net_errors.h BUG=247225 R=jam@chromium.org, palmer@chromium.org, yzshen@chromium.org Review URL: https://codereview.chromium.org/16819002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208259 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_HostResolver out of dev.yzshen@chromium.org2013-06-243-5/+5
| | | | | | | | | BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/17291015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208147 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_UDPSocket out of dev.yzshen@chromium.org2013-06-247-11/+11
| | | | | | | | | BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/17094015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208145 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_TCPSocket out of dev.yzshen@chromium.org2013-06-224-10/+10
| | | | | | | | | BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/17314012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208094 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_NetAddress out of dev.yzshen@chromium.org2013-06-225-16/+16
| | | | | | | | | BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/17419008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208080 0039d316-1c4b-4281-b951-d872f2087c98
* TCPSocket/UDPSocket: don't update internal state or access user buffers ↵yzshen@chromium.org2013-06-221-9/+17
| | | | | | | | | | | | | after Close(). The CL also removes some NOTREACHED()s which could be reached in normal (but rare) cases. BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/17247004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208029 0039d316-1c4b-4281-b951-d872f2087c98
* Convert most of extensions and some other random stuff to using the base ↵brettw@chromium.org2013-06-211-2/+2
| | | | | | | | | | namespace for Values. R=finnur@chromium.org Review URL: https://codereview.chromium.org/16915006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207937 0039d316-1c4b-4281-b951-d872f2087c98
* Define a LoggingSettings struct to use for InitLogging()akalin@chromium.org2013-06-211-6/+3
| | | | | | | | | | | | | | | | | | Update all callers of InitLogging() to use LoggingSettings, only setting fields that need a non-default value. Turn LoggingDestination enum into a bit field and define add LOG_DEFAULT and LOG_ALL constants. Fix erroneous comment saying that the default was to not lock the log file. BUG=247594 TBR=brettw@chromium.org, cpu@chromium.org, gene@chromium.org, jam@chromium.org, rch@chromium.org, scherkus@chromium.org, sergeyu@chromium.org, sky@chromium.org, tkent@chromium.org, yfriedman@chromium.org, zea@chromium.org Review URL: https://codereview.chromium.org/16519003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207920 0039d316-1c4b-4281-b951-d872f2087c98
* Implement PPB_TCPSocket_Dev: part 2yzshen@chromium.org2013-06-205-67/+68
| | | | | | | | | | | | | | This CL: - adds apps permission check; - supports TCP socket options that PPB_TCPSocket_Private doesn't support; - map net::Error to PP_Error. BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/17291003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207567 0039d316-1c4b-4281-b951-d872f2087c98
* Remove hand written code from autogenerated Pepper ImageData thunk.bbudge@chromium.org2013-06-203-9/+44
| | | | | | | | | | | | | Autogenerate ppb_image_data_thunk.cc and make changes to resource creation API to allow Simple and PlatformImageData resources to be specified. BUG=none TEST=browser_tests, --gtest_filter="PPAPINaClPNaClTest.ImageData" Review URL: https://chromiumcodereview.appspot.com/17286011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207384 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_VarArray and PPB_VarDictionary out of dev.raymes@chromium.org2013-06-201-2/+2
| | | | | | | | | | | | Note that this completely removes the dev versions of the interfaces. The discussion on moving these interfaces to stable happened here: https://codereview.chromium.org/16136009 BUG=236958 Review URL: https://chromiumcodereview.appspot.com/17005006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207369 0039d316-1c4b-4281-b951-d872f2087c98
* Implement PPB_HostResolver_Dev: part 2yzshen@chromium.org2013-06-205-13/+51
| | | | | | | | | | | | | | | This CL: - adds apps permission check; - map net::Error to PP_Error. BUG=247225 TEST=None TBR=brettw@chromium.org (TBR Brett because of renaming two files in content/content_browser.gypi) Review URL: https://chromiumcodereview.appspot.com/16933003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207366 0039d316-1c4b-4281-b951-d872f2087c98
* Remove PP_HOSTRESOLVER_FLAGS_LOOPBACK_ONLY from PPB_HostResolver_Dev.yzshen@chromium.org2013-06-201-2/+0
| | | | | | | | | | | This was first introduced by PPB_HostResolver_Private. It mirrored net::HOST_RESOLVER_LOOPBACK_ONLY. However, it is actually a flag used by the host resolver implementation internally. It doesn't make sense to define this flag, because there is no way for a plugin to detect that only loopback addresses are configured and provide this information to the Pepper API. BUG=247225 TEST=None Review URL: https://chromiumcodereview.appspot.com/16831011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207354 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-06 10:41:59 +0000