summaryrefslogtreecommitdiffstats
path: root/ppapi
Commit message (Collapse)AuthorAgeFilesLines
* Pepper: Remove program_url in trusted plugin.teravest@chromium.org2014-07-035-16/+10
| | | | | | | | | | This reduces the amount of state in the trusted plugin (slightly). BUG=239656 Review URL: https://codereview.chromium.org/367153004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281127 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for strong future sync points.jbauman@chromium.org2014-07-016-0/+60
| | | | | | | | | | If InsertFutureSyncPoint() is used (this is only allowed from the browser process), the sync point will not be retired automatically but can be retired later manually with RetireSyncPoint. Any command buffer that waits on it before it's retired can wait indefinitely. BUG=365454 Review URL: https://codereview.chromium.org/284233008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280708 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: SelLdrLauncherChrome cleanup.teravest@chromium.org2014-06-304-60/+32
| | | | | | | | | | | | The indirection provided by SelLdrLauncherChrome::Start() isn't much help. Instead, we just expose the channel so that ServiceRuntime can pass it to LaunchSelLdr directly. BUG=239656 Review URL: https://codereview.chromium.org/360743002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280686 0039d316-1c4b-4281-b951-d872f2087c98
* Enable mmap and identity-based validation caching on pnacl-{llc,ld}.nexejvoung@chromium.org2014-06-308-72/+71
| | | | | | | | | | | | | | | Register the executables w/ the validation cache on open and hook up the file info just like the other NaCl module loads. The non-executable files are still opened without the windows-specific FLAG_EXEC. BUG= https://code.google.com/p/nativeclient/issues/detail?id=3614 TEST= ValidationCacheOfTranslatorNexes Review URL: https://codereview.chromium.org/356923004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280605 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Remove PPP_ManifestService.teravest@chromium.org2014-06-276-132/+17
| | | | | | | | | | | | Now that enough logic has been moved out of the trusted plugin, we can get rid of PPP_ManifestService and make the code in the Non-SFI case much clearer. CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 BUG=239656 Review URL: https://codereview.chromium.org/356053004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280204 0039d316-1c4b-4281-b951-d872f2087c98
* NaCl: begin reducing ServiceRuntime's dependance on Plugin in the trusted ↵ncbray@chromium.org2014-06-273-57/+57
| | | | | | | | | | plugin. BUG= Review URL: https://codereview.chromium.org/342553004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280197 0039d316-1c4b-4281-b951-d872f2087c98
* Add preliminary support for UBsan's vptr in Chromium (-fsanitize=vptr).byoungyoung@google.com2014-06-261-1/+1
| | | | | | | | | | | | | | Can be enabled using ubsan_vptr=1, but not working right now due to 1) Clang's missing library and 2) Clang's link bug. BUG=174801 TBR=kbr@chromium.org, mseaborn@chromium.org R=glider@chromium.org, inferno@chromium.org Review URL: https://codereview.chromium.org/357753002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280162 0039d316-1c4b-4281-b951-d872f2087c98
* Move PPB_TrueTypeFont_Dev host from renderer to browser.bbudge@chromium.org2014-06-263-33/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Linux, we can now use FontConfig in the browser process since it is thread-safe, so we don't need to use SandboxIPC from the renderer. In the browser, the needed functionality is split off from SandboxIPCHandler into a static function, MatchFontFaceWithFallback in a new pair of files, font_utils_linux.*. This change also moves any potentially blocking font creation and reading to the browser's blocking thread pool. I reworked the PepperTrueTypeFont base class to make Create simpler. - Added an Initialize method which returns the font descriptor. - Removed the Describe method. I reworked the resource to delay calls to the host until we receive the desc and initialization is complete. Describe will now wait until Initialize completes, and the host uses a SequencedTaskRunner to serialize tasks, so Initialize completes before GetTableTags and GetTable calls complete. The Describe method can be implemented without IPC since we have the desc on the plugin side. BUG=382729 Review URL: https://codereview.chromium.org/337203003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280107 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Move Pnacl init time out of trusted plugin.teravest@chromium.org2014-06-265-16/+26
| | | | | | | | | | | | This change moves the pnacl init time out of the trusted plugin. It's part of a series of changes to get rid of the trusted plugin entirely. BUG=239656 R=bbudge@chromium.org Review URL: https://codereview.chromium.org/356923002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280082 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Remove method in PluginReverseInterface.teravest@chromium.org2014-06-262-8/+0
| | | | | | | | | | | | CloseManifestEntry has been removed from nacl::ReverseInterface, so we don't need this empty method anymore. BUG= R=bbudge@chromium.org Review URL: https://codereview.chromium.org/357863002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280079 0039d316-1c4b-4281-b951-d872f2087c98
* Export ResourceMessageParams::SerializedHandleshans@chromium.org2014-06-261-1/+1
| | | | | | | | | | | | | | | | The inner class needs to be exported in case any uses in inline methods or vtable of the outer class causes it to potentially be referenced across a shared library boundary. On non-Windows, the visibility attribute is already inherited by nested classes, but on Windows, they must be marked explicitly. BUG=82385 TEST=use Clang to do component release build of chrome on Windows Review URL: https://codereview.chromium.org/359503003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280038 0039d316-1c4b-4281-b951-d872f2087c98
* Tidy up stray semicolons and missing include guard in message files.tsepez@chromium.org2014-06-261-7/+7
| | | | | | | | | | | These manifested when I was making an ipc utility for my own use. They are wrong even if they don't break the builders. R=jam@chromium.org Review URL: https://codereview.chromium.org/354493007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279919 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Refactor OpenManifestEntry.teravest@chromium.org2014-06-254-83/+35
| | | | | | | | | | | | | | | | This change pulls out more OpenManifestEntry code from service_runtime, making PPB_NaCl_Private a smallter interface, and preparing for pulling all of that logic out of the trusted plugin. An earlier version of this change was reverted after breaking Non-SFI mode, but the Non-SFI case no longer uses this codepath. CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 BUG=239656 Review URL: https://codereview.chromium.org/357773002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279832 0039d316-1c4b-4281-b951-d872f2087c98
* NaCl: clean up nexe loading logic in trusted plugin.ncbray@chromium.org2014-06-254-170/+99
| | | | | | | | | CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 BUG=none Review URL: https://codereview.chromium.org/338353008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279762 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Remove callback for resource loading.teravest@chromium.org2014-06-244-43/+11
| | | | | | | | | | | | | Using CompletionCallbacks for PNaClResources::StartLoad and PNaClResources::ReadResourceInfo doesn't accomplish much and makes the code more complicated. This is in preparation for a larger change to get rid of FileDownloader for streaming pexe downloads. BUG=239656 Review URL: https://codereview.chromium.org/358443002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279485 0039d316-1c4b-4281-b951-d872f2087c98
* Remove bogus include paths from NaCl gyp files.sergeyu@chromium.org2014-06-241-1/+0
| | | | | | | | BUG=384752 Review URL: https://codereview.chromium.org/336023003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279241 0039d316-1c4b-4281-b951-d872f2087c98
* Ppapi: Avoid entering resource three times in BindGraphics().penghuang@chromium.org2014-06-231-23/+14
| | | | | | | | BUG=None Review URL: https://codereview.chromium.org/355493002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279227 0039d316-1c4b-4281-b951-d872f2087c98
* Remove LoadModule SRPC for non-SFI mode.hidehiko@chromium.org2014-06-238-5/+57
| | | | | | | | | | | | | | This CL removes LoadModule SRPC for non-SFI mode. Instead, non-SFI mode starts to pass nexe file to LaunchSelLdr, which is eventually passed to NaClListener::OnStart(). TEST=Ran browser_tests --gtest_filter=*NonSfi*:*NonSFI* locally and trybots BUG=333950 CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 Review URL: https://codereview.chromium.org/337463002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279069 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Simplify OpenResource() for Non-SFI.teravest@chromium.org2014-06-204-123/+4
| | | | | | | | | | | | | | The Non-SFI implementation of OpenResource is pretty complicated. This is because the logic to support that operation was entirely in the trusted plugin. Now, we can perform the necessary logic entirely in Chromium, so it can be made much simpler. CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 BUG=239656 Review URL: https://codereview.chromium.org/339213003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278859 0039d316-1c4b-4281-b951-d872f2087c98
* [PPAPI] Add browser tests for compositor APIpenghuang@chromium.org2014-06-208-39/+556
| | | | | | | | | | | | | | And fix a bug found with the tests. BindGraphics() does not work for a device which is in the same type with the current bound device. BUG=374383 R=piman@chromium.org, raymes@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278728 Review URL: https://codereview.chromium.org/324983005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278779 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 278728 "[PPAPI] Add browser tests for compositor API"yzshen@chromium.org2014-06-208-556/+39
| | | | | | | | | | | | | | | | | | > [PPAPI] Add browser tests for compositor API > > And fix a bug found with the tests. > BindGraphics() does not work for a device which is in the same type with the current bound device. > > BUG=374383 > R=piman@chromium.org, raymes@chromium.org > > Review URL: https://codereview.chromium.org/324983005 TBR=penghuang@chromium.org Review URL: https://codereview.chromium.org/342323006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278765 0039d316-1c4b-4281-b951-d872f2087c98
* [PPAPI] Add browser tests for compositor APIpenghuang@chromium.org2014-06-208-39/+556
| | | | | | | | | | | | And fix a bug found with the tests. BindGraphics() does not work for a device which is in the same type with the current bound device. BUG=374383 R=piman@chromium.org, raymes@chromium.org Review URL: https://codereview.chromium.org/324983005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278728 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Whitelist dev channel APIs for some apps.hamaji@chromium.org2014-06-201-0/+12
| | | | | | | | | | | | | | | Some applications need access to dev channel APIs before they will become stable. As written, this change only performs whitelisting for NaCl apps; I don't think we have any need for enabling this in the non-NaCl case. I need to find a good way to test this. BUG=383937 R=dmichael@chromium.org, jln@chromium.org, piman@chromium.org Review URL: https://codereview.chromium.org/338523007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278645 0039d316-1c4b-4281-b951-d872f2087c98
* PPAPI: Move PPB_TraceEvent to have dev-channel permissionsdmichael@chromium.org2014-06-192-2/+8
| | | | | | | | | | | PPB_TraceEvent should be available to developers who want to profile their application on Dev channel or Canary. BUG=386636 Review URL: https://codereview.chromium.org/349443002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278499 0039d316-1c4b-4281-b951-d872f2087c98
* Quiet output when untarring the pnacl translator from gyp builds.dyen@chromium.org2014-06-191-0/+1
| | | | | | | | | BUG= https://code.google.com/p/chromium/issues/detail?id=370981 TEST= Run "ninja -C out/Debug untar_pnacl_translator" Review URL: https://codereview.chromium.org/341893003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278451 0039d316-1c4b-4281-b951-d872f2087c98
* Removed ManagedMemoryStats.vmpstr@chromium.org2014-06-192-7/+0
| | | | | | | | | | | | | As of r277056, the gpu memory manager is no longer using managed memory stats produced. This patch removes the managed memory stats calculation from the tile manager. As well, this patch removes ununsed gpu managed memory stats plumbing. BUG=377065 Review URL: https://codereview.chromium.org/342483007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278294 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Add latency calculation to ppapi/examples/video_decode.bbudge@chromium.org2014-06-191-10/+42
| | | | | | | | | | Also, flip picture vertically (it was upside down). BUG=281689 Review URL: https://codereview.chromium.org/340163002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278281 0039d316-1c4b-4281-b951-d872f2087c98
* Add GetScrollOffset function to PPB_Viewraymes@chromium.org2014-06-1913-17/+172
| | | | | | | | | | | | | | | | | | This adds a function to PPB_View which allows plugins to know the scroll offset of the page when they are in view. This is useful for OOP PDF which uses the scroll offset of the window it is contained in to determine the document's scroll location. A web page can send scroll location via postMessage but this is slow. Sending the offset directly via view messages is much faster and seems reasonable. We don't send the scroll offset in the cases where the plugin is off screen to avoid any more additional IPC traffic than what currently exists. BUG=303491 Review URL: https://codereview.chromium.org/329033003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278236 0039d316-1c4b-4281-b951-d872f2087c98
* [PPAPI] Using ProduceTextureDirectCHROMIUM() to avoid binding and restoring ↵penghuang@chromium.org2014-06-181-20/+2
| | | | | | | | | | | GL_TEXTURE_2D target. BUG=374383 R=bbudge@chromium.org, danakj@chromium.org Review URL: https://codereview.chromium.org/341893004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278165 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Small cleanup in PNaClCoordinator.teravest@chromium.org2014-06-182-15/+9
| | | | | | | | | | | | | | | This simplifies the case where we fail to open all the necessary temporary files as part of PNaCl translation. RunTranslate() isn't called if num_object_files_opened_ != split_module_count_ anyway, so this should be equivalent to the existing behavior. BUG=239656 R=dmichael@chromium.org Review URL: https://codereview.chromium.org/335343006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278143 0039d316-1c4b-4281-b951-d872f2087c98
* PPAPI: Implement synchronous postMessagedmichael@chromium.org2014-06-1818-21/+717
| | | | | | | | BUG=367896 Review URL: https://codereview.chromium.org/264303002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278102 0039d316-1c4b-4281-b951-d872f2087c98
* Possible narrowing conversion build fix for Compositor API example.pgal.u-szeged@partner.samsung.com2014-06-181-2/+2
| | | | | | | | | | | | In the compositor API example gcc 4.9 reports a possible type narrowing when using fabs call. To fix this use fabsf instead. BUG=385988 R=dmichael Review URL: https://codereview.chromium.org/335383003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278076 0039d316-1c4b-4281-b951-d872f2087c98
* Remove redundant use_nonsfi_mode from SelLdrStartParams.hidehiko@chromium.org2014-06-174-25/+19
| | | | | | | | | | | | | | The params is used for ServiceRuntime::StartSelLdr(). ServiceRuntime already knows if it is for nonsfi or not. So the use_nonsfi_mode in SelLdrStartParams is just redundant. Remove it. BUG=333950 TEST=Ran browser_tests --gtest_filter=*NonSfi*:*NonSFI* locally, and trybots. CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 Review URL: https://codereview.chromium.org/332333003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277864 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Add VP9 support to PPB_VideoDecoder API.bbudge@chromium.org2014-06-173-6/+7
| | | | | | | | | | | Improve documentation for Reset and Flush. Use VpxVideoDecoder in implementation code. BUG=281689 Review URL: https://codereview.chromium.org/336833003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277860 0039d316-1c4b-4281-b951-d872f2087c98
* Fix race condition on ManifestService initialization.hidehiko@chromium.org2014-06-171-1/+44
| | | | | | | | | | | | | | | | | | | | SyncMessageFilter::Send() returns false immediately, if the IPC connection is not yet established. As connecting is done asynchronously, there is no guarantee that the connection is established on the first Send() invocation. By this CL, Send() blocks the caller thread if the connection is not yet established. Note that currently the ratio should be probably low, because there are some more initialization steps between the ManifestService creation and the first Send() invocation. We're switching to changing the initialization procedure, and then this race would be hit more easily. TEST=Ran browser_tests --gtest_filter=*NonSfi*:*NonSFI* locally, and trybots. Also, locally modified the code to delay OnFilterAdded with and without this CL, and made sure this CL works well. BUG=333950 CQ_EXTRA_TRYBOTS=tryserver.chromium:linux_rel_precise32 Review URL: https://codereview.chromium.org/334593004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277840 0039d316-1c4b-4281-b951-d872f2087c98
* Remove --disable-extensions-resource-whitelist.pkasting@chromium.org2014-06-171-4/+0
| | | | | | | | | | BUG=357720 TEST=none R=kalman@chromium.org Review URL: https://codereview.chromium.org/324793002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277625 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Fix texture management in VideoDecoderShim on Reset.bbudge@chromium.org2014-06-162-24/+56
| | | | | | | | | | | | | | - Use a hash_set to track available textures. This prevents weird behavior if the plugin recycles a texture twice. - Fix Reset, so all textures are made available on completion. - Fix the plugin, which had a bug that allowed pictures to jump the queue and didn't behave correctly on Reset. BUG=281689 Review URL: https://codereview.chromium.org/337743003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277548 0039d316-1c4b-4281-b951-d872f2087c98
* Remove apps in Pepper support and the PPB_Alarms_Dev API.yzshen@chromium.org2014-06-1634-2912/+0
| | | | | | | | | BUG=366304 TEST=None Review URL: https://codereview.chromium.org/314823002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277513 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Move more UMA stuff out of trusted plugin.teravest@chromium.org2014-06-166-63/+29
| | | | | | | | | | | | | | This is yet another pretty simple change just lifting more lines out of the trusted plugin. I added a method to PPB_NaCl_Private that's safe to call from a background thread (there are other methods that are safe as well) to keep the call sites simple. BUG=239656 R=dmichael@chromium.org Review URL: https://codereview.chromium.org/331973002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277424 0039d316-1c4b-4281-b951-d872f2087c98
* [PPAPI] Compositor API implementation.penghuang@chromium.org2014-06-1619-23/+1755
| | | | | | | | | | | | | | Implement the compositor API which allows a plugin to combine different sources of visual data efficiently, such as PPB_ImageData and OpengGL texture. API Proposal http://goo.gl/V7xcu3 BUG=374383 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277208 Review URL: https://codereview.chromium.org/298023004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277422 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 277208 "[PPAPI] Compositor API implementation."tzik@chromium.org2014-06-1619-1755/+23
| | | | | | | | | | | | | | | | | | | | | | | | This CL seems to cause perf bot failure. The log said nacl_helper-data size gained exceeds expectation. http://build.chromium.org/p/chromium/builders/Linux/builds/50560 https://chromeperf.appspot.com/report?masters=Chromium&bots=chromium-rel-linux&tests=sizes%2Fnacl_helper-data&rev=277212&checked=core > [PPAPI] Compositor API implementation. > > Implement the compositor API which allows a plugin to combine different sources of visual data efficiently, such as PPB_ImageData and OpengGL texture. > > API Proposal http://goo.gl/V7xcu3 > > BUG=374383 > > Review URL: https://codereview.chromium.org/298023004 TBR=penghuang@chromium.org Review URL: https://codereview.chromium.org/331123003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277372 0039d316-1c4b-4281-b951-d872f2087c98
* [PPAPI] Compositor API implementation.penghuang@chromium.org2014-06-1419-23/+1755
| | | | | | | | | | | | Implement the compositor API which allows a plugin to combine different sources of visual data efficiently, such as PPB_ImageData and OpengGL texture. API Proposal http://goo.gl/V7xcu3 BUG=374383 Review URL: https://codereview.chromium.org/298023004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277208 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Refactor some coordinator histograms.teravest@chromium.org2014-06-135-68/+24
| | | | | | | | | | | | | | | We have a bunch of histograms that log data at the same time as ReportTranslationFinished(), so we may as well move some of that logic out of the trusted plugin. We use a different time range for translation for other metrics, so I had to add a "HistogramTimeTranslate" method in addition to the existing ones for "small", "medium", and "large" times. BUG=239656 R=dmichael@chromium.org Review URL: https://codereview.chromium.org/307403005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277073 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of Revert of Implement software fallback for PPB_VideoDecoder. ↵schenney@chromium.org2014-06-138-24/+186
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/337683002/) Reason for revert: Revert of revert, which apparently wasn't a full revert. Original issue's description: > Revert of Implement software fallback for PPB_VideoDecoder. (https://codereview.chromium.org/311853005/) > > Reason for revert: > Broke blink Linux tests compile.http://build.chromium.org/p/chromium.webkit/builders/Linux%20Tests/builds/37259 > > Original issue's description: > > Implement software fallback for PPB_VideoDecoder. > > This modifies the proxy to implement software fallback mode. > > The main change is to the host, which now can work with > > media::VideoDecoders. > > > > media::VideoDecoder works differently from media::VideoDecodeAccelerator > > so an adapter object, content::VideoDecoderShim is defined. It lives on the main thread and drives the actual decoder on the media thread via a child DecoderImpl class, which sends back frames of video. VideoDecoderShim receives those and converts frames to GL textures. > > > > gpu::Mailboxes are used so the host can create textures that are aliased > > to the plugin's textures. > > > > The test plugin has been changed to include bitstream data for VP8 in order to > > test the software decoder. The data is in ppapi/examples/video_decode/testdata.h > > alongside the H264 data. The file diff is too large for this site but is structured > > something like this: > > > > const unsigned char kData[] = { > > #if defined USE_VP8_TESTDATA_INSTEAD_OF_H264 > > ... lots of VP8 data > > > > #else // !USE_VP8_TESTDATA_INSTEAD_OF_H264 > > ... lots of H264 data > > > > #endif // USE_VP8_TESTDATA_INSTEAD_OF_H264 > > }; > > > > > > There is a TODO to convert the example to load a file. I'm not sure how to go > > about that but am willing to do the work if someone can point the way. > > > > BUG=281689 > > R=dmichael@chromium.org, fischman@chromium.org, sievers@chromium.org, tsepez@chromium.org > > > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277012 > > TBR=dmichael@chromium.org,fischman@chromium.org,igorc@chromium.org,piman@chromium.org,sievers@chromium.org,tsepez@chromium.org,bbudge@chromium.org > NOTREECHECKS=true > NOTRY=true > BUG=281689 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277015 TBR=dmichael@chromium.org,fischman@chromium.org,igorc@chromium.org,piman@chromium.org,sievers@chromium.org,tsepez@chromium.org,bbudge@chromium.org NOTREECHECKS=true NOTRY=true BUG=281689 Review URL: https://codereview.chromium.org/333903002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277020 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of Implement software fallback for PPB_VideoDecoder. ↵schenney@chromium.org2014-06-138-186/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/311853005/) Reason for revert: Broke blink Linux tests compile.http://build.chromium.org/p/chromium.webkit/builders/Linux%20Tests/builds/37259 Original issue's description: > Implement software fallback for PPB_VideoDecoder. > This modifies the proxy to implement software fallback mode. > The main change is to the host, which now can work with > media::VideoDecoders. > > media::VideoDecoder works differently from media::VideoDecodeAccelerator > so an adapter object, content::VideoDecoderShim is defined. It lives on the main thread and drives the actual decoder on the media thread via a child DecoderImpl class, which sends back frames of video. VideoDecoderShim receives those and converts frames to GL textures. > > gpu::Mailboxes are used so the host can create textures that are aliased > to the plugin's textures. > > The test plugin has been changed to include bitstream data for VP8 in order to > test the software decoder. The data is in ppapi/examples/video_decode/testdata.h > alongside the H264 data. The file diff is too large for this site but is structured > something like this: > > const unsigned char kData[] = { > #if defined USE_VP8_TESTDATA_INSTEAD_OF_H264 > ... lots of VP8 data > > #else // !USE_VP8_TESTDATA_INSTEAD_OF_H264 > ... lots of H264 data > > #endif // USE_VP8_TESTDATA_INSTEAD_OF_H264 > }; > > > There is a TODO to convert the example to load a file. I'm not sure how to go > about that but am willing to do the work if someone can point the way. > > BUG=281689 > R=dmichael@chromium.org, fischman@chromium.org, sievers@chromium.org, tsepez@chromium.org > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277012 TBR=dmichael@chromium.org,fischman@chromium.org,igorc@chromium.org,piman@chromium.org,sievers@chromium.org,tsepez@chromium.org,bbudge@chromium.org NOTREECHECKS=true NOTRY=true BUG=281689 Review URL: https://codereview.chromium.org/337683002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277015 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the dev version of the PPAPI video decoder plugin.bbudge@chromium.org2014-06-131-2/+0
| | | | | | | | | | TBR=dmichael@chromium.org BUG=281689 Review URL: https://codereview.chromium.org/334883002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277013 0039d316-1c4b-4281-b951-d872f2087c98
* Implement software fallback for PPB_VideoDecoder.bbudge@chromium.org2014-06-139-12546/+33084
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This modifies the proxy to implement software fallback mode. The main change is to the host, which now can work with media::VideoDecoders. media::VideoDecoder works differently from media::VideoDecodeAccelerator so an adapter object, content::VideoDecoderShim is defined. It lives on the main thread and drives the actual decoder on the media thread via a child DecoderImpl class, which sends back frames of video. VideoDecoderShim receives those and converts frames to GL textures. gpu::Mailboxes are used so the host can create textures that are aliased to the plugin's textures. The test plugin has been changed to include bitstream data for VP8 in order to test the software decoder. The data is in ppapi/examples/video_decode/testdata.h alongside the H264 data. The file diff is too large for this site but is structured something like this: const unsigned char kData[] = { #if defined USE_VP8_TESTDATA_INSTEAD_OF_H264 ... lots of VP8 data #else // !USE_VP8_TESTDATA_INSTEAD_OF_H264 ... lots of H264 data #endif // USE_VP8_TESTDATA_INSTEAD_OF_H264 }; There is a TODO to convert the example to load a file. I'm not sure how to go about that but am willing to do the work if someone can point the way. BUG=281689 R=dmichael@chromium.org, fischman@chromium.org, sievers@chromium.org, tsepez@chromium.org Review URL: https://codereview.chromium.org/311853005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277012 0039d316-1c4b-4281-b951-d872f2087c98
* Decouple IPC::MessageFilter from IPC::Channelmorrita@chromium.org2014-06-132-9/+9
| | | | | | | | | | | | | This change gets rid of Channel dependency from MessageFilter so that it depends only on IPC::Sender. TEST=none BUG=377980 R=jam@chromium.org, darin@chromium.org Review URL: https://codereview.chromium.org/324143002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276939 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper: Remove nacl_entry_points.h.teravest@chromium.org2014-06-114-62/+17
| | | | | | | | | | | | | | This function indirection is no longer necessary, and I was getting frustrated by having the type signature of LaunchSelLdr twice in the source. I came across this in an experimental CL modifying LaunchSelLdr to also receive information for the user nexe. BUG=333950 R=dmichael@chromium.org Review URL: https://codereview.chromium.org/322403005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276466 0039d316-1c4b-4281-b951-d872f2087c98
* Add limitation that LoadNexeAndStart must be called on the main thread.hidehiko@chromium.org2014-06-113-9/+19
| | | | | | | | | | | | | | | | Now we're switching LoadNexeAndStart from synchronous implementation to asynchronous one. Under PPAPI implementation, the async callbacks are handled on main thread. To keep the similicity of the thread model, this CL introduces the limitation that Plugin::LoadNexeAndStart must be called on the main thread. TEST=Ran browser_tests --gtest_filter=NaCl* locally and trybots. BUG=333950 R=teravest@chromium.org Review URL: https://codereview.chromium.org/321053004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276371 0039d316-1c4b-4281-b951-d872f2087c98