| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
TEST=included
BUG=http://crbug.com/134394
Review URL: https://chromiumcodereview.appspot.com/10658037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144623 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Deallocate on the plugin's GetInstanceObject ScriptableObject gets invoked
twice during plugin destruction, typically resulting in a crash, for proxied
plugins that expose a scriptable object. Once by PPP_ClassProxy::OnMsgDeallocate
and once by PluginVarTracker::DidDeleteInstance. This patch removes the object
from the var tracker when it is destroyed in PPP_Class_Proxy.
It also tweaks a unittest to verify Deallocate is invoked only once during the
test.
BUG=133950
TEST=Run gmail, reload page (ctrl-r) and observe for plugin crashes
TEST=Run ppapi_example out-of-process, reload page, observe for plugn crashes
TEST=ppapi_unittests --gtest_filter="PluginVar*"
Review URL: https://chromiumcodereview.appspot.com/10678007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144531 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
provides an implementation for setting crash URLs.
BUG=134176
TEST=Added test case to sanity check API call. Also hooked up to flash and produced a crash and checked that the report included the document url (http://crash/reportdetail?reportid=1719cae3a745f633)
Review URL: https://chromiumcodereview.appspot.com/10681006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144516 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
OSMesa as needed for bots. Add AudioConfig tests to ppapi_uitest
untrusted (NaCl) suite. Tweak gyp for untrusted NaCl build so apps using
TLS links without errors.
Addendum: don't add -mno-tls-use-call to arm build (otherwise pnacl compiler
will fail because it doesn't recognize this option.)
BUG=none
TEST=this is the test
Review URL: https://chromiumcodereview.appspot.com/10682005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144512 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Test file edition.
BUG=none
TEST=none
TBR=remaining owners
Review URL: https://chromiumcodereview.appspot.com/10680013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144464 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
files were orphaned.
Review URL: https://chromiumcodereview.appspot.com/10661044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144279 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should help avoid stuttering on older XP machines, or vista machines
where hardware sample rate != application sample rate. The downside is
audio latency will be higher.
Old value was for 30ms buffers.
New value is for 50ms buffers.
TEST=verify w/ nhu...@adobe.com once in canary
BUG=http://code.google.com/p/chromium/issues/detail?id=133393
Review URL: https://chromiumcodereview.appspot.com/10656027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144258 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is needed to get some of the new extension functions to link properly.
BUG=
TEST=
Review URL: https://chromiumcodereview.appspot.com/10597006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144235 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This updates the flash_topmost example to demonstrate the case when the plugin is within a frame.
BUG=127185
TEST=Manually checked the plugin turns red when covered.
Review URL: https://chromiumcodereview.appspot.com/10572055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144206 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10650008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144204 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For Delegate/Observer-type classes that specify an
interface but do not have any particular lifetime
requirements, make their destructors protected. This is to
allow their interfaces to be implemented safely by
RefCounted types. With public destructors, it's possible to
do "scoped_ptr<Delegate> foo", and then assign a
RefCountedDelegateImpl, which would lead to a double free.
As none of these Delegates actually need public destructors
(ownership of the Delegate* is not transferred during a
function call / class constructor), mark the destructors
protected so that it becomes a compile warning to try to
delete them via the Delegate*.
BUG=123295
TEST=it compiles
Review URL: https://chromiumcodereview.appspot.com/10383262
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144086 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
every process launch
This fixes nacl subprocess creation off the main thread for LoadHelperNaClModule (used by pnacl coordinator).
GetNaclInterface calls GetBrowserInterface, which is required to be called from the main thread. So we call it
once and cache the result for calling off the main thread
(only for starting nacl helper processes).
R=dmichael@chromium.org,jvoung@chromium.org
BUG=(broken nacl/chrome integration bots)
TEST=nacl_integration
Review URL: https://chromiumcodereview.appspot.com/10665036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144083 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
suite, using
OSMesa as needed for bots. Add AudioConfig tests to ppapi_uitest
untrusted (NaCl) suite. Tweak gyp for untrusted NaCl build so apps using
TLS links without errors.
BUG=none
TEST=this is the test
Review URL: https://chromiumcodereview.appspot.com/10591004
TBR=nfullagar@google.com
Review URL: https://chromiumcodereview.appspot.com/10659023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144076 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NotAllowMixedReadWrite starts a Write, then checks that various other calls aren't allowed at the same time. But each of these uses TestCompletionCallback::WaitForResult, which runs a nested message loop. So the Write can complete any time that happens, meaning the later calls won't always fail with PP_ERROR_INPROGRESS.
Also re-enable PPAPINaCl[GLibc|NewLib]Test.FileIO_NotAllowMixedReadWrite
BUG=134442
TEST=
Review URL: https://chromiumcodereview.appspot.com/10660010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144060 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10638007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144032 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
OSMesa as needed for bots. Add AudioConfig tests to ppapi_uitest
untrusted (NaCl) suite. Tweak gyp for untrusted NaCl build so apps using
TLS links without errors.
BUG=none
TEST=this is the test
Review URL: https://chromiumcodereview.appspot.com/10591004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144030 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removed the default case to allow the compiler to yell if any of the enum cases
are not handled.
Move the intended behavior to the outer return statement.
CID_COUNT=1
CID=104330
BUG=none
TEST=none
R=groby
TBR=brettw
Review URL: https://chromiumcodereview.appspot.com/10669013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143978 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10665005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143924 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This also makes scoped_refptr<TrackedCallback> the "new" way to pass completion callbacks in an API. This allows the Enter object to handle checking for blocking callbacks on the main thread to report error, and blocking if on the background thread. This way, interfaces don't have to write any special cases for blocking callbacks.
When built with enable_pepper_threading=1 locally, URLLoader tests all pass for blocking completion callbacks. I haven't updated all tests yet.
BUG=92909
TEST=
Review URL: https://chromiumcodereview.appspot.com/10081020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143806 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=116317
TEST=compiles
Review URL: https://chromiumcodereview.appspot.com/10661017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143805 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=116317
TEST=ppapi, nacl tests, manual testing for experimental IPC proxy.
TBR=brettw@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10661002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143708 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
renderer.
BUG=116317
TEST=ppapi, nacl tests, manual testing for experimental IPC proxy.
Review URL: https://chromiumcodereview.appspot.com/10641016
TBR=bbudge@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10625007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143665 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=116317
TEST=ppapi, nacl tests, manual testing for experimental IPC proxy.
Review URL: https://chromiumcodereview.appspot.com/10641016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143656 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If a test was run with a filter that did not name a valid test, no test would be run and success would be reported. This is bad because it allows a test to be added to ppapi_uitest.cc which does not run and no error is raised.
This also fixes all existing invalid tests.
BUG=
TEST=
Review URL: https://chromiumcodereview.appspot.com/10627012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143655 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
bug=109814
Review URL: https://chromiumcodereview.appspot.com/10634012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143647 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
I think what is happening is that the list is mutated as we iterate over it.
This is actually quite a resonable thing to do: we're deleting objects, and
those objects may hold refs to other objects in our list.
BUG=http://crbug.com/133951
Review URL: https://chromiumcodereview.appspot.com/10633019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143644 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=128236
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10634017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143642 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
renderer.
BUG=116317
TEST=manual
Review URL: https://chromiumcodereview.appspot.com/10214007
TBR=bbudge@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10635014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143491 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The corresponding webkit side patch is at https://bugs.webkit.org/show_bug.cgi?id=89089
BUG=128236
TEST=manually, using the browser-plugin
Review URL: https://chromiumcodereview.appspot.com/10543159
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143486 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=116317
TEST=manual
Review URL: https://chromiumcodereview.appspot.com/10214007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143483 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
These messages are not yet used. They will allow us to route messages directly to a resource implementation in the proxy or the host using information in a common header. The actual content of the message (as interpreted by the specifi resource in the plugin or class in the host) is a nested message.
TEST=none
BUT=none
Review URL: https://chromiumcodereview.appspot.com/10560030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143463 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reverting due to: http://code.google.com/p/chromium/issues/detail?id=133972
This accesses the Chrome printing code to determine the default print settings for the default printer. Ideally we would query these settings every time they are requested, however this cannot be done (see the description in pepper_print_settings_initializer.h). Instead the settings are grabbed once at startup and cached.
BUG=
TEST=Added ppapi test.
Review URL: https://chromiumcodereview.appspot.com/10539171
TBR=raymes@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10633008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143409 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This is upstreaming Gentoo Linux patch.
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10537124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143364 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This accesses the Chrome printing code to determine the default print settings for the default printer. Ideally we would query these settings every time they are requested, however this cannot be done (see the description in pepper_print_settings_initializer.h). Instead the settings are grabbed once at startup and cached.
BUG=
TEST=Added ppapi test.
Review URL: https://chromiumcodereview.appspot.com/10539171
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143294 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is the new name in preparation for deleting the backwards-compatible typedef.
BUG=
TEST=
Review URL: https://chromiumcodereview.appspot.com/10532162
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143282 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also plumb service runtime errors in the translation processes
back to the coordinator explicitly.
R=jvoung@google.com,sehr@google.com,roberm@google.com
BUG=none
TEST=nacl_integration (but only on nacl integration bots, not chromium bots)
Review URL: https://chromiumcodereview.appspot.com/10575025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143232 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
I believe this is caused by a shutdown race condition. My theory is if a pugin
is cread and rapidly destroyed, object creation will fail in the renderer.
TEST=manual
BUG=133581
Review URL: https://chromiumcodereview.appspot.com/10580027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143201 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Defer reporting of translation failure until after translation process
returns, and after temporary file cleanup.
R= jvoung@google.com, sehr@google.com
BUG=none
TEST=(no chromium tests exercise this code yet, only tests in the nacl tree)
Review URL: https://chromiumcodereview.appspot.com/10579009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143014 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also refactor the PPP_Instance version checking so I don't have to write the fall-back from 1.1 to 1.0 yet again.
BUG=116317
TEST=
Review URL: https://chromiumcodereview.appspot.com/10543029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143006 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will be used in unit tests and (possibly in the future) son proxy implementation code. This is basically copied from ScopedPPResource.
TEST=none
BUG=none
Review URL: https://chromiumcodereview.appspot.com/10562033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142982 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=112299
TEST=None
Review URL: https://chromiumcodereview.appspot.com/10541185
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142869 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=this is the test
Review URL: https://chromiumcodereview.appspot.com/10534152
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142867 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10533163
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142830 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
plugin holds a reference beyond the lifetime of the instance.
Review URL: https://chromiumcodereview.appspot.com/10542150
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142787 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If a label is not specified for a source IDL file contianing interfaces
then the version of the interface is assumed to be 1.0 and within the
range of the version being generated. This CL adds a warning that the
interface label is unspecified.
BUG=106504
TBR=sehr@google.com
Review URL: https://chromiumcodereview.appspot.com/10565023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142633 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This patch refactors ppapi_shared.gypi and ppapi_proxy.gypi into proper includes, adds ppapi_shared_untrusted and ppapi_proxy_untrusted .gyp files, and integrates them into the nacl_irt build (ppapi/native_client/native_client.gyp). In order to build without link errors, it includes our plugin side initialization of PluginDispatcher, and a PpapiPluginMain definition. When the 'build_ppapi_ipc_proxy_untrusted' gyp flag is set to '1', this will build a working NaCl IRT using the Chrome IPC proxy.
BUG=116317
TEST=compiles, runs HelloWorld and GetURL SDK examples.
Review URL: https://chromiumcodereview.appspot.com/10565015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142540 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) Move pnacl translation SRPC methods to non-static method, called from thread entry point, rather than using the entry point directly
2) Add a few comments
R= jvoung@chromium.org,robertm@chromium.org,sehr@google.com
BUG=none
TEST=nacl_integration
Review URL: https://chromiumcodereview.appspot.com/10532154
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142527 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=129807
TEST=None
Review URL: https://chromiumcodereview.appspot.com/10534045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142512 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch refactors ppapi_shared.gypi and ppapi_proxy.gypi into proper includes, adds ppapi_shared_untrusted and ppapi_proxy_untrusted .gyp files, and integrates them into the nacl_irt build (ppapi/native_client/native_client.gyp). In order to build without link errors, it includes our plugin side initialization of PluginDispatcher, and a PpapiPluginMain definition. When the 'build_ppapi_ipc_proxy_untrusted' gyp flag is set to '1', this will build a working NaCl IRT using the Chrome IPC proxy.
BUG=116317
TEST=compiles, runs HelloWorld and GetURL SDK examples.
Review URL: https://chromiumcodereview.appspot.com/10546140
TBR=bbudge@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10565012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142494 0039d316-1c4b-4281-b951-d872f2087c98
|