| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL adds a new IPC Channel between NaCl plugin and the renderer process
with introducing ManifestService (in the plugin) and ManifestServiceChannel
(in the renderer) as its end points.
Currently, ManifestService is just an empty service. Its functions will be
added in following CLs. The service will be used only for non-SFI mode
as a first step. On other platforms, IPC Channel will not be created.
TEST=Ran trybots.
BUG=358431
Review URL: https://codereview.chromium.org/231793003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264477 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL has some refactoring to prepare implementation of EmbedderService.
1) Moves irt_ppapi.c, plugin_main_irt.cc and plugin_main_irt.h to
ppapi/nacl_irt.
2) Introduces irt_start.cc to handle startup of the plugin in SFI-mode.
The file is extracted from irt_ppapi.cc. The main purpose of this separation
is to avoid conflicting unistd.h. irt_ppapi.cc needs to include
native_client/src/trusted/service_runtime/include/sys/unistd.h.
However, start up code needs to include base/at_exit.h which indirectly
include unistd.h. As a result they conflict, and
NACL_ABI__SC_NACL_PNACL_MODE sometimes can be used, but sometimes not.
3) Moves AtExitManager, IOThread creation and global FD setting to
irt_start.cc and plugin_startup.cc.
TEST=Ran trybots
BUG=358431
Review URL: https://codereview.chromium.org/225093006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@262649 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
adds a function named SetPluginToHandleFindRequests which can be called by embedded plugins which want to handle browser find. This is needed for the new out of process PDF plugin, which will be embedded in an extensions page.
BUG=303491
R=jam@chromium.org, jschuh@chromium.org, yzshen@chromium.org
Review URL: https://codereview.chromium.org/188323002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@258171 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By this CL, plugin starts to talk with hosts via IPC.
For that purpose, ppapi_proxy is linked to nacl_helper (temporarily). This increase the size of nacl_helper, but when we split nacl_helper for non-sfi mode, it'll be resolved.
In SFI mode, this CL shouldn't affect the concept of IPC
channel connection between the plugin and the hosts. We
still use NaClIPCAdapter to wrap the IPC channel, and
NaClDesc for the plugin-side IPC file descriptors.
In non-SFI mode, we neither intercept nor rewrite the
message using NaClIPCAdapter, and the channels are
connected between the plugin and the hosts directly.
Note: plugin_main_nacl.cc is renamed to plugin_main.cc, because files with _nacl.cc suffix are automatically excluded from the sources of non-untrusted libs.
This increases the size of nacl_helper (temporarily) intentionally.
GYP_DEFINES="target_arch=ia32 remove_webcore_debug_symbols=1 linux_strip_symbols=1" GYP_GENERATORS="ninja" gclient runhooks
Before:
text data bss dec hex filename
1469882 15576 108644 1594102 1852f6 out/Release/nacl_helper
After:
text data bss dec hex filename
5641443 124636 126980 5893059 59ebc3 out/Release/nacl_helper
GYP_DEFINES="target_arch=x64 remove_webcore_debug_symbols=1 linux_strip_symbols=1" GYP_GENERATORS="ninja" gclient runhooks
Before:
text data bss dec hex filename
2063530 27910 213872 2305312 232d20 out/Release/nacl_helper
After:
text data bss dec hex filename
6304467 234424 247984 6786875 678f3b out/Release/nacl_helper
BUG=https://code.google.com/p/nativeclient/issues/detail?id=3734
TEST=Ran trybot.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252503
Review URL: https://codereview.chromium.org/140573003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252556 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
This reverts commit 3d41a2a63bbd307fcee6e6c5547ce5fe29e2f4a5.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252506 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By this CL, plugin starts to talk with hosts via IPC.
For that purpose, ppapi_proxy is linked to nacl_helper (temporarily). This increase the size of nacl_helper, but when we split nacl_helper for non-sfi mode, it'll be resolved.
In SFI mode, this CL shouldn't affect the concept of IPC
channel connection between the plugin and the hosts. We
still use NaClIPCAdapter to wrap the IPC channel, and
NaClDesc for the plugin-side IPC file descriptors.
In non-SFI mode, we neither intercept nor rewrite the
message using NaClIPCAdapter, and the channels are
connected between the plugin and the hosts directly.
Note: plugin_main_nacl.cc is renamed to plugin_main.cc, because files with _nacl.cc suffix are automatically excluded from the sources of non-untrusted libs.
This increases the size of nacl_helper (temporarily) intentionally.
GYP_DEFINES="target_arch=ia32 remove_webcore_debug_symbols=1 linux_strip_symbols=1" GYP_GENERATORS="ninja" gclient runhooks
Before:
text data bss dec hex filename
1469882 15576 108644 1594102 1852f6 out/Release/nacl_helper
After:
text data bss dec hex filename
5641443 124636 126980 5893059 59ebc3 out/Release/nacl_helper
GYP_DEFINES="target_arch=x64 remove_webcore_debug_symbols=1 linux_strip_symbols=1" GYP_GENERATORS="ninja" gclient runhooks
Before:
text data bss dec hex filename
2063530 27910 213872 2305312 232d20 out/Release/nacl_helper
After:
text data bss dec hex filename
6304467 234424 247984 6786875 678f3b out/Release/nacl_helper
BUG=https://code.google.com/p/nativeclient/issues/detail?id=3734
TEST=Ran trybot.
Review URL: https://codereview.chromium.org/140573003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252503 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
For audio, a frame usually has the same meaning as sample,
so AudioFrame is not a good name for a bunch of audio
samples. Change it to AudioBuffer.
BUG=330851
Review URL: https://codereview.chromium.org/156863005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250460 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will allow irt_ppapi.{c,h} to be dropped from the NaCl side.
Until then, there are two copies, and which copy gets linked in will
depend on library search order.
This creates a new directory, ppapi/nacl_irt/, for irt_ppapi.h since
it defines structs that are part of NaCl's stable ABI. This new
directory is therefore similar to ppapi/c/, but covers how PPAPI is
hooked up in NaCl. This also avoids any checkdeps problems with
#including from ppapi/proxy/.
I made the following changes to irt_ppapi.{c,h} when moving them:
* Changed "*" spacing to follow Chromium style.
* Added a comment for the assignment to "g_is_main_thread".
BUG= https://code.google.com/p/nativeclient/issues/detail?id=3787
TEST= tested browser_tests with irt_ppapi.c removed from the NaCl side
Review URL: https://codereview.chromium.org/136333013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250319 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
This makes the Rotate Clockwise / Counterclockwise context menu items work.
Review URL: https://codereview.chromium.org/142413006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249366 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=jamesr@chromium.org
BUG=330851
Review URL: https://codereview.chromium.org/140783004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249245 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=83774
R=bbudge@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247473
Review URL: https://codereview.chromium.org/69663002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247546 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> PPAPI: Implement PPB_FileMapping on POSIX
>
> BUG=83774
> R=bbudge@chromium.org
>
> Review URL: https://codereview.chromium.org/69663002
TBR=dmichael@chromium.org
Review URL: https://codereview.chromium.org/144383006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247477 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=83774
R=bbudge@chromium.org
Review URL: https://codereview.chromium.org/69663002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247473 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=317833
Review URL: https://codereview.chromium.org/61643022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246962 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=330851
Review URL: https://codereview.chromium.org/128683003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245150 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
resource reply messages.
BUG=269737
TEST=None
Review URL: https://codereview.chromium.org/46433002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@240282 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change generalizes PPB_Ext_CrxFileSystem_Private, built on the isolated
filesystem, so that we can easily reuse its components when introducing new
isolated filesystem in PPAPI.
This adds PPB_IsolatedFileSystem_Private API and enum indicating isolated
filesystem type. PPB_Ext_CrxFileSystem_Private API is still remaining since
some plugins may use the API. Both API share pepper resource and host
implementations.
BUG=286242
TEST=manual (see [1] and [2])
TBR=cpu@chromium.org
[1] https://code.google.com/p/chromium/issues/detail?id=271126#c7
[2] https://codereview.chromium.org/59203002/
Review URL: https://codereview.chromium.org/51653004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233807 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=256538
R=cpu@chromium.org, dalecurtis@chromium.org, dmichael@chromium.org, marcheu@chromium.org, wuchengli@chromium.org
Review URL: https://codereview.chromium.org/24039002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225849 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ResourceVar is now an abstract base class with subclasses
HostResourceVar and PluginResourceVar. The PluginResourceVar has a
reference counted Resource instead of a PP_Resource.
VarTracker has MakeResourceVar and MakeResourcePPVar methods, to
abstract over the creation of a resource var of the correct subclass.
Also, the creation_message is now NULL when empty, instead of being an
empty message object.
BUG=290713
Review URL: https://chromiumcodereview.appspot.com/23809016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224717 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves the FileRef implementation from the previous one in the "old"
resource model (ppb_file_ref_impl.cc) to the "new" resource model
(pepper_file_ref_host.cc), and from the renderer to the browser.
As many as possible of the supporting changes were split off to other changes
to minimize the size of this change. Unfortunately, a lot of changes for
URLLoader had to be rolled into this change.
The data structures for CreateInfo have changed, and all users of FileRef have
to be moved over, which is what causes this change to be so large.
TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
BUG=225441
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=218305
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=219911
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221284
Review URL: https://chromiumcodereview.appspot.com/21966004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223963 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Plumbs all relevant sections of the PPAPI portions of the
API for ChromeOS only. The test is currently disabled
pending changes to implement the UI portions of this
feature.
BUG=270294
TEST=browser_tests --gtest_filter=*PlatformVerification*
TBR=mnissler
Review URL: https://chromiumcodereview.appspot.com/23523028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223687 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new proxy is based on ppapi::proxy::PluginResource and
ppapi::host::ResourceHost which simplifies code significantly. Also
the permission check is consistent with socket APIs now.
BUG=281781
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223482
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223494
R=brettw@chromium.org, yzshen@chromium.org
Review URL: https://codereview.chromium.org/23819033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223535 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=sergeyu@chromium.org
BUG=281781
Review URL: https://codereview.chromium.org/23514062
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223497 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new proxy is based on ppapi::proxy::PluginResource and
ppapi::host::ResourceHost which simplifies code significantly. Also
the permission check is consistent with socket APIs now.
BUG=281781
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223482
Review URL: https://chromiumcodereview.appspot.com/23819033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223494 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Simplify PPB_NetworkMonitor proxy.
>
> The new proxy is based on ppapi::proxy::PluginResource and
> ppapi::host::ResourceHost which simplifies code significantly. Also
> the permission check is consistent with socket APIs now.
>
> BUG=281781
>
> Review URL: https://chromiumcodereview.appspot.com/23819033
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/23463037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223484 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new proxy is based on ppapi::proxy::PluginResource and
ppapi::host::ResourceHost which simplifies code significantly. Also
the permission check is consistent with socket APIs now.
BUG=281781
Review URL: https://chromiumcodereview.appspot.com/23819033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223482 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Pepper: Move FileRef to the "new" resource proxy.
>
> This change moves the FileRef implementation from the previous one in the "old"
> resource model (ppb_file_ref_impl.cc) to the "new" resource model
> (pepper_file_ref_host.cc), and from the renderer to the browser.
>
> As many as possible of the supporting changes were split off to other changes
> to minimize the size of this change. Unfortunately, a lot of changes for
> URLLoader had to be rolled into this change.
>
> The data structures for CreateInfo have changed, and all users of FileRef have
> to be moved over, which is what causes this change to be so large.
>
> TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
> BUG=225441
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=218305
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=219911
>
> Review URL: https://codereview.chromium.org/21966004
TBR=teravest@chromium.org
Review URL: https://codereview.chromium.org/23647008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221544 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves the FileRef implementation from the previous one in the "old"
resource model (ppb_file_ref_impl.cc) to the "new" resource model
(pepper_file_ref_host.cc), and from the renderer to the browser.
As many as possible of the supporting changes were split off to other changes
to minimize the size of this change. Unfortunately, a lot of changes for
URLLoader had to be rolled into this change.
The data structures for CreateInfo have changed, and all users of FileRef have
to be moved over, which is what causes this change to be so large.
TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
BUG=225441
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=218305
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=219911
Review URL: https://codereview.chromium.org/21966004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221284 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There have been multiple test failures among users of the CRX filesystem, which
is not adequately tested by existing tests. I'm not sure at all what's broken,
but I guess I'll have to do manual testing to sort it out.
> Pepper: Move FileRef to the "new" resource proxy.
>
> This change moves the FileRef implementation from the previous one in the "old"
> resource model (ppb_file_ref_impl.cc) to the "new" resource model
> (pepper_file_ref_host.cc), and from the renderer to the browser.
>
> As many as possible of the supporting changes were split off to other changes
> to minimize the size of this change. Unfortunately, a lot of changes for
> URLLoader had to be rolled into this change.
>
> The data structures for CreateInfo have changed, and all users of FileRef have
> to be moved over, which is what causes this change to be so large.
>
> TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
> BUG=225441
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=218305
>
> Review URL: https://chromiumcodereview.appspot.com/21966004
TBR=teravest@chromium.org
Review URL: https://codereview.chromium.org/23462016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220165 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=230784
TEST=browser_tests:*TCPSocket*, *TCPServerSocket*
Review URL: https://chromiumcodereview.appspot.com/22923014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220073 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves the FileRef implementation from the previous one in the "old"
resource model (ppb_file_ref_impl.cc) to the "new" resource model
(pepper_file_ref_host.cc), and from the renderer to the browser.
As many as possible of the supporting changes were split off to other changes
to minimize the size of this change. Unfortunately, a lot of changes for
URLLoader had to be rolled into this change.
The data structures for CreateInfo have changed, and all users of FileRef have
to be moved over, which is what causes this change to be so large.
TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
BUG=225441
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=218305
Review URL: https://chromiumcodereview.appspot.com/21966004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219911 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change is being reverted because it was causing some crashes for an
application that I haven't been able to debug yet.
> Pepper: Move FileRef to the "new" resource proxy.
>
> This change moves the FileRef implementation from the previous one in the "old"
> resource model (ppb_file_ref_impl.cc) to the "new" resource model
> (pepper_file_ref_host.cc), and from the renderer to the browser.
>
> As many as possible of the supporting changes were split off to other changes
> to minimize the size of this change. Unfortunately, a lot of changes for
> URLLoader had to be rolled into this change.
>
> The data structures for CreateInfo have changed, and all users of FileRef have
> to be moved over, which is what causes this change to be so large.
>
> TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
> BUG=225441
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
>
> Review URL: https://chromiumcodereview.appspot.com/21966004
TBR=teravest@chromium.org
Review URL: https://codereview.chromium.org/22901012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218544 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves the FileRef implementation from the previous one in the "old"
resource model (ppb_file_ref_impl.cc) to the "new" resource model
(pepper_file_ref_host.cc), and from the renderer to the browser.
As many as possible of the supporting changes were split off to other changes
to minimize the size of this change. Unfortunately, a lot of changes for
URLLoader had to be rolled into this change.
The data structures for CreateInfo have changed, and all users of FileRef have
to be moved over, which is what causes this change to be so large.
TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
BUG=225441
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216744
Review URL: https://chromiumcodereview.appspot.com/21966004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218305 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Pepper: Move FileRef to the "new" resource proxy.
>
> This change moves the FileRef implementation from the previous one in the "old"
> resource model (ppb_file_ref_impl.cc) to the "new" resource model
> (pepper_file_ref_host.cc), and from the renderer to the browser.
>
> As many as possible of the supporting changes were split off to other changes
> to minimize the size of this change. Unfortunately, a lot of changes for
> URLLoader had to be rolled into this change.
>
> The data structures for CreateInfo have changed, and all users of FileRef have
> to be moved over, which is what causes this change to be so large.
>
> TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
> BUG=225441
>
> Review URL: https://codereview.chromium.org/21966004
TBR=teravest@google.com
Review URL: https://codereview.chromium.org/22903002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217036 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves the FileRef implementation from the previous one in the "old"
resource model (ppb_file_ref_impl.cc) to the "new" resource model
(pepper_file_ref_host.cc), and from the renderer to the browser.
As many as possible of the supporting changes were split off to other changes
to minimize the size of this change. Unfortunately, a lot of changes for
URLLoader had to be rolled into this change.
The data structures for CreateInfo have changed, and all users of FileRef have
to be moved over, which is what causes this change to be so large.
TBR=dmichael@chromium.org, jschuh@chromium.org, yzshen@chromium.org
BUG=225441
Review URL: https://codereview.chromium.org/21966004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216744 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=230784
TEST=browser_tests:*TCPServerSocketPrivate
Review URL: https://chromiumcodereview.appspot.com/19005006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215597 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Needed to break dependency of ppapi_proxy on webkit code.
resource_creation_proxy now creates BrowserFontResource_Trusted
by going through PluginGlobals -> PluginProxyDelegate implemented
in content by PpapiThread.
R=jam@chromium.org, yzshen@chromium.org
BUG=237249
Review URL: https://codereview.chromium.org/20214004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213684 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The proxy doesn't currently extend PPB_FileRef_API because the this class and
the current implementation return different types for GetCreateInfo().
This change was tested (as part of a larger local change) and passed tests in
FileRef.
BUG=225441
Review URL: https://chromiumcodereview.appspot.com/16225007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208613 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds support for PPB_NetworkProxy_Dev for out-of-process and NaCl (when using --enable-nacl).
Work still to do in future CLs:
- Add permissions checking
- Translate appropriate error codes from net_errors.h
BUG=247225
R=jam@chromium.org, palmer@chromium.org, yzshen@chromium.org
Review URL: https://codereview.chromium.org/16819002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208259 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
I need to add whitelist testing to prevent arbitrary NaCl apps from using these resources.
BUG=230980
TEST=browser_tests, --gtest_filter="PPAPINaClPNaClTest.Video*"
Review URL: https://chromiumcodereview.appspot.com/16335018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207348 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change exposes the PPB_HostResolver_Dev interface and makes it to share the same backend as PPB_HostResolver_Private.
It doesn't include apps permission check, which will be implemented in separate CLs.
BUG=247225
TEST=newly added test_host_resolver.{h,cc}.
Review URL: https://chromiumcodereview.appspot.com/16727002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206321 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change exposes the PPB_UDPSocket_Dev interface and makes it to share the same backend as PPB_UDPSocket_Private.
It doesn't include:
- apps permission check;
- UDP socket options that PPB_UDPSocket_Private doesn't support.
These will be implemented in separate CLs.
BUG=247225
TEST=newly added test_udp_socket.{h,cc}.
Review URL: https://chromiumcodereview.appspot.com/16282005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206183 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change exposes the PPB_TCPSocket_Dev interface and makes it to share the same backend as PPB_TCPSocket_Private.
It doesn't include:
- apps permission check;
- TCP socket options that PPB_TCPSocket_Private doesn't support.
These will be implemented in separate CLs.
BUG=247225
TEST=newly added test_tcp_socket.{h,cc}.
Review URL: https://chromiumcodereview.appspot.com/16667002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206014 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=247225
TEST=Newly added tests.
Review URL: https://chromiumcodereview.appspot.com/16331007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205176 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is mainly a mechanical change to rename PPB_Flash_DeviceID to PPB_Flash_DRM. It deprecates the old interface. The reason is that we will be adding more DRM-related APIs and it makes sense to put them into a single interface.
BUG=242241
TBR=brettw for chrome_browser.gypi
Review URL: https://chromiumcodereview.appspot.com/15491006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202286 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change is essentially the same as: https://codereview.chromium.org/11416363
Here's the description from that CL:
"This doesn't use the resource call/reply infrastructure, but rather pipes WebKit callbacks to the
plugin via unsolicited callbacks. This eliminates state tracking in the host which makes things
simpler.
This fixes some bugs in Close() as well to fix the below-mentioned bug."
Other things contained in the original CL:
- Add a GetPluginPID method to RendererPpapiHost. This is needed when the loader host Opens()
a request.
- Add a HandleDocumentLoad method to PluginDelegate and implements it in PepperPluginDelegateImpl.
This creates the host for both in- and out-of-process proxies.
- Removes the GetURLLoaderBufferedBytes function from the PPB_Proxy_Private interface.
- Removes the HandleDocumentLoad implementation in the PPP_Instance_Proxy class.
- Removes the document_loader_ field from webkit::ppapi::WebPluginImpl and changes the
implementation to forward document load notifications to the PluginInstance.
- Changes the PluginInstance to manage the document loader.
This CL differs from the original in two ways.
First, the trusted interface keeps the RegisterStatusCallback function. The NaCl plugin relies on
this to generate progress messages back to the embedding page.
Second, PluginInstance is changed to recognize when it's a NaCl instance, and to defer calling
the plugin delegate's HandleDocumentLoad method until after the proxy is switched. In the meantime,
it saves document events in a special loader object. When the proxy is switched, the delegate's
HandleDocumentLoad method is called and the response and document events are then replayed
through the new loader resource.
BUG=69457
R=brettw@chromium.org
Review URL: https://codereview.chromium.org/14371021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200412 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
pp::ExtCrxFileSystemPrivate is introduced in this change to allow plugin to mount (readonly) CRX extension directory as a filesystem. Files can be access through pp::ExtCrxFileRefPrivate (which is a subclass of pp::FileRef) just like normal file, and the path would look like "/manifest.json". See ppapi/example/crxfs for example.
Some keypoints in this change:
* pepper resource/host architecture:
- please refer to ppapi/proxy/ext_crx_file_system_private_resource.h.
* webkit/fileapi related:
- Changes run in browser
- Isoloated filesystem is the underlying filesystem
- Grant read permission to corresponding renderer of the plugin
- See chrome/browser/renderer_host/pepper/pepper_ext_crx_file_system_browser_host.cc
* extension related:
- Changes run in browser
- This is for getting extension installed directory to mount
TEST=out/Debug/chrome
--register-pepper-plugins="out/Debug/lib/libppapi_example_crxfs.so#PPAPI Tests##1.2.3;application/x-ppapi-example-crxfs" \
ppapi/examples/crxfs/crxfs.html
BUG=223301
Review URL: https://chromiumcodereview.appspot.com/14188019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198938 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds IPC messages to support the new resources.
Adds stubbed out host methods to hold implementation
TBR=tsepez@chromium.org, yzshen@chromium.org
BUG=230980
TESTS=none
Review URL: https://codereview.chromium.org/13771020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198258 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This also means this API becomes a stable API.
While DirectoryReader was using the new pepper proxy API, FileRef is
using the old one. As updating FileRef would take some time, the
implementation of ReadEntries is converted from the new design to the
old one for now.
BUG=234513
TEST=browser_tests
R=avi@chromium.org, binji@chromium.org, dmichael@chromium.org, palmer@chromium.org, raymes@chromium.org, teravest@chromium.org
Review URL: https://codereview.chromium.org/14784002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198204 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add global FileSystem registry for new FileSystem Support later. New file system will need to implement PepperFileSystemRegistry::FileSystemPeeker for FileRef to access.
Due to the fact that 1) FileSystem and FileRef are tightly couple, 2) we wants to move both of them to browser together, there are two temporary changes in this CL.
1. webkit/plugins/ppapi/ppb_file_ref_impl.cc needs some delegation to access content::PepperFileSystemHost. The delegation code should be removed after FileRef is moved to content.
2. PepperFileSystemHost stays in renderer for now, and will be moved to browser together in the FileRef refactoring.
Also, NullFileSystemCallbackDispatcher is added with default NOTREACHED callbacks.
TEST=browser_tests --gtest_filter='*PPAPI*.*File*'
BUG=227033
Review URL: https://chromiumcodereview.appspot.com/13726024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193995 0039d316-1c4b-4281-b951-d872f2087c98
|