| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Actually, the interface is currently known as PPB_Flash_TCPSocket.
BUG=103993
TEST=ui_tests OutOfProcessPPAPITest.TCPSocketPrivate
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8564034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110026 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
consistent with
untrusted.gypi. Ninja has a different layout, which this was breaking.
BUG=None
TEST=None
R=noelallen@google.com,wez@google.com
Review URL: http://codereview.chromium.org/8510068
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110015 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also, convert the version number for the PPB_Flash_Clipboard interface to the
standard form.
BUG=104001,104184
TEST=(The existing) Pepper Flash still works.
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8559003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109999 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Clean up pychecker warnings.
Move common code from idl_c_header (header generator) to idl_c_proto, the
'C' code style prototyping helper.
BUG= http://code.google.com/p/chromium/issues/detail?id=91607
TEST= cd src/ppapi/generators && ./generator.py
Review URL: http://codereview.chromium.org/8538029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109958 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=54005
TEST=build and run the examples on Mac (try both debug and release build)
Review URL: http://codereview.chromium.org/8551001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109933 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8522002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109818 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This reduces the warnings when running the generator and makes the definitions
more robust (or at least makes it more obvious when someone changes things).
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8538038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109817 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8539041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109816 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This will make reviewing the next change, to actually generate ppb_flash.h from
ppb_flash.idl, less painful.
TBR=noelallen@chromium.org
Review URL: http://codereview.chromium.org/8530002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109786 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=103955,103969
TEST=ui_tests: {OutOfProcessPPAPITests,PPAPITests}.NetAddressPrivate
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8537026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109783 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change: http://codereview.chromium.org/8502036 added a size regression
since it failed to strip debug information from the IRT. This CL
pulls in the change to build_nexe.py via DEPS roll and adds the
strip flag.
BUG= http://code.google.com/p/chromium/issues/detail?id=96782
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8550001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109780 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a strip script based on the old build_nacl_irt.py (just keep strip piece)
Rename nacl_irt target to nacl_irt_unstripped and add a new target called
nacl_irt which strips the previous target.
NOTE: Resubmit of http://codereview.chromium.org/8539039/ which broke
32 bit due to a missing comma in the gyp file.
Review URL: http://codereview.chromium.org/8537031
TBR=noelallen@google.com
Review URL: http://codereview.chromium.org/8546001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109776 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a strip script based on the old build_nacl_irt.py (just keep strip piece)
Rename nacl_irt target to nacl_irt_unstripped and add a new target called
nacl_irt which strips the previous target.
NOTE: Resubmit of http://codereview.chromium.org/8539039/ which broke
32 bit due to a missing comma in the gyp file.
Review URL: http://codereview.chromium.org/8537031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109771 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a strip script based on the old build_nacl_irt.py (just keep strip piece)
Rename nacl_irt target to nacl_irt_unstripped and add a new target called
nacl_irt which strips the previous target.
Review URL: http://codereview.chromium.org/8539039
TBR=noelallen@google.com
Review URL: http://codereview.chromium.org/8538036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109766 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Add a strip script based on the old build_nacl_irt.py (just keep strip piece)
Rename nacl_irt target to nacl_irt_unstripped and add a new target called
nacl_irt which strips the previous target.
Review URL: http://codereview.chromium.org/8539039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109764 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reconstruct of CL to enable untrsuted builds see:
http://codereview.chromium.org/8416025.
This CL will enable untrusted builds in Chrome. The first step
is to enable enable building the IRT via this technique instead
of using scons.
DEPS:
We bump NaCl deps to fix untrusted.gypi in NaCl Repo
ppapi/ppapi_cpp.gypi+
Add ppapi_cpp library for NEXE
ppapi/native_client/native_client.gyp
Add ppapi library for NEXE (copy linker script to toolchain dir)
Add nacl_irt target (the actual IRT.NEXE)
ppapi/ppapi.gyp & ppapi/ppapi_internal.gyp
Pull in untrusted.gypi from NaCl repo
ppapi/native_client/src/shared/ppapi_proxy/ppapi_proxy.gyp
Add ppruntime library for NEXE
chrome/nacl.gypi
Remove scons header scan and scons irt build
Add dependency to untrusted IRT build in native_client.gyp
BUG= http://code.google.com/p/chromium/issues/detail?id=96782
TEST= try
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8502036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109739 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=102448
TEST=Compiles
Review URL: http://codereview.chromium.org/8524002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109737 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=PPAPITest.FlashClipboard passes.
Review URL: http://codereview.chromium.org/8515014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109724 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8527017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109691 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
URL used for the link resources. In the future that base will be the pnacl
manifest file.
BUG= none
TEST= none
Review URL: http://codereview.chromium.org/8525014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109659 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8527020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109595 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Define WebSocket Pepper API as IDL and generate the C interface from it.
This IDL takes over from http://codereview.chromium.org/7837022/
BUG=87310
TEST=n/a because only the interface is defined in this change.
Review URL: http://codereview.chromium.org/8395037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109592 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
instead of duplicating it.
The Issue:
After broker creates the render-broker channel, the original code will dup() the client file descriptor (fd) and send the duplicate to the render. As a result, there are two copies of the same client fd. One is in the broker process; the other one is sent over the IPC channel and will be owned by the browser and finally by the render. Normally, when connection establishes correctly, the render will send a HelloMessage to the broker, so that the broker is notified and has a chance to call IPC::Channel::ChannelImpl::ProcessIncomingMessages(), which will call CloseClientFileDescriptor() and close its copy of the client fd in the broker. After this, there will be only one copy of the client fd in the render. Whenever the render closes this fd, the broker will get notified with an event.
However, if before the connection establishes, something goes wrong (e.g. the render crashed), closing the cliend fd in the browser or render will not work. The broker will keep its server fd and never get notified. The reason is as follows. Since the connection is not established, the HelloMessage never comes. So the broker will never call CloseClientFileDescriptor() to close its copy of the client fd. When there are two copies of the client fd open, closing one of them (e.g. the one owned by the render or browser) won't generate an event in the broker. The broker will keep waiting and never get notified.
The Fix:
Do not duplicate the client fd. Thus whenever/whereever the client fd is closed, the broker will get notified with an event.
BUG=82176
TEST=Tested with ui broker test and normal usage. Also crashed render before connection established and verified that the broker released the render-broker IPC channel.
Review URL: http://codereview.chromium.org/8497026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109488 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8438005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109463 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
the pexe. This will allow separating these tasks more cleanly, and also
makes the translation of individual pexes/psos more cleanly separated from
the common portions.
BUG= http://code.google.com/p/nativeclient/issues/detail?id=2409
TEST= run_pnacl_example_browser_test
Review URL: http://codereview.chromium.org/8490018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109359 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8507030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109358 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Oops.
TBR=dmichael
Review URL: http://codereview.chromium.org/8497058
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109353 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Note that, though the PP_Flash_NetAddress struct is being renamed, binary
compatibility is being maintained. It's also safe to just rename the
PPB_Flash_NetAddress interface without maintaining backwards compatibility since
Pepper Flash isn't using it yet.
Review URL: http://codereview.chromium.org/8511032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109335 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8511022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109330 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Committing for robertm:
http://codereview.chromium.org/8495030/
Description
Remove unused function. The pnacl toolchain -- which uses clang complains about this.
Not sure why our clang bot does not
BUG= http://code.google.com/p/nativeclient/issues/detail?id=2425
Review URL: http://codereview.chromium.org/8498031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109308 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=Pepper Flash should render the text on http://www.adventmedia.net/#/home correctly.
Review URL: http://codereview.chromium.org/8479024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109189 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These files normally handle ctors/dtors/jtors and EH initialization.
For PNaCl, they are only responsible for EH initialization.
This task was previously inside the bitcode (part of nacl_startup.bc)
BUG= http://code.google.com/p/nativeclient/issues/detail?id=2423
Review URL: http://codereview.chromium.org/8492022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109173 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
[The Win components build was already fixed, and the broken unrevert reverted. The original review was here: http://codereview.chromium.org/8357030 .]
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8496045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109165 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
dealing with PP_Flash_NetAddress.
[The Win components breakage should be fixed by r109156.]
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8357030
TBR=viettrungluu@chromium.org
Review URL: http://codereview.chromium.org/8499045
TBR=thakis@chromium.org
Review URL: http://codereview.chromium.org/8496043
TBR=viettrungluu@chromium.org
Review URL: http://codereview.chromium.org/8498037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109161 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
PP_Flash_NetAddress.
[The Win components breakage should be fixed by r109156.]
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8357030
TBR=viettrungluu@chromium.org
Review URL: http://codereview.chromium.org/8499045
TBR=thakis@chromium.org
Review URL: http://codereview.chromium.org/8496043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109160 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
This should fix the component build on Windows.
Review URL: http://codereview.chromium.org/8502041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109156 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8357030
TBR=viettrungluu@chromium.org
Review URL: http://codereview.chromium.org/8499045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109153 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Turned Mac valgrind bots red:
OutOfProcessPPAPITest.PostMessage_MessageEvent
OutOfProcessPPAPITest.PostMessage_ExtraParam
PPAPITest.PostMessage_NoHandler
OutOfProcessPPAPITest.PostMessage_SendInInit
Probably others
- Make it possible to enable/disable specific ppapi tests. Migrate PostMessage tests.
Most of these files were changed by a sed script, so it's not as bad as it looks.
The testcase attribute now can include a 'filter'. If it's omitted, everything works the same as before. This way we can migrate tests over bit-by-bit if we want to. We can also still run the tests manually the same way as before.
This only runs PostMessage testss the new way, and re-enables all oop PostMessage tests that pass on Windows. I can do the other tests in this CL if desired, but it might be easier to land in a few pieces.
BUG=102885,95557
TEST=N/A
Review URL: http://codereview.chromium.org/8477015
TBR=dmichael@chromium.org
Review URL: http://codereview.chromium.org/8497030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109142 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
tests.
Most of these files were changed by a sed script, so it's not as bad as it looks.
The testcase attribute now can include a 'filter'. If it's omitted, everything works the same as before. This way we can migrate tests over bit-by-bit if we want to. We can also still run the tests manually the same way as before.
This only runs PostMessage testss the new way, and re-enables all oop PostMessage tests that pass on Windows. I can do the other tests in this CL if desired, but it might be easier to land in a few pieces.
BUG=102885,95557
TEST=N/A
Review URL: http://codereview.chromium.org/8477015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109114 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8357030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109086 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implicitly coercing an integer type to bool gives a warning on Windows.
BUG= none
TEST= none
R=sehr@google.com
Review URL: http://codereview.chromium.org/8479045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108941 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
The comments state that PP_ERROR_BADARGUMENT is returned. The actual
behavior is that the loader returns PP_ERROR_NOACCESS.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8479040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108931 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
SetProperty accepts plenty of invalid values. The error checking is delayed until url loading time.
BUG=94790
TEST=this is
Review URL: http://codereview.chromium.org/8478019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108926 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=101600,94925
TEST=none
TBR=vandebo,viettrungluu
Review URL: http://codereview.chromium.org/8487001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108838 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
byAssociatedURLLoader now.
BUG=none
TEST=ui_tests,nacl_integration
Review URL: http://codereview.chromium.org/8359009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108801 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=use pp::VarPrivate.AsScriptableObject in out-of-process pepper plugin
Review URL: http://codereview.chromium.org/8476004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108746 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
translator to the linker.
BUG= http://code.google.com/p/nativeclient/issues/detail?id=2409
TEST= ./scons run_pnacl_example_browser_test
Review URL: http://codereview.chromium.org/8463007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108710 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Change the default 'range' for the presubmit and generator to be start,end
which represent the first and last Label found in the IDL.
BUG= http://code.google.com/p/chromium/issues/detail?id=103037\
TRY= ./generate.py
Review URL: http://codereview.chromium.org/8478005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108697 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
http://build.chromium.org/p/chromium/builders/Win%20Builder%202010%20%28dbg%29/builds/12897
This CL is based on the patch:
http://codereview.chromium.org/8416025
Use GYP to build nacl_irt
Converts Chrome from using scons to using GYP when building nacl_irt.
This change causes chrome to use GYP based components for untrusted
code. As a benifit, GYP will no longer need to run and scrape scons
output to determine irt dependencies. This also allows us to avoid
invoking scons during the build process.
We roll NaCl deps to get a fix for a simultaneous 32/64 bit build
issue on Windows.
This broke the mac build due to a missing dependency where pthread.h was
unavailable. This header gets copied to the correct location during the
prep_toolchain action. I've added the appropriate dependency shown bellow
to the original patch.
This add the addition dependency to ppruntime
'dependencies': [
'<(DEPTH)/native_client/src/tools.gyp:prep_toolchain'
],
TEST= try
BUG= http://code.google.com/p/chromium/issues/detail?id=96782
Review URL: http://codereview.chromium.org/8440004
TBR=noelallen@google.com
Review URL: http://codereview.chromium.org/8423080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108446 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The state of fullscreen can change other than by the script calling
requestFullScreen()/cancelFullScreen(), so the ToggleFullscreen function should
query document.webkitIsFullScreen instead of maintaining its own state.
BUG=100531
TEST=none
Review URL: http://codereview.chromium.org/8351090
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108435 0039d316-1c4b-4281-b951-d872f2087c98
|