| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
(NaCl, registration of Chrome plugins like pdf/remoting/flash) behind.
Review URL: http://codereview.chromium.org/6869051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81959 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
of NaCl modules.
Review URL: http://codereview.chromium.org/6821038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81545 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also make --ppapi-flash-in-process accessible and useful on Chrome OS.
BUG=none
TEST=check in about:flags on Chrome OS, go to youtube on Chrome OS, with or
without the flag.
Review URL: http://codereview.chromium.org/6764003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80187 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
force it to run in-process.
TEST=manual
BUG=none
Review URL: http://codereview.chromium.org/6780026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80071 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
out-of-process PPAPI plugins. This mostly just forwards notifications up and
provides a way to get the sad plugin image in the PPAPI rendering code.
I moved the NPAPI sad tab painting to a new file so that it could be shared
between the NPAPI and PPAPI.
This also hooks up the name and version information in the task manager for
PPAPI plugins.
R=jam@chromium.org
Review URL: http://codereview.chromium.org/6685091
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78816 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
to within the renderer.
This works by having the renderer check with the plugin that a PP_Instance is
available before using it. If it's already seen, the renderer will generate a
new PP_Instance and retry.
For performance, this message is handled on the I/O thread of the plugin so it
will not be blocked by the actual plugin code.
This requires an unfortunate amount of plumbing. Since the renderer can't
depend directly on the proxy, we have a new PPB_Proxy function to set the
verification function used to perform this check.
There is also a new plugin dispatcher delegate where I moved some of the global
state to that used to go into the init function. Adding yet another parameter
there seemed unfortunate.
TEST=manual
BUG=74961
Review URL: http://codereview.chromium.org/6628019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77693 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This allows us to properly specify MIME types, extensions, etc.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6650004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77364 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also eliminate stupid and unnecessary conversion of data to a big string, only
to be parsed back.
BUG=none
TEST=everything still works
Review URL: http://codereview.chromium.org/6635009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77215 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
loads plugins before all command line information is available. In particular, the about:flags flag for NaCl isn't loaded yet. The fix is to always load the plugin and have the PluginService check the command line flags and set the WebPluginInfo.enabled field based on that.
TEST=manual
BUG=71989
Review URL: http://codereview.chromium.org/6610005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76779 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
crashed and was still referenced by a plugin in the same renderer, it would try
to re-use the crashed one, which obviously doesn't work very well. This change
removes crashed plugins from the renderer's list.
TEST=Manual: instantiate two tabs sharing the same renderer using a plugin,
kill the plugin, then reload one of the tabs.
BUG=none
Review URL: http://codereview.chromium.org/6594036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76194 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:12139
TEST=run with "--register-pepper-plugins=$FLASH_ROOT/out/$FLASH_TYPE/libpepflashplayer.so#Shockwave Flash#Shockwave Flash 10.1 r103#10.1 r103;application/x-shockwave-flash" and check that the version shows up in about:plugins, and that the flash plugin is correctly attached to the Flash group.%
Review URL: http://codereview.chromium.org/6525040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75166 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
regression from that patch by properly using the command line switch for enabling NaCl.
Review URL: http://codereview.chromium.org/6334141
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73957 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
PluginRegistry and is likely the cause of the startup regressions.
Review URL: http://codereview.chromium.org/6413011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73944 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
static funciton and combines the extra and internal plugins.
Review URL: http://codereview.chromium.org/6334140
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73943 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
the constructor (checking for perf regressions).
Review URL: http://codereview.chromium.org/6312177
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73934 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73933 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
unified (checking for perf regressions).
Review URL: http://codereview.chromium.org/6413009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73932 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
bugfix) to see if it is responsible for the startup performance regression.
TEST=startup perf XP
Review URL: http://codereview.chromium.org/6368120
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73931 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/6250179
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73916 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
ifdef out kRemotingPluginMimeType introduced by commit 499e891bb2de3cd345064ef09a24991c3af984b8 (http://codereview.chromium.org/6246031) to fix Chromium Linux Redux fatal warning "chrome/common/pepper_plugin_registry.cc:30: error: <unnamed>::kRemotingPluginMimeType defined but not used".
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6349073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73671 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
also does some refactoring of the plugin registry to make it more clear and to
avoid recomputilg the plugin list all the time.
Review URL: http://codereview.chromium.org/6246031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73427 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6250050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73252 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CL http://codereview.chromium.org/6162008 broke descriptions for pepper plugins.
This puts it back to the previous state (we don't have a way to separately give
mime type descriptions vs plugin descriptions)
BUG=cros:11298
TEST=Flash on CNN
Review URL: http://codereview.chromium.org/6327019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72569 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
adds PP_Instance to the necessary places in the API to make this possible.
String and Object vars used to be PP_Resources. But it is not practical to
assocaited strings with an instance since then we can't have implicit var
constructors and have to litter every string with an instance. So this changes
vars to use their own tracking system associated with the module (i.e. keeping
the current semantics) and making it no longer a resource. I made the internal
Var IDs 32 bits since Neb is about to land his 64->32 change.
Now it force-deletes resources associated with an instance when that instance
goes away. I added some additional code and tracking in ResourceTracker to do
this. I could then remove the Instance::Observer class since the resource can
use the (now renamed) StoppedTracking to know that it's being deleted in
response to the instance being destroyed.
TEST=ppapi ui tests
BUG=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71544 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Another CL will remove the old MIME type (x-ppapi-nacl-srpc) after the NaCl tree has been updated, to avoid a window where NaCl is broken.
BUG=none
TESTS=manual
Review URL: http://codereview.chromium.org/6117005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71094 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
sites with multiple plugins will load that many child processes, which is bad.
This works by having creators of out-of-process plugins add it to a list in the
PepperPluginRegistry. The items are deleted from this list by a new delegate
implementation for the module to tell the registry that it's being destroyed.
It's a little awkward since the PluginModule can't know about the
PepperPluginRegistry, and all other pepper "delegates" are associated with the
Instance/RenderView, so by definition can't be used when the module is being
torn down since there are no instances.
It might have been nice to have an Add function on the lifetime delegate so
that the module will add itself to the list. But the plugin path name is
currently not available in all code paths, and it seemed messy to add.
TEST=manual
BUG=none
Review URL: http://codereview.chromium.org/6085007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70457 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
webkit/plugins/ppapi/*. This renamed the files and interface implementation
classes from foo.cc/Foo to ppb_foo_impl/PPB_Foo_Impl to match the proxy
ppb_foo_proxy/PPB_Foo_Proxy.
This moves plugin_switches.* from webkit/glue/plugins to webkit/plugins.
Review URL: http://codereview.chromium.org/5828003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69424 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added constants for NaCl.
Register plugin (only if --enable-nacl)
BUG= http://code.google.com/p/nativeclient/issues/detail?id=933
TEST=none
Original patch by Noel Allen.
Review URL: http://codereview.chromium.org/5848001/
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69290 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
things that use webkit but otherwise don't need IPC to include the IPC
directory.
This patch moves the set-up of the proxy into the renderer. I also did a lot
of clean-up of the initialization and it seems much nicer now.
BUG=63684
TEST=manual PPAPI proxy testing
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68567 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
directly. This causes
things that use webkit but otherwise don't need IPC to include the IPC
directory.
This patch moves the set-up of the proxy into the renderer. I also did a lot
of clean-up of the initialization and it seems much nicer now.
BUG=63684
TEST=manual PPAPI proxy testing
Review URL: http://codereview.chromium.org/5592005
TBR=brettw@chromium.org
Review URL: http://codereview.chromium.org/5616004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68486 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
things that use webkit but otherwise don't need IPC to include the IPC
directory.
This patch moves the set-up of the proxy into the renderer. I also did a lot
of clean-up of the initialization and it seems much nicer now.
BUG=63684
TEST=manual PPAPI proxy testing
Review URL: http://codereview.chromium.org/5592005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68482 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Leaking this is better than poking into bad memory on shutdown.
BUG=63234
Review URL: http://codereview.chromium.org/5556002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68045 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since out-of-process tests override the EXE path to look like the bundled app, it makes sense to override AmIBundled() as well.
This is important because the renderer process started from browser_tests runs as bundled, and if browser and renderer process don't agree on bundled-ness, the "load plugin" requests for internal plugins from the renderer have the wrong plugin path, causing the plugin load to fail.
Also add a DCHECK that makes sure that AmIBundled() doesn't flip-flop.
This makes PDFBrowserTest work on mac, so enable it.
It looks like even unit_tests uses the out-of-process test runner, so this change is a bit hairy :-/
BUG=61258,63183
TEST=all existing tests still pass, PDFBrowserTest.* passes.
Review URL: http://codereview.chromium.org/4947002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66156 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
quite simple and just sets up the PPAPI dispatcher and loads the library.
There is a new command line switch --ppapi-out-of-process which runs PPAPI
plugins out of process using the new code path. There is some logic in
RenderView and PepperPluginModule for setting up this connection, but it should
be straightforward.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/3915002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65614 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:7492
TEST=run pepper flash (with LSO support) on http://www.bestflashanimationsite.com/tutorials/4/ , check that it properly stores the objects.
Review URL: http://codereview.chromium.org/3800010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63003 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=trybots
Review URL: http://codereview.chromium.org/3750001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62483 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=58329
Review URL: http://codereview.chromium.org/3618015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61864 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
(~1.3 megs off Debug linux .a files)
BUG=none
TEST=compiles
Review URL: http://codereview.chromium.org/3556013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61587 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
string_split.h
BUG=None
TEST=trybos
Review URL: http://codereview.chromium.org/3447008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60422 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On linux first check succeeded, while next one will fail and no plugin information got returned.
So, I am inverting logic a little bit. IFF check for plugin file succeeded, we omit all
subsequent for this file in the process.
BUG=none
TEST=Check PDF plugin is working in Chrome linux sandbox, and no PDF plugin is in Chromium build.
Review URL: http://codereview.chromium.org/3181003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55784 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We preload plugins before we sandbox, then later attempt to fully
load them once we're sandboxed. We can't ask whether the plugin
exists at the second point because we don't have disk access at
that point. So instead, the first time we're asked about plugins
(before we're sandboxed), record whether the plugin is actually
available and use that to skip loading it in the second pass.
BUG=51546
Review URL: http://codereview.chromium.org/3114005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55609 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/3057033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55176 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Renaming the flag so we don't expose the internal name.
Review URL: http://codereview.chromium.org/3037043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55006 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=Run linux Chrome with sandbox and verify internal PDF viewer displaying PDFs.
Review URL: http://codereview.chromium.org/3081013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54986 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
This reverts commit r54966.
Meant to try it, not commit it, sorry everyone. :(
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54969 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/3057033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54966 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=49702
TEST=Integrated PDF plugin should work on Linux Chrome, and --register-pepper-plugins works on Linux even with sandbox enabled.
Review URL: http://codereview.chromium.org/3031011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53977 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
on the command line. This is necessary to emulate known plugin types with
Pepper since web pages do sniffing on these to detect if the corresponding
plugin is installed.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2985010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52672 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/2891016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52575 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
showing up twice in Linux (not sure why not on Windows).
BUG=48181
Review URL: http://codereview.chromium.org/2947010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52166 0039d316-1c4b-4281-b951-d872f2087c98
|