| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> [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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|