| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Prior to this change //mojo/environment depended upon //mojo/common. //mojo/common's deps are wrong and it needs to be depend upon various things in //third_party/mojo/src/mojo... When these DEPS are added link errors result because Environment and and a couple of other things need to be defined. These are defined in //mojo/environment. But //mojo/environment can't be a dep of //mojo/common, else we get a cycle.
To straighten this out I've moved the files //mojo/environment needs from //mojo/common to //mojo/message_pump. //mojo/environment no longer depends upon //mojo/common, only //mojo/message_pump. //mojo/common can then depend upon //mojo/environment and we no longer have the cycle. Yay!
BUG=none
TEST=none
Committed: https://crrev.com/a11b4d76a27816fe025af647825becac5f633c34
Cr-Commit-Position: refs/heads/master@{#341979}
Review URL: https://codereview.chromium.org/1262173005
Cr-Commit-Position: refs/heads/master@{#342252}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1262173005/ )
Reason for revert:
Broke MediaRouterMojoImplTest.QueuedWhileAsleep on linux debug.
Original issue's description:
> Straightens outs DEPS in mojo/common
>
> Prior to this change //mojo/environment depended upon //mojo/common. //mojo/common's deps are wrong and it needs to be depend upon various things in //third_party/mojo/src/mojo... When these DEPS are added link errors result because Environment and and a couple of other things need to be defined. These are defined in //mojo/environment. But //mojo/environment can't be a dep of //mojo/common, else we get a cycle.
>
> To straighten this out I've moved the files //mojo/environment needs from //mojo/common to //mojo/message_pump. //mojo/environment no longer depends upon //mojo/common, only //mojo/message_pump. //mojo/common can then depend upon //mojo/environment and we no longer have the cycle. Yay!
>
> BUG=none
> TEST=none
>
> Committed: https://crrev.com/a11b4d76a27816fe025af647825becac5f633c34
> Cr-Commit-Position: refs/heads/master@{#341979}
TBR=jam@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=none
Review URL: https://codereview.chromium.org/1280463003
Cr-Commit-Position: refs/heads/master@{#342014}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Prior to this change //mojo/environment depended upon //mojo/common. //mojo/common's deps are wrong and it needs to be depend upon various things in //third_party/mojo/src/mojo... When these DEPS are added link errors result because Environment and and a couple of other things need to be defined. These are defined in //mojo/environment. But //mojo/environment can't be a dep of //mojo/common, else we get a cycle.
To straighten this out I've moved the files //mojo/environment needs from //mojo/common to //mojo/message_pump. //mojo/environment no longer depends upon //mojo/common, only //mojo/message_pump. //mojo/common can then depend upon //mojo/environment and we no longer have the cycle. Yay!
BUG=none
TEST=none
Review URL: https://codereview.chromium.org/1262173005
Cr-Commit-Position: refs/heads/master@{#341979}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This embeds mojo/shell's ApplicationManager in content/browser
and provides a way for arbitrary browser code to connect to
Mojo apps as if the browser itself were a Mojo app.
This is a basic implementation of Mojo app support which only
loads static apps either in the browser process or a (per-app)
utility process.
Future CLs will address connection to apps from arbitrary render
frames (i.e. connection requests which include the requestor's
origin) as well as refactoring the utility process code further
so that it serves strictly as a Mojo app runner.
BUG=492422
Review URL: https://codereview.chromium.org/1149833007
Cr-Commit-Position: refs/heads/master@{#332974}
|
|
|
|
|
|
|
|
| |
BUG=None
Review URL: https://codereview.chromium.org/1138003007
Cr-Commit-Position: refs/heads/master@{#330393}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For ensuring that the blink is always initialized with WebThread, the
call sites of blink must implement WebThread to expose the task runner.
UtilityThread is the only case where blink is run without exposing the
platform thread. This CL adds new WebThread implementation for
UtilityThread. After this change blink will always initialized with
platform thread and task runner exposed.
BUG=486782
Review URL: https://codereview.chromium.org/1133563007
Cr-Commit-Position: refs/heads/master@{#329442}
|
|
|
|
|
|
|
|
|
|
| |
Includes updates to ipc/mojo/ipc_channel_mojo.cc for mojo::embedder API
changes and updates to use mojo::Binding<> in ui/keyboard and
device/battery.
Review URL: https://codereview.chromium.org/728133002
Cr-Commit-Position: refs/heads/master@{#304731}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The targets in the mojo public SDK, i.e. those in mojo/public/, should
be free of dependencies outside of mojo/public/ and usable in a
standalone fashion. This moves the gyp definitions from
mojo/mojo_base.gyp, which also contains a number of non-public targets,
to mojo/public/mojo_public.gyp.
R=viettrungluu@chromium.org
TBR=darin@chromium.org for updating various gyp entries to point to new target location
Review URL: https://codereview.chromium.org/632853002
Cr-Commit-Position: refs/heads/master@{#298422}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/423613002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286725 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change extract essential part of Mojo, which is needed
to build chrome and its submodules, to mojo_base.gyp.
Other chrome submodules can safely depends on mojo_base.gyp
without worrying about excessive dependencies behind.
Caveat: mojo_service_manager stays on mojo.gyp even though
it is required to build chrome. This change leaves it
as it depends on net.gyp, which has non trivial dependencies.
BUG=397672
TEST=none
R=jamesr@chromium.org, viettrungluu@chromium.org, rockot@chromium.org
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/422163004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286627 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/358743002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280569 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
their respective hosts.
This introduces ServiceRegistry as an abstraction around providing
services to and accessing services from a remote peer. In particular,
this adds peered service registries to RenderProcessHost and
RenderThread, and to RenderFrameHost and RenderFrame - the RenderFrame
setup is implemented using the RenderProcessHost/RenderThread
ServiceRegistry pair.
This replaces the existing WebUI handle setup by adding a webUI
controller service to the frame host registry and a corresponding
request for the webUI controller service to the frame registry.
BUG=386155
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279557
Review URL: https://codereview.chromium.org/285333003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279623 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Try #2 of https://codereview.chromium.org/298653010/
Fixed newly introduced GN dependency.
TBR=darin@chromium.org
BUG=
Review URL: https://codereview.chromium.org/298003008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272989 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Change Shell / ShellClient to ServiceProvider
>
> BUG=
> R=darin@chromium.org
>
> Review URL: https://codereview.chromium.org/298653010
TBR=davemoore@chromium.org
Review URL: https://codereview.chromium.org/304593002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272984 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=
R=darin@chromium.org
Review URL: https://codereview.chromium.org/298653010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272983 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL introduces two new content classes:
- MojoApplicationHost encapsulates what's needed to host a Mojo App using Chrome IPC to bootstrap.
- MojoApplication represents what's needed to be a Mojo App using Chrome IPC to bootstrap.
The RenderProcess and RenderProcessHost interfaces are replaced with WebUISetup and WebUISetupClient interfaces. This way the interface is more specific to the service of setting up WebUI.
WebUISetupClient is empty and uninteresting. RenderProcessHostImpl no longer deals with WebUI setup. That is all done directly by RenderViewHostImpl by talking to the WebUISetup service.
Service names get defined in content/common/mojo/mojo_service_names.{h,cc}.
TBR=sky@chromium.org, tsepez@chromium.org
Originally reviewed at https://codereview.chromium.org/236813002/
Review URL: https://codereview.chromium.org/256403004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@265962 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is causing runhooks failures on the iOS build bots.
> Move Mojo channel initialization closer to IPC::Channel setup
>
> This CL introduces two new content classes:
> - MojoApplicationHost encapsulates what's needed to host a Mojo App using Chrome IPC to bootstrap.
> - MojoApplication represents what's needed to be a Mojo App using Chrome IPC to bootstrap.
>
> The RenderProcess and RenderProcessHost interfaces are replaced with WebUISetup and WebUISetupClient interfaces. This way the interface is more specific to the service of setting up WebUI.
>
> WebUISetupClient is empty and uninteresting. RenderProcessHostImpl no longer deals with WebUI setup. That is all done directly by RenderViewHostImpl by talking to the WebUISetup service.
>
> Service names get defined in content/common/mojo/mojo_service_names.{h,cc}.
>
> R=sky@chromium.org, tsepez@chromium.org
>
> Review URL: https://codereview.chromium.org/236813002
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/247953005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@265705 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL introduces two new content classes:
- MojoApplicationHost encapsulates what's needed to host a Mojo App using Chrome IPC to bootstrap.
- MojoApplication represents what's needed to be a Mojo App using Chrome IPC to bootstrap.
The RenderProcess and RenderProcessHost interfaces are replaced with WebUISetup and WebUISetupClient interfaces. This way the interface is more specific to the service of setting up WebUI.
WebUISetupClient is empty and uninteresting. RenderProcessHostImpl no longer deals with WebUI setup. That is all done directly by RenderViewHostImpl by talking to the WebUISetup service.
Service names get defined in content/common/mojo/mojo_service_names.{h,cc}.
R=sky@chromium.org, tsepez@chromium.org
Review URL: https://codereview.chromium.org/236813002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@265693 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Second part of a 2-part changelist.
The first part is https://codereview.chromium.org/25883006/
BUG=304879
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=259726
Review URL: https://codereview.chromium.org/25909005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260243 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(https://codereview.chromium.org/25909005/)
Reason for revert:
linux asan failure
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%2BLSan%20Tests%20%282%29/builds/901/steps/unit_tests/logs/DifferentialUpdateFails
Original issue's description:
> Use UtilityProcessHost to patch files.
>
> Second part of a 2-part changelist.
> The first part is https://codereview.chromium.org/25883006/
>
> BUG=304879
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=259726
TBR=sorin@chromium.org,dgarrett@chromium.org,sky@chromium.org,cdn@chromium.org,waffles@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=304879
Review URL: https://codereview.chromium.org/213923002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259740 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Second part of a 2-part changelist.
The first part is https://codereview.chromium.org/25883006/
BUG=304879
Review URL: https://codereview.chromium.org/25909005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259726 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Follow-up after http://crrev.com/217968 per comments on
https://chromiumcodereview.appspot.com/23235002/
R=jam@chromium.org
BUG=237249
Review URL: https://chromiumcodereview.appspot.com/23452016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221199 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This code was previously #ifdef'd out based on CHROME_MULTIPLE_DLL. This works
for chrome, but not for test targets which link content_browser. content_browser
needs to not link against child-only targets (as they'll cause linking blink
into the browser dll).
Instead of having utility_process_host_impl, et al. own the in-process
implementation, use a factory to create them that's installed in test code, and
in chrome for supporting --single-process.
At the same time, remove the global CHROME_MULTIPLE_DLL define and localize it
to chrome_exe.gypi because it's too easy to use incorrectly.
TBR=darin
R=piman@chromium.org,jam@chromium.org
BUG=237249
Review URL: https://chromiumcodereview.appspot.com/23235002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217968 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
ContentUtilityClient, and ContentPluginClient to make it easier to embed content.
BUG=98716
Review URL: https://chromiumcodereview.appspot.com/10449063
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139432 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Also, move all mac sandbox unittests to content
BUG=90443
TEST=content_unittests
Review URL: http://codereview.chromium.org/8589001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111614 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
interface is implemented
by the UtilityThreadImpl class in content\utility\utility_thread_impl.cc/.h which are renamed incarnations
of the content\utility\utility_thread.cc/.h files.
Changes as part of creating a content API.
BUG=98716
TEST=No change in functionality.
Review URL: http://codereview.chromium.org/8276026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105610 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Changes as part of creating a Content API.
BUG=98716
TEST=No change in functionality.
Review URL: http://codereview.chromium.org/8252005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105398 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a whitespace-only change to clean up from the refactoring of a
couple weeks ago for the component build.
R=jam@chromium.org
BUG=90442
TEST=none
Review URL: http://codereview.chromium.org/8046015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102800 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=darin
BUG=
TEST=
Review URL: http://codereview.chromium.org/7889027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101011 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=darin
BUG=
TEST=
Review URL: http://codereview.chromium.org/7892021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100996 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
contents of the library are pulled from individual gypis that are now just collections of settings.
There are three changes here ...
1) Move omx_video_decode_unittest into content_tests.gypi, and move a couple of other definitions into the top level content.gyp
2) Change all of the dependencies in the chrome gypi's from content_browser et al. to just "content"
3) Remove the "targets", "target_name", and "target_type" keys from the content_* gypis
This change depends on http://codereview.chromium.org/7697008/ landing first to ensure we have the right dependencies set for content.
BUG=90442
TEST=everything still compiles and links
Review URL: http://codereview.chromium.org/7618029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100988 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=87332
TEST=existing
Review URL: http://codereview.chromium.org/7242016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90910 0039d316-1c4b-4281-b951-d872f2087c98
|
|
to the clients. This allows the content layer to use the class.
BUG=76697
Review URL: http://codereview.chromium.org/6995095
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88586 0039d316-1c4b-4281-b951-d872f2087c98
|