summaryrefslogtreecommitdiffstats
path: root/content/ppapi_plugin/broker_process_dispatcher.cc
Commit message (Collapse)AuthorAgeFilesLines
* Implement ScopedChildProcessReference classkinuko@chromium.org2013-11-121-6/+1
| | | | | | | | | | An innocent attempt. BUG=none Review URL: https://codereview.chromium.org/66443002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234618 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of utf_string_conversions.h in content/.avi@chromium.org2013-06-071-1/+1
| | | | | | | | | | BUG=none TEST=none TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/16408017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204917 0039d316-1c4b-4281-b951-d872f2087c98
* Move a bunch of child-only code from content/common to content/childscottmg@chromium.org2013-06-061-1/+1
| | | | | | | | | | | | Was just trying to move content/common/resource_dispatcher.{cc,h}, but that ends up needing a lot of other things to move. TBR=jam@chromium.org BUG=246357 Review URL: https://codereview.chromium.org/16328003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204569 0039d316-1c4b-4281-b951-d872f2087c98
* Update content/ to use WeakPtr<T>::get() instead of implicit "operator T*"akalin@chromium.org2013-06-031-1/+1
| | | | | | | | | BUG=245942 R=avi@chromium.org Review URL: https://codereview.chromium.org/16256018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203775 0039d316-1c4b-4281-b951-d872f2087c98
* content: Use base::MessageLoop.xhwang@chromium.org2013-05-041-1/+1
| | | | | | | | | BUG=236029 R=avi@chromium.org Review URL: https://chromiumcodereview.appspot.com/14335017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198316 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite scoped_array<T> to scoped_ptr<T[]> in content/, Linux edition.dcheng@chromium.org2013-04-071-1/+1
| | | | | | | | | | This changelist was automatically generated using a clang tool. BUG=171111 Review URL: https://codereview.chromium.org/13749004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192769 0039d316-1c4b-4281-b951-d872f2087c98
* Add explicit base to FilePath.brettw@chromium.org2013-02-071-15/+16
| | | | | | | | This changes the "easy" content subdirectories (everything but browser, renderer, and common) and the ppapi top level directory. Review URL: https://codereview.chromium.org/12208057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181191 0039d316-1c4b-4281-b951-d872f2087c98
* Rename IPC handlers that are in the form of OnMsgFoo to OnFoo. This is how ↵jam@chromium.org2013-01-021-14/+12
| | | | | | | | almost all the handlers are called. Review URL: https://codereview.chromium.org/11695005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174868 0039d316-1c4b-4281-b951-d872f2087c98
* Move content's a plugin, ppapi_plugin, utility, and worker subdirectories to ↵jam@chromium.org2012-10-211-0/+3
| | | | | | | | the content namespace. Review URL: https://codereview.chromium.org/11231016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163210 0039d316-1c4b-4281-b951-d872f2087c98
* Add GetSitesWithData and FreeSiteList methods to PPP_Flash_BrowserOperations ↵bauerb@chromium.org2012-07-271-30/+102
| | | | | | | | | | | | interface and hook them up in PepperFlashSettingsManager. BUG=132409 TEST=none Review URL: https://chromiumcodereview.appspot.com/10825018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148839 0039d316-1c4b-4281-b951-d872f2087c98
* Change PP_Flash_BrowserOperations_SiteSetting.site from PP_Var to const char*.yzshen@chromium.org2012-06-181-24/+17
| | | | | | | | | | | | This change is needed because PP_Var cannot be used in a broker process. This change also adds support for cstr_t in ppapi IDL files. BUG=112190 TEST=None Review URL: https://chromiumcodereview.appspot.com/10566014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142670 0039d316-1c4b-4281-b951-d872f2087c98
* Reland: Pepper Flash settings integration - camera and microphone.yzshen@chromium.org2012-06-121-27/+165
| | | | | | | | | | | | The original code review: http://codereview.chromium.org/10479015/ TBR=csilv@chromium.org, brettw@chromium.org BUG=112190 TEST=None Review URL: https://chromiumcodereview.appspot.com/10541108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141665 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 141482 - Pepper Flash settings integration - camera and microphone.yzshen@chromium.org2012-06-111-165/+27
| | | | | | | | | | | | BUG=112190 TEST=None Review URL: https://chromiumcodereview.appspot.com/10479015 TBR=yzshen@chromium.org Review URL: https://chromiumcodereview.appspot.com/10536103 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141492 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper Flash settings integration - camera and microphone.yzshen@chromium.org2012-06-111-27/+165
| | | | | | | | | BUG=112190 TEST=None Review URL: https://chromiumcodereview.appspot.com/10479015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141482 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper Flash: Make ClearSiteData work with PPP_Flash_BrowserOperations 1.1.viettrungluu@chromium.org2012-06-071-9/+24
| | | | | | | | | | | | It should try the 1.1 interface, and if that's not available, fall back to 1.0. TEST=clear site data still works in Pepper Flash BUG=131395 Review URL: https://chromiumcodereview.appspot.com/10535042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140948 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper Flash settings integration: implement "deauthorize content licenses".yzshen@chromium.org2012-05-271-7/+35
| | | | | | | | | | | | | A few notes about PepperFlashSettingsManager: - It doesn't re-establish a channel for each request. It might seem unnecessary at this point. But that is needed for implementing content settings (camera/mic and peer networking), which requires more interactions with the broker process. - Similarly, the support of multiple in-flight requests isn't very useful for deauthorizing content licenses, but that is useful for content settings, e.g., sending multiple GetPermissionSettings requests for different setting types. BUG=112190 TEST=None Review URL: https://chromiumcodereview.appspot.com/10391173 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139210 0039d316-1c4b-4281-b951-d872f2087c98
* Broker out PPAPI handle duplicationjschuh@chromium.org2012-05-111-3/+1
| | | | | | | BUG=127449 Review URL: https://chromiumcodereview.appspot.com/10378057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136686 0039d316-1c4b-4281-b951-d872f2087c98
* Actually hook up the Pepper clear site data.brettw@chromium.org2012-04-201-1/+49
| | | | | | | | | | | This fixes the message routing. The message is sent over the "broker" channel in this case which normally connects to the renderer. Switching this over to be the main plugin <-> browser channel is extra work since we'd need a way to plumb through the response. Since we already do this weird routing for the NPAPI plugin, doing the same thing is the simplest here. I moved the code to the broker dispatcher which is the endpoint for the channel that's used. This removes the timeout for sending the clear message. This made it challenging to debug since if you load the plugin in the debugger, the message never gets sent. This could legitimately happen if the computer is swapping and is super slow to load the plugin. It seems if you want to clear your data, we shouldn't "forget" to clear it in certain cases. So I just left this code out. Review URL: https://chromiumcodereview.appspot.com/10155001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133277 0039d316-1c4b-4281-b951-d872f2087c98
* Convert use of int ms to TimeDelta in files owned by brettw.tedvessenes@gmail.com2012-01-271-3/+3
| | | | | | | | | | BUG=108171 TEST= Review URL: http://codereview.chromium.org/9233018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119321 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 117824 - Convert use of int ms to TimeDelta in files owned by brettw.hbono@chromium.org2012-01-161-3/+3
| | | | | | | | | | | | | | | | | | | | This change broke Linux and Mac bots. I would recommend to ask committers to run your next change on trybots. (*1) http://chromegw.corp.google.com/i/chromium/builders/Mac/builds/10788/steps/compile/logs/stdio (*2) http://chromegw.corp.google.com/i/chromium/builders/Linux/builds/19313/steps/compile/logs/stdio (*3) http://chromegw.corp.google.com/i/chromium/builders/Linux%20x64/builds/21385/steps/compile/logs/stdio R=brettw@chromium.org BUG=108171 TEST= Review URL: http://codereview.chromium.org/9185026 TBR=tedvessenes@gmail.com Review URL: http://codereview.chromium.org/9215005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117825 0039d316-1c4b-4281-b951-d872f2087c98
* Convert use of int ms to TimeDelta in files owned by brettw.tedvessenes@gmail.com2012-01-161-3/+3
| | | | | | | | | | | R=brettw@chromium.org BUG=108171 TEST= Review URL: http://codereview.chromium.org/9185026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117824 0039d316-1c4b-4281-b951-d872f2087c98
* Remove custom Task implementations from content/ppapi_plugindcheng@chromium.org2011-12-081-10/+7
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/8879003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113555 0039d316-1c4b-4281-b951-d872f2087c98
* Added logging useful for understanding Pepper broker lifetime.ddorwin@chromium.org2011-10-261-5/+7
| | | | | | | | | | | | Also, added support for VLOG to Pepper OOP plugin and Pepper broker processes by forwarding --vmodule to these processes. BUG=none TEST=Launch a dbg build of Chrome with --vmodule=ppapi_plugin_process_host=1,broker_process_dispatcher=1,ppapi_broker_main=1,broker_dispatcher=1,proxy_channel=1", visit a page that uses the broker, and navigate away from that page. The new log statements should be displayed. Review URL: http://codereview.chromium.org/8347009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107451 0039d316-1c4b-4281-b951-d872f2087c98
* Convert the pp::proxy namespace to the ppapi::proxy namespace.brettw@chromium.org2011-08-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is more consistent with the stuff in shared_impl, and removes a lot of namespace using goop. Add a unified resource tracker shared between the proxy and the impl. This renames the old ResourceObjectBase to Resource and removes the old PluginResource. It moves the resource tracker from the impl to the shared_impl, and makes the proxy use it. Some things become a little less neat because there's no proxy resource base class. In particular GetDispatcher() is now gone. I considered whether to add a helper base class that provides this function, but decided against it and had individual resource classes implement this when their implementation would find it useful. This is because ultimately I want more of this functionality to move into the shared_impl, and it's easier to do that if there are fewer proxy-specific things in the resources. This changes the way that plugins are added to the tracker. Previously they would only be in the tracker if the plugin had a reference to them, although they could be alive if the impl had a scoped_ptr referencing an object. This actually has the bug that if we then give the resource back to the plugin, it wouldn't be refcounted properly and everything would get confused. Now the tracker tracks all live resource objects whether or not the plugin has a ref. This works basically like the var tracker (it would be nice if the var and resource trackers shared more code, but that would further complicate this already overcomplicated patch). The resource tracker takes an extra ref whenever the plugin has one or more, and otherwise just tracks live resources. BUG= TEST= Review URL: http://codereview.chromium.org/7655002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97367 0039d316-1c4b-4281-b951-d872f2087c98
* Fix two bugs in proxying of Broker Connect callback and shutdown Broker when ↵ddorwin@chromium.org2011-04-181-1/+1
| | | | | | | | | | | renderer exits. BUG=none TEST=none Review URL: http://codereview.chromium.org/6882020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82017 0039d316-1c4b-4281-b951-d872f2087c98
* Refactored ppapi Dispatcher to share common code between the plugin and ↵ddorwin@chromium.org2011-04-161-0/+38
broker dispatchers. Common code is in DispatcherBase. The base of the dispatcher for plugins remains Dispatcher. The base of the dispatcher for Brokers is BrokerDispatcher. BUG=none TEST=ppapi out-of-process plugins Review URL: http://codereview.chromium.org/6859003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81883 0039d316-1c4b-4281-b951-d872f2087c98