| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Passed chrome-ui-review: https://groups.google.com/a/google.com/d/msg/chrome-ui-review/FwuQ2TWc5n8/H3PSuV4L_cAJ
BUG=467165
TBR=tsepez
(comment changes only in render_messages)
NOPRESUBMIT=true
(legacy LOG messages)
Review URL: https://codereview.chromium.org/1007443004
Cr-Commit-Position: refs/heads/master@{#320782}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change introduces two new values to be used with SetOption(), to
set up the behavior of the multicast packets:
- PP_UDPSOCKET_OPTION_MULTICAST_LOOP (Boolean)
Whether packets sent from the host to the multicast group will be
looped back to the host or not.
- PP_UDPSOCKET_OPTION_MULTICAST_TTL (Integer)
Sets time-to-live of multicast packets sent to the multicast group.
It also introduces two new functions JoinGroup() and LeaveGroup(), both
of them expecting the address of the multicast group as specified by
the PPB_NetAddress API. These two functions should only be called after
the socket is bound.
This CL adds specific test cases to browser_tests in order to exercise
the different versions of the SetOption function. For this reason, the
PPAPI UDPSocket tests were split into multiple tests, instead of a
single one that enclosed all of them. This way we get clearer
information about the results of each test.
BUG=430939
TEST=browser_tests
NOPRESUBMIT=true
Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>
Review URL: https://codereview.chromium.org/704133005
Cr-Commit-Position: refs/heads/master@{#320504}
|
|
|
|
|
|
|
|
|
|
|
|
| |
configuration disallows access to them.
Based on the configured value of distinctiveIdentifier and persistentState, we compute permission bits and pass them through PPAPI to the CDM adapter. This enables us to enforce 'not-allowed' configs even when user permission has been granted.
BUG=487452
Review URL: https://codereview.chromium.org/985113003
Cr-Commit-Position: refs/heads/master@{#319798}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's unused and will probably never go stable. Delete it; we can pull it
from source control later if we change our minds.
TBRing thestig for chrome/common/ppapi_utils, and binji for native_client_sd
BUG=83774
TBR=thestig@chromium.org,binji@chromium.org
Review URL: https://codereview.chromium.org/959333002
Cr-Commit-Position: refs/heads/master@{#318530}
|
|
|
|
|
|
|
|
|
| |
BUG=387547
TEST=Call the API from a trusted plugin.
Review URL: https://codereview.chromium.org/929053003
Cr-Commit-Position: refs/heads/master@{#317271}
|
|
|
|
|
|
|
|
|
| |
BUG=387547
TEST=Call the API from a trusted plugin and print all video capture formats.
Review URL: https://codereview.chromium.org/911623002
Cr-Commit-Position: refs/heads/master@{#317230}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Support PPB_CameraCapabilities_Private.GetSupportedPreviewSizes()
2. Implement basic skeleton of image capture PPAPI
BUG=387547
TEST=Call the API from a trusted plugin and print all preview
sizes.
Review URL: https://codereview.chromium.org/848863002
Cr-Commit-Position: refs/heads/master@{#316552}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since PPB_NaCl_Private is both implemented and consumed in
components/nacl/renderer/, there's no need for the interface to be
defined in ppapi/. The fact that this is a PPAPI interface at all is
just a holdover from when the code for the NaCl trusted plugin lived
outside the Chromium repo.
We can drop the ppb_nacl_private.idl file.
Benefits:
* This reduces the friction for changing ppb_nacl_private.h. It's no
longer necessary to change ppb_nacl_private.idl and re-run the
generator.
* This removes PPB_NaCl_Private from the PPAPI shims, reducing code
size.
* ppb_nacl_private.h no longer has to follow the PPAPI C header
style. For example, it would no longer need to define its own
"struct PP_NaClFileInfo" type -- it could share a definition that's
marshallable over Chrome IPC.
Changes to ppb_nacl_private.h:
* Add comment to explain the status of this interface.
* Remove unused "*_1_0" variants of #define and typedef.
BUG=239656
BUG=251460
TEST=trybots
Review URL: https://codereview.chromium.org/911463003
Cr-Commit-Position: refs/heads/master@{#315641}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's no longer necessary for the NaCl side to define and refer to the
NaClFileToken struct.
So we can change the Chromium side to define NaClFileToken, instead of
using the definition from nacl_file_info.h. This will let us remove
nacl_file_info.h from the NaCl side.
We copy the comment from nacl_file_info.h.
BUG=https://code.google.com/p/nativeclient/issues/detail?id=4073
TEST=build
Review URL: https://codereview.chromium.org/903333002
Cr-Commit-Position: refs/heads/master@{#315317}
|
|
|
|
|
|
|
|
|
|
| |
Also changes copyright notices to the new style.
BUG=455409
Review URL: https://codereview.chromium.org/885803003
Cr-Commit-Position: refs/heads/master@{#314893}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Remove ppb_image_capture_config_private.
2. Remove PPB_ImageCapture_Private.{Set|Get}Config
3. Remove PPB_ImageCapture_Private.CaptureStillImage
BUG=387547
TEST=None
Review URL: https://codereview.chromium.org/884483007
Cr-Commit-Position: refs/heads/master@{#314834}
|
|
|
|
|
|
|
|
|
|
|
| |
Since this bumps the milestone definition in pp_macros.h
NOPRESUBMIT=true
BUG=417589
Review URL: https://codereview.chromium.org/842293003
Cr-Commit-Position: refs/heads/master@{#314645}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There's no need for this flag to be sent from ppb_nacl_private_impl.cc
to plugin.cc and back again. It can stay within
ppb_nacl_private_impl.cc.
This merges two error-handling checks in LaunchSelLdr().
BUG=239656
TEST=browser_tests
Review URL: https://codereview.chromium.org/874603002
Cr-Commit-Position: refs/heads/master@{#313010}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The last active use of PluginReverseInterface was via its
StartupInitializationComplete()/"init_done" method, but recent
cleanups have made that into a no-op.
We can therefore remove PluginReverseInterface. We no longer need to
call NaCl's "reverse_setup" SRPC call.
This removes the Chromium-side parts of the old SRPC-based
open_resource() implementation, which was no longer called. This
allows us to remove OpenManifestEntry() from PPB_NaCl_Private.
Add some #includes to fix include-what-you-use problems that become
apparent after removing code.
BUG=https://code.google.com/p/nativeclient/issues/detail?id=3864
TEST=NaCl tests in browser_tests
Review URL: https://codereview.chromium.org/870923004
Cr-Commit-Position: refs/heads/master@{#312953}
|
|
|
|
|
|
|
|
|
| |
BUG=428030
TEST=build
Review URL: https://codereview.chromium.org/865303003
Cr-Commit-Position: refs/heads/master@{#312846}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Remove Plugin::BitcodeDidTranslateContinuation(), which gets called
via the "init_done" SRPC call (which calls
StartupInitializationComplete() in plugin/service_runtime.cc).
Instead, arrange for its work to be done by the newer
ManifestServiceProxy::StartupInitializationComplete(), by enabling
ManifestServiceProxy for PNaCl.
* To ensure that the open_resource() IPC stays disabled for PNaCl, we
conditionalise ManifestServiceProxy::OpenResource().
* To ensure that the pexe size still gets reported on the "loadend"
event, we stash it away in ReportTranslationFinished().
BUG=428030
TEST=NaClBrowserTestPnacl.ProgressEvents (checks reported pexe size is non-zero)
Review URL: https://codereview.chromium.org/845013003
Cr-Commit-Position: refs/heads/master@{#312674}
|
|
|
|
|
|
|
|
|
|
|
| |
Web session ID was introduced when we also have integer session IDs. Now we
have removed integer session IDs and we don't need the "web" prefix any more.
TEST=Renaming only.
Review URL: https://codereview.chromium.org/858313002
Cr-Commit-Position: refs/heads/master@{#312661}
|
|
|
|
|
|
|
|
|
| |
BUG=428030
TEST=build
Review URL: https://codereview.chromium.org/866583002
Cr-Commit-Position: refs/heads/master@{#312505}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Work was started which would use these fields to convey information to the
printing system but this work was never finished (see https://code.google.com/p/chromium/issues/detail?id=169120).
This fields are never initialized.
BUG=448798
Review URL: https://codereview.chromium.org/828233004
Cr-Commit-Position: refs/heads/master@{#311831}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes the implementation to allow multiple SendTo calls to be in
flight at the same time. The plugin can now call SendTo multiple times
before getting PP_ERROR_INPROGRESS. The number of calls is an
implementation detail. Plugins that want maximum performance should
call SendTo continuously until they get an in progress result, then wait
for one or more previous calls to complete before calling Send again.
The resource is changed to queue SendTo callbacks. It checks that the
plugin stays within the limit, which is currently 8.
The browser message filter is changed to call SendTo until the socket is
busy, then queue the sends. It checks that the plugin can't grow the
queue past the limit. In theory, a plugin could go over the limit by
bypassing the resource checks but not saturating the socket. This is
not really a problem.
Adds a test to send multiple messages in the "optional callbacks" case.
BUG=154338
Review URL: https://codereview.chromium.org/632113003
Cr-Commit-Position: refs/heads/master@{#311563}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Recent changes to CDM_7 removed |destination_url| as it is no longer
used in the EME spec. However legacy applications using the prefixed
EME API depend on it, so adding it back in so that it is available
to those applications.
BUG=448242
TEST=existing EME tests pass + Play movies play
Review URL: https://codereview.chromium.org/813683005
Cr-Commit-Position: refs/heads/master@{#311371}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add support for CDM_7 and update Pepper to support the updated
interface.
Changes:
- Rename CreateSession() to CreateSessionAndGenerateRequest()
with reordered parameter list.
- Add |session_type| to LoadSession().
- OnSessionMessage() adds |message_type|, removes |destination_url|.
- OnSessionKeysChange() adds |key_information|.
- Remove GetUsableKeyIds()
- Remove OnPromiseResolvedWithKeyIds()
- Remove OnSessionReady().
BUG=428384
TEST=existing EME test cases pass
Review URL: https://codereview.chromium.org/811923002
Cr-Commit-Position: refs/heads/master@{#310615}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently PPAPI has timing limitation for sockets' SetOption.
NODELAY, and BROADCAST need to be before Connect() or Bind(),
while RCVBUF_SIZE and SNFBUF_SIZE need to be after it.
This CL removes such a limitation.
Along with the change, pepper_udp_socket_message_filter starts to use UDPSocket instead of UDPServerSocket, so that the implementation direction gets closer to TCP message filter a little bit.
BUG=425563, 420697
TEST=Ran trybots.
Review URL: https://codereview.chromium.org/690903002
Cr-Commit-Position: refs/heads/master@{#307867}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Chrome side changes to support NumCopies print preset option.
Copies in print preview dialog should be set to value set for
NumCopies in pdf source document.
BUG=169120
Review URL: https://codereview.chromium.org/375253002
Cr-Commit-Position: refs/heads/master@{#304785}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds an out parameter to GetPicture to return the subrectangle of
the picture texture that contains the picture.
Adds a new 0.3 version of PPB_VideoDecoder.
BUG=429071
NOPRESUBMIT=true
R=binji@chromium.org, dmichael@chromium.org, mpearson@chromium.org, tsepez@chromium.org
Review URL: https://codereview.chromium.org/703753002
Cr-Commit-Position: refs/heads/master@{#303102}
|
|
|
|
|
|
|
|
|
|
|
|
| |
from about 4ms to 0.7ms for each 720p frame.
If the current video frame is of the same size as the previous frame, reuse the shared memory and overwrite the previous frame. Pepper does not hold onto a frame after it has been processed, and it indicates processing is done by sending a new request, so overwriting the previous frame should be fine.
BUG=423365
Review URL: https://codereview.chromium.org/653873003
Cr-Commit-Position: refs/heads/master@{#301657}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There was already an enum called NaClProcessType (for the
broker, etc.), so I called this NaClAppProcessType instead.
Renamed the original NaClProcessType to NaClTrustedProcessType.
This type can then be used to decide on various things
like crash throttling, nexe size limits, dyncode
enablement, IRT usage rather than plumb separate flags
through the various layers and IPCs. Then we can tweak
the "uses_irt" to be "optional" for the PNaClTranslator
as well.
Misc: removed enable_dev_interfaces from service_runtime
SelLdrStartParams struct, since it looked unused.
BUG= https://code.google.com/p/nativeclient/issues/detail?id=3914
Review URL: https://codereview.chromium.org/614373004
Cr-Commit-Position: refs/heads/master@{#297987}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds postMessageAndAwaitResponse() support to Pepper stable.
Open issues:
1) This is arguably an expansion on what JavaScript can do today. In particular,
it would be possible to make JavaScript block on PNaCl, which blocks on another
thread, which blocks on something like synchronous FileIO.
2) Not all types are supported yet; e.g. FileRef and FileSystem. I want to fix
that prior to M39. (see https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/pepper/message_channel.cc&l=305)
BUG=367896
Review URL: https://codereview.chromium.org/600893002
Cr-Commit-Position: refs/heads/master@{#297026}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes it so that the plugin can not use blocking completion callbacks while servicing HandleBlockingMessage. This is to discourage leaving JavaScript hung while doing long-running things (e.g. file IO).
(Of course, the plugin can get around this by bouncing to another thread; but I think this adds a useful guideline/deterrent.)
BUG=415351
Review URL: https://codereview.chromium.org/600553002
Cr-Commit-Position: refs/heads/master@{#296422}
|
|
|
|
|
|
|
|
| |
BUG=367896
Review URL: https://codereview.chromium.org/318763003
Cr-Commit-Position: refs/heads/master@{#295495}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The PNaCl ABI does not work in general with passing structs by-value.
Rather than add more complication to the shim, this changes
HandleBlockingMessage's parameter to a const-pointer param, and the return
to an out-param by pointer.
Note I wanted to pass PP_Var by const-pointer, while its "default" is
by value. So I added a new "constptr_in" param style.
(Maybe I should invert that and add a byvalue instead?)
Verified backwards compat with manual testing.
BUG=384539
R=asvitkine@chromium.org, binji@chromium.org, teravest@chromium.org
Review URL: https://codereview.chromium.org/564573002
Cr-Commit-Position: refs/heads/master@{#295341}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Notes for discussion:
1) This API provides a low level interface to Chrome's video decoders. Chrome's implementations differ by codec and platform. H264 and VPX expect very different formats, so the plugin must know details about these codecs. In particular, some parsing of the stream is required to extract the frames, and during Reset, the plugin may have to parse into the frames to find start points and make sure the decoder is in the proper state to resume. Some platforms may only provide software implementations. Software and hardware decoders have very different latency and throughput.
2) The PP_VideoPicture struct (in pp_codecs.idl) exposes low level details about the texture format of the decoded pictures so the plugin can render the textures. These are platform-specific.
GL_TEXTURE_2D is the standard one.
GL_TEXTURE_RECTANGLE_ARB will be used by Mac in the future (but is not yet used).
GL_TEXTURE_EXTERNAL_OES is used on ARM ChromeOS systems.
Plugins should be prepared to handle all of them.
3) The plugin continually calls GetPicture to pump the decoder. There is a little weirdness with Flush, where we signal the end of stream by returning PP_ERROR_ABORTED to the plugin's last GetPicture call.
BUG=281689
Review URL: https://codereview.chromium.org/540393003
Cr-Commit-Position: refs/heads/master@{#295174}
|
|
|
|
|
|
|
|
|
|
|
| |
The dev marker is removed, making this a stable pepper API.
The context around this decision can be found here:
https://groups.google.com/a/chromium.org/forum/#!topic/pepper-dev/Dw-UGrTuYuI
Review URL: https://codereview.chromium.org/547523002
Cr-Commit-Position: refs/heads/master@{#294056}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, we were directly generating header files for
OpenGLES2 interface. This is not aligned with other Pepper
APIs. The header file generated from the generated IDL file
should be compatible with the previous one. To keep OpenGLES2
interfaces in struct namespace, this change introduces
"force_struct_namespace" attribute to the generator script.
TEST=trybots
BUG=411799
Review URL: https://codereview.chromium.org/538903002
Cr-Commit-Position: refs/heads/master@{#293904}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To support CDM_6, make the following changes:
- add SetServerCertificate
- add GetUsableKeyIds
- rename ReleaseSession to CloseSession
- add RemoveSession
- add SessionKeysChange event
- add SessionExpirationChange event
Includes changes to cdm_adapter for the new functionality. Changes to
use these new interfaces in blink in a future CL.
BUG=358271
TEST=existing EME tests still pass + manual testing
Review URL: https://codereview.chromium.org/496143002
Cr-Commit-Position: refs/heads/master@{#293672}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change uses the OnChannelError() callback in TrustedPluginChannel to
detect that the channel between the renderer and NaCl processes has closed.
Previously, closure of the SRPC channel between those two processes was used to
indicate a plugin crash.
I wasn't sure at first that OnChannelError() would be called, but it appears to
be called reliably in the testing that I've done.
This change removes a call to ReportCrash() in ServiceRuntime::ReapLogs() since
ReportCrash() has become a no-op.
This change reenables the NaClBrowserTest*.Crash tests on Linux to ensure that
there's good coverage-- they seem to pass.
TEST=NaClBrowserTest*.Crash
BUG=366334,401105
R=mseaborn@chromium.org
Review URL: https://codereview.chromium.org/512323002
Cr-Commit-Position: refs/heads/master@{#293297}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
After sending an email to the native-client-discuss list, it was determined
that this feature for redirecting stdout and stderr to JavaScript isn't very
useful. This drops support for that feature; a follow up change to the NaCl
repository will remove support there (and the SRPC message).
BUG=406558
Review URL: https://codereview.chromium.org/513273003
Cr-Commit-Position: refs/heads/master@{#293224}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/498123002/)
Reason for revert:
I should've checked sooner; this is apparently still used by Flash. (See go/UMA, histograms, Pepper.InterfaceUsed)
Original issue's description:
> Remove unused PPB_View_Dev
>
> TBR=binji@chromium.org
>
> Committed: https://chromium.googlesource.com/chromium/src/+/2210d5fb49cd4b75972f2e304b29637c050de4f1
TBR=binji@chromium.org,raymes@chromium.org
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/512703003
Cr-Commit-Position: refs/heads/master@{#292152}
|
|
|
|
|
|
|
|
| |
TBR=binji@chromium.org
Review URL: https://codereview.chromium.org/498123002
Cr-Commit-Position: refs/heads/master@{#292086}
|
|
|
|
|
|
|
|
|
|
|
| |
Add 'Initialize' method that takes an enum value to specify hardware
acceleration policy. Policies are always, with fallback and only software.
BUG=406194
Review URL: https://codereview.chromium.org/496203002
Cr-Commit-Position: refs/heads/master@{#291606}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change uses a NaCl embedder interface for getting the exit status inside
the loader process instead of receiving that information over SRPC.
The IPC message introduced here must be synchronous so that we're guaranteed to
report the exit status before the loader process exits.
BUG=397161
TEST=NaClBrowserTest.ExitStatus*
R=dmichael@chromium.org, jschuh@chromium.org, mseaborn@chromium.org
Review URL: https://codereview.chromium.org/484783002
Cr-Commit-Position: refs/heads/master@{#291480}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291480 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Originally an IDL argument like "[out] int[] foo" will generate
int* foo[]. It means an array of pointers to integers and what we
want is a pointer to an array of integers. Change the output to
int** foo to be not confusing.
BUG=387547
TEST=Run generator and build chrome.
Review URL: https://codereview.chromium.org/490433002
Cr-Commit-Position: refs/heads/master@{#291349}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291349 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
nacl_helper reports the last few log messages to the renderer when a LOG_FATAL
message is logged in the plugin. This currently uses the "bootstrap" channel,
which we would like to get rid of.
This uses a shared memory segment instead of IPC, since something bad may have
happened in the plugin process, so we'd like to keep the behavior for handling
that case as simple as possible. For that reason, the shared memory segment is
mapped before the nexe is loaded.
I've tested this locally by calling RemoteLog(LOG_FATAL, ...) after StartModule
in ServiceRuntime, though I'd like to find some better way to test this as
there's a decent amount of logic here.
BUG=391039
R=jschuh@chromium.org, mseaborn@chromium.org
Review URL: https://codereview.chromium.org/469423002
Cr-Commit-Position: refs/heads/master@{#290910}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290910 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
API proposal:
https://docs.google.com/a/chromium.org/document/d/1deeCrsvT9xA77UP4yWMhqbHgS3FvYCTF_8meIMQRrDo/edit#
BUG=387547
TEST=Build chrome, run PPAPITest, and run an app that uses ppapi.
Review URL: https://codereview.chromium.org/391323002
Cr-Commit-Position: refs/heads/master@{#290186}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290186 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=403504
Review URL: https://codereview.chromium.org/475123003
Cr-Commit-Position: refs/heads/master@{#290016}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290016 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
VP8 and VP9 do not take profile into account. Using PROFILE_MAIN is confusing.
This patch uses PROFILE_ANY for these codecs.
TBR=noelallen@chromium.org
BUG=361676
Review URL: https://codereview.chromium.org/418193003
Cr-Commit-Position: refs/heads/master@{#289765}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289765 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds a function to PPB_Testing to run the garbage collector. This is
necessary when using gin-backed v8 objects so that we can test that
destruction works correctly when the gin-backed objects are no longer
referenced. It also modifies the tests to run garbage collection and also
to be careful with scoping such that any handles on the stack to v8 objects
are destroyed before running the GC.
BUG=351636
Review URL: https://codereview.chromium.org/445813002
Cr-Commit-Position: refs/heads/master@{#288647}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288647 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A refactor of FileDownloader (r284961) broke the caching behavior for nexes
that were translated from pexes; the cache would always end up with entries of
length 0, breaking future loads of the pexe. This is the translated nexe was
written to the wrong file handle, preventing caching of the result.
This change causes the translated nexe to be written to the correct file
handle, fixing the caching behavior.
I manually tested this change on the "game of life" demo at
https://gonativeclient.appspot.com/demo/life, verifying that a cached
translated nexe for the pexe was successfully loaded.
BUG=401121
Review URL: https://codereview.chromium.org/447013002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287912 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This simplifies PnaclCoordinator considerably and reduces the total
amount of code in the trusted plugin.
BUG=239656
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284684
R=dmichael@chromium.org
Review URL: https://codereview.chromium.org/393693004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284961 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reverting as broke chrome and chromeos dbg builders:
http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/32615/steps/browser_tests/logs/SuccessfulLoadUMA
[ RUN ] NaClBrowserTestPnacl.SuccessfulLoadUMA
Xlib: extension "RANDR" missing on display ":9".
Xlib: extension "RANDR" missing on display ":9".
[18889:18889:0722/130648:WARNING:password_store_factory.cc(215)] Using
basic (unencrypted) store for password storage. See
http://code.google.com/p/chromium/wiki/LinuxPasswordStorage for more
information about password storage options.
HTTP server started on http://127.0.0.1:56115...
sending server_data: {"host": "127.0.0.1", "port": 56115} (36 bytes)
[9:10:0722/130650:ERROR:pnacl_translation_resource_host.cc(135)] Got
invalid platformfilefortransit
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/debug/vector:461:
error: attempt to insert into container with an iterator from a
different container.
Objects involved in the operation:
sequence "this" @ 0x0x392ba6c93260 {
}
iterator "__position" @ 0x0x7fff9ff24e20 {
state = past-the-end;
references sequence @ 0x0x7fff9ff24e20
}
[18889:18938:0722/130651:WARNING:raw_channel_posix.cc(214)] recvmsg:
Connection reset by peer
BrowserTestBase signal handler received SIGTERM. Backtrace:
#0 0x7fe4bf491cee base::debug::StackTrace::StackTrace()
#1 0x000004af93fa content::(anonymous
namespace)::DumpStackTraceSignalHandler()
#2 0x7fe4b68074a0 \u003Cunknown>
#3 0x7fe4b68b9a43 __poll
#4 0x7fe4b76faff6 \u003Cunknown>
#5 0x7fe4b76fb124 g_main_context_iteration
#6 0x7fe4bf44ff75 base::MessagePumpGlib::Run()
#7 0x7fe4bf54b1f0 base::MessageLoop::RunHandler()
#8 0x7fe4bf5b18e2 base::RunLoop::Run()
#9 0x000004b63969 content::RunThisRunLoop()
#10 0x000004b638fa content::RunMessageLoop()
#11 0x000004b0ea15 content::JavascriptTestObserver::Run()
#12 0x000001312927 NaClBrowserTestBase::RunJavascriptTest()
#13 0x000001312a48 NaClBrowserTestBase::RunLoadTest()
#14 0x00000130f514 (anonymous
namespace)::NaClBrowserTestPnacl_SuccessfulLoadUMA_Test::RunTestOnMai
...
> Pepper: Delete FileDownloader in trusted plugin.
>
> This simplifies PnaclCoordinator considerably and reduces the total
> amount of code in the trusted plugin.
>
> BUG=239656
>
> Review URL: https://codereview.chromium.org/393693004
TBR=teravest@chromium.org
Review URL: https://codereview.chromium.org/406323003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284791 0039d316-1c4b-4281-b951-d872f2087c98
|