| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make content/gpu and content/plugin pass GN check. Significant work for most of content/test
Makes most of content/shell and content/test pass check. These are not all enabled yet pending a few remaining issues that are more difficult to solve.
The private content targets now allow content/test to include headers from them for non-component builds. Checking is disabled for content/test in component builds as described in the comment at the top of //content/BUILD.gn
Renames the ui/events/ipc according to GN style.
Annotates all content and gpu visibility declarations with why the're there, and fixes some incorrect ones that people added without understanding.
Reland of https://codereview.chromium.org/1828483002/ with significant changes.
TBR=jschuh (non-security-sensitive ipc build changes)
Review URL: https://codereview.chromium.org/1833193002
Cr-Commit-Position: refs/heads/master@{#383358}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1828483002/ )
Reason for revert:
Broke https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20%28dbg%29/builds/19740/steps/compile/logs/stdio
Original issue's description:
> Content "gn check" work.
>
> Make content/gpu and content/plugin pass GN check.
>
> Makes most of content/shell and content/test pass check. These are not all enabled yet pending a few remaining issues that are more difficult to solve.
>
> The private content targets now allow content/test to include headers from them for non-component builds. Checking is disabled for content/test in component builds as described in the comment at the top of //content/BUILD.gn
>
> Renames the ui/events/ipc according to GN style
>
> BUG=
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
> TBR=jschuh (IPC build tweak)
>
> Committed: https://crrev.com/f510be61306f5e4986eceb1f8686bfcc934cd400
> Cr-Commit-Position: refs/heads/master@{#383208}
TBR=jam@chromium.org,jschuh@chromium.org,dcheng@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
Review URL: https://codereview.chromium.org/1836533002
Cr-Commit-Position: refs/heads/master@{#383289}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make content/gpu and content/plugin pass GN check.
Makes most of content/shell and content/test pass check. These are not all enabled yet pending a few remaining issues that are more difficult to solve.
The private content targets now allow content/test to include headers from them for non-component builds. Checking is disabled for content/test in component builds as described in the comment at the top of //content/BUILD.gn
Renames the ui/events/ipc according to GN style
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=jschuh (IPC build tweak)
Review URL: https://codereview.chromium.org/1828483002
Cr-Commit-Position: refs/heads/master@{#383208}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
'module content;' --> 'module content.mojom;'
This way generated bindings will consistently live in a mojom
sub-namespace and avoid collision with non-mojom types.
This CL is for these .mojom files:
content/common/application_setup.mojom
content/common/background_sync_service.mojom
content/common/geolocation_service.mojom
content/common/process_control.mojom
content/common/render_frame_setup.mojom
content/common/service_port_service.mojom
content/common/service_worker/embedded_worker_setup.mojom
content/public/common/background_sync.mojom
content/public/common/mojo_geoposition.mojom
content/public/common/service_worker_event_status.mojom
BUG=588964
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1758233002
Cr-Commit-Position: refs/heads/master@{#382512}
|
|
|
|
|
|
|
|
|
|
|
| |
Now initializeWithoutV8/shutdownWithoutV8 is idential to Platform::initialize/shutdown.
This CL removes initializeWithoutV8/shutdownWithoutV8.
BUG=593269
Review URL: https://codereview.chromium.org/1787643003
Cr-Commit-Position: refs/heads/master@{#380682}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes cases where renaming a method from thisStyle to ThisStyle
makes it collide with something, usually another method with the same
name.
For the case in PrintStream.h, the collision function is actually dead
code so remove it.
TBR=avi,pdr
R=pdr
BUG=584539
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1753023002
Cr-Commit-Position: refs/heads/master@{#378868}
|
|
|
|
|
|
|
|
|
| |
TBR=sky@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1750673002
Cr-Commit-Position: refs/heads/master@{#378260}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
type:path
Remove GURLs from all over the place. This eliminates the need to register Mojo and Exe schemes.
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1743473002
Cr-Commit-Position: refs/heads/master@{#378130}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL also renames the element type (WeakBinding/WeakInterfacePtr) to Element and hide them in the corresonding *Set class.
BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Committed: https://crrev.com/632d319842096e6998d174e2f251865979ce2b72
Cr-Commit-Position: refs/heads/master@{#377384}
Review URL: https://codereview.chromium.org/1735583002
Cr-Commit-Position: refs/heads/master@{#377528}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BindingSet/InterfacePtrSet."
This reverts commit 632d319842096e6998d174e2f251865979ce2b72.
TBR=jam@chromium.org
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1734803002 .
Cr-Commit-Position: refs/heads/master@{#377393}
|
|
|
|
|
|
|
|
|
|
|
| |
This CL also renames the element type (WeakBinding/WeakInterfacePtr) to Element and hide them in the corresonding *Set class.
BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1735583002
Cr-Commit-Position: refs/heads/master@{#377384}
|
|
|
|
|
|
|
|
|
| |
TBR=rockot@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1716473002
Cr-Commit-Position: refs/heads/master@{#376629}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL makes mojo media services highly configurable by
providing finer control on what service is enabled on which
platform. For example, on Android and Desktop, we want use the
default media Renderer but use the mojo CDM. On ChromeCast, we
want to use the mojo Renderer and the mojo CDM.
Tested with various gn argument combinations, e.g.
- mojo_media_host="utility", mojo_media_services=["cdm"]
Both clear and encrypted playback (Clear Key) work. In the
encrypted case, we use DecryptingDemuxerStream and
MojoDecryptor to do decrypt-only.
- mojo_media_host="utility", mojo_media_services=["renderer"]
Clear playback works using remote renderer. Encrypted playback
doesn't work because a remote renderer doesn't support a local
CDM.
- mojo_media_host="gpu", mojo_media_services=["renderer", "cdm"]
Both clear and encrypted playback work. In both cases a remote
renderer is used. In the encrytped case, the remote renderer
uses a remote CDM in the GPU process.
BUG=571247
TEST=See above.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1712033002
Cr-Commit-Position: refs/heads/master@{#376498}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Rename DefaultMojoMediaClient to TestMojoMediaClient.
- Add a new stub DefaultMojoMediaClient, which will be used by Chromium for
non-Android platforms.
- Add gn variable enable_test_mojo_media_client to enable TestMojoMediaClient
for Chromium builds. This can be used for local tests.
- Separate MojoMediaApplication from it's creation so that it's easier to create
MojoMediaApplication with a customized MojoMediaClient.
- Add CreateMojoMediaApplication() to create the app with default MojoMediaClient.
- App tests will always use TestMojoMediaClient.
- Update BUILD.gn to support all above.
BUG=570357
TEST=Tests still work. Tried enable_test_mojo_media_client locally.
Review URL: https://codereview.chromium.org/1702803002
Cr-Commit-Position: refs/heads/master@{#376193}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
They are:
- weak_binding_set.h
- weak_interface_ptr_set.h
BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Review URL: https://codereview.chromium.org/1692783002
Cr-Commit-Position: refs/heads/master@{#375180}
|
|
|
|
|
|
|
|
|
| |
TBR=sky@chromium.org
http://crbug.com/575308
Review URL: https://codereview.chromium.org/1565343003
Cr-Commit-Position: refs/heads/master@{#368776}
|
|
|
|
|
|
|
|
|
| |
BUG=557422
R=avi@chromium.org
Review URL: https://codereview.chromium.org/1544293002
Cr-Commit-Position: refs/heads/master@{#366905}
|
|
|
|
|
|
|
|
|
| |
BUG=138542
TBR=jam@chromium.org
Review URL: https://codereview.chromium.org/1544273002
Cr-Commit-Position: refs/heads/master@{#366875}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:40001 of https://codereview.chromium.org/1462753002/ )
Reason for revert:
Not needed.
Original issue's description:
> Define MOJO_SHELL_CLIENT uniformly across content
>
> Separated out of https://codereview.chromium.org/1452823003/
>
> TBR=jam@chromium.org
> http://crbug.com/551253
>
> Committed: https://crrev.com/5e739169f376c3bf9c2c35a4efa0ebb6cbb3a9ee
> Cr-Commit-Position: refs/heads/master@{#360514}
TBR=jam@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1455423002
Cr-Commit-Position: refs/heads/master@{#360522}
|
|
|
|
|
|
|
|
|
|
|
| |
Separated out of https://codereview.chromium.org/1452823003/
TBR=jam@chromium.org
http://crbug.com/551253
Review URL: https://codereview.chromium.org/1462753002
Cr-Commit-Position: refs/heads/master@{#360514}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1452823003/ )
Reason for revert:
This change is making hundreds of webkit_tests crash on linux_chromium_rel_ng with:
[1:1:1118/141621:108326049405:FATAL:lock.cc(32)] Check failed: owning_thread_ref_.is_null().
#0 0x00000067e2de base::debug::StackTrace::StackTrace()
#1 0x00000057a71f logging::LogMessage::~LogMessage()
#2 0x0000005eb69b base::Lock::CheckUnheldAndMark()
#3 0x000000518e1c base::Lock::Acquire()
#4 0x000000518b43 base::AutoLock::AutoLock()
#5 0x0000005b4014 base::SequenceCheckerImpl::CalledOnValidSequencedThread()
#6 0x000006f074ae IDMap<>::Lookup()
#7 0x000006f0598c content::RendererBlinkPlatformImpl::SetPlatformEventObserverForTesting()
#8 0x000008459264 content::SetMockGamepadProvider()
Original issue's description:
> Bind Application in renderer.
>
> This involved changing how we get the client handle to the renderer. In the first iteration I was passing this on the command line but that turns out not to work with the sandbox. So instead I an approach used by the Mojo-in-Chrome MojoApplication class and pass the primordial handle via Chrome IPC.
>
> I had to twiddle a bunch of BUILD.gn files in content to get this to work without crashing due to inconsistencies in how MOJO_SHELL_CLIENT was defined.
>
> R=jam@chromium.org,tsepez@chromium.org
> http://crbug.com/551253
>
> Committed: https://crrev.com/3edb97198bc5fbc22c5cf13286e8af80449ddfb9
> Cr-Commit-Position: refs/heads/master@{#360293}
>
> Committed: https://crrev.com/2c716f9d5ebff610641f4506c17ec81d4b89b7bb
> Cr-Commit-Position: refs/heads/master@{#360396}
TBR=jam@chromium.org,tsepez@chromium.org,ben@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1457173002
Cr-Commit-Position: refs/heads/master@{#360446}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This involved changing how we get the client handle to the renderer. In the first iteration I was passing this on the command line but that turns out not to work with the sandbox. So instead I an approach used by the Mojo-in-Chrome MojoApplication class and pass the primordial handle via Chrome IPC.
I had to twiddle a bunch of BUILD.gn files in content to get this to work without crashing due to inconsistencies in how MOJO_SHELL_CLIENT was defined.
R=jam@chromium.org,tsepez@chromium.org
http://crbug.com/551253
Committed: https://crrev.com/3edb97198bc5fbc22c5cf13286e8af80449ddfb9
Cr-Commit-Position: refs/heads/master@{#360293}
Review URL: https://codereview.chromium.org/1452823003
Cr-Commit-Position: refs/heads/master@{#360396}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1452823003/ )
Reason for revert:
Speculatively reverting this, as this CL landed exactly at the time when renderer started crashing on linux_chromium_rel_ng.
Original issue's description:
> Bind Application in renderer.
>
> This involved changing how we get the client handle to the renderer. In the first iteration I was passing this on the command line but that turns out not to work with the sandbox. So instead I an approach used by the Mojo-in-Chrome MojoApplication class and pass the primordial handle via Chrome IPC.
>
> I had to twiddle a bunch of BUILD.gn files in content to get this to work without crashing due to inconsistencies in how MOJO_SHELL_CLIENT was defined.
>
> R=jam@chromium.org,tsepez@chromium.org
> http://crbug.com/551253
>
> Committed: https://crrev.com/3edb97198bc5fbc22c5cf13286e8af80449ddfb9
> Cr-Commit-Position: refs/heads/master@{#360293}
TBR=jam@chromium.org,tsepez@chromium.org,ben@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557732
Review URL: https://codereview.chromium.org/1460713002
Cr-Commit-Position: refs/heads/master@{#360331}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This involved changing how we get the client handle to the renderer. In the first iteration I was passing this on the command line but that turns out not to work with the sandbox. So instead I an approach used by the Mojo-in-Chrome MojoApplication class and pass the primordial handle via Chrome IPC.
I had to twiddle a bunch of BUILD.gn files in content to get this to work without crashing due to inconsistencies in how MOJO_SHELL_CLIENT was defined.
R=jam@chromium.org,tsepez@chromium.org
http://crbug.com/551253
Review URL: https://codereview.chromium.org/1452823003
Cr-Commit-Position: refs/heads/master@{#360293}
|
|
|
|
|
|
|
|
|
| |
BUG=None
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/1410053006
Cr-Commit-Position: refs/heads/master@{#359461}
|
|
|
|
|
|
|
|
|
| |
BUG=521731
TEST=Can play audio in GPU process with MojoMediaApplication.
Review URL: https://codereview.chromium.org/1320453003
Cr-Commit-Position: refs/heads/master@{#353893}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This causes the utility process to shut itself down in the
event that it receives a ProcessControl::LoadApplication
request that it cannot fulfill.
BUG=526325
R=jam@chromium.org
Review URL: https://codereview.chromium.org/1312063003
Cr-Commit-Position: refs/heads/master@{#346523}
|
|
|
|
|
|
|
|
|
|
|
| |
List from: git gs base/memory/scoped_vector.h|cut -d: -f 1|uniq|xargs grep -L ScopedVector
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=keybuk@chromium.org
Review URL: https://codereview.chromium.org/1307643003
Cr-Commit-Position: refs/heads/master@{#344925}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Refactor UtilityProcessControlImpl and move common (non-utility-specific) code
into a base class ProcessControlImpl. UtilityProcessControlImpl inherits
ProcessControlImpl and has the utility specific code. This allows us to easily
add ProcessControl implementation to other processes, e.g. the GPU process.
BUG=521755
Review URL: https://codereview.chromium.org/1295363003
Cr-Commit-Position: refs/heads/master@{#344402}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
fires.
The functionality to do this opens up security holes. Currently this was working only for debug Linux builds. However our trybots are release builds, and we need to be able to see stack traces from processes on all platforms and not just Linux (i.e. to be able to debug the large flakiness that occurred last week). This is disabled for official builds.
Also make release (non-official) builds print the callstack on asserts, just like debug builds. This makes it easier to debug test failures on the CQ (for example, DCHECKs for non-threadsafe usage of pointers).
Add a regression test that both renderer and browser process crashes print the callstack.
BUG=517488,358267,521148
NOPRESUBMIT=true
Committed: https://crrev.com/8ba532e170befc312e66d032587fa2ad04bac975
Cr-Commit-Position: refs/heads/master@{#343240}
Review URL: https://codereview.chromium.org/1291553003
Cr-Commit-Position: refs/heads/master@{#343626}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #5 id:80001 of https://codereview.chromium.org/1291553003/ )
Reason for revert:
This is unfortunately breaking Windows 10 in a major way. I did a local bisect and narrowed it down to this CL.
BUG=521242
Original issue's description:
> Print stack traces in child processes when browser tests failed.
>
> The functionality to do this opens up security holes. Currently this was working only for debug Linux builds. However our trybots are release builds, and we need to be able to see stack traces from child processes on all platforms (i.e. to be able to debug the large flakiness that occurred since last week). This is disabled for official builds.
>
> BUG=517488,358267
>
> Committed: https://crrev.com/8ba532e170befc312e66d032587fa2ad04bac975
> Cr-Commit-Position: refs/heads/master@{#343240}
TBR=scottmg@chromium.org,jln@chromium.org,wfh@chromium.org,jam@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=517488,358267
Review URL: https://codereview.chromium.org/1299583002
Cr-Commit-Position: refs/heads/master@{#343603}
|
|
|
|
|
|
|
|
|
|
| |
The functionality to do this opens up security holes. Currently this was working only for debug Linux builds. However our trybots are release builds, and we need to be able to see stack traces from child processes on all platforms (i.e. to be able to debug the large flakiness that occurred since last week). This is disabled for official builds.
BUG=517488,358267
Review URL: https://codereview.chromium.org/1291553003
Cr-Commit-Position: refs/heads/master@{#343240}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, in a utility process containing a v8 proxy resolver, both
blink and the proxy resolver attempt to use v8 in incompatible ways,
causing a crash on shutdown. This cl changes the utility process to
only initialize blink when needed.
On Windows, initializing blink with v8 before enabling the sandbox
calls RtlGenRandom(), which leaves it available from within the sandbox.
To avoid breaking base::RandBytes(), this cl also adds a call to
base::RandBytes before enabling the sandbox.
BUG=506439
Review URL: https://codereview.chromium.org/1251823002
Cr-Commit-Position: refs/heads/master@{#340835}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Audited the sandbox files to enusre all rules are necessary and removed
any incorrect comments.
The OS X Sandbox code is refactored to get rid of all string processing
of the raw scheme code in the sandbox profiles. By using the ability to
pass parameters into sandbox profiles, the variable substitution logic
is performed by the scheme interpreter itself.
BUG=242704
Committed: https://crrev.com/76eaa8778a322803dc3ac20cda05525e615dec4d
Cr-Commit-Position: refs/heads/master@{#336610}
Review URL: https://codereview.chromium.org/1186233004
Cr-Commit-Position: refs/heads/master@{#339038}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#6 id:120001 of https://codereview.chromium.org/1186233004/)
Reason for revert:
Speculative revert to see if this could be a possible reason for MacOS 10.6 layout test bot failures starting since http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.6/builds/41745
Original issue's description:
> Refactor OS X sandbox processing and audit sandbox files
>
> Audited the sandbox files to enusre all rules are necessary and removed
> any incorrect comments.
>
> The OS X Sandbox code is refactored to get rid of all string processing
> of the raw scheme code in the sandbox profiles. By using the ability to
> pass parameters into sandbox profiles, the variable substitution logic
> is performed by the scheme interpreter itself.
>
> BUG=242704
>
> Committed: https://crrev.com/76eaa8778a322803dc3ac20cda05525e615dec4d
> Cr-Commit-Position: refs/heads/master@{#336610}
TBR=rsesek@chromium.org,kerrnel@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=242704
Review URL: https://codereview.chromium.org/1213113006
Cr-Commit-Position: refs/heads/master@{#336757}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Audited the sandbox files to enusre all rules are necessary and removed
any incorrect comments.
The OS X Sandbox code is refactored to get rid of all string processing
of the raw scheme code in the sandbox profiles. By using the ability to
pass parameters into sandbox profiles, the variable substitution logic
is performed by the scheme interpreter itself.
BUG=242704
Review URL: https://codereview.chromium.org/1186233004
Cr-Commit-Position: refs/heads/master@{#336610}
|
|
|
|
|
|
|
|
|
|
| |
I'll take over ownership of the Mac sandbox files.
R=avi@chromium.org, jeremy@chromium.org, thakis@chromium.org
Review URL: https://codereview.chromium.org/1200263007.
Cr-Commit-Position: refs/heads/master@{#336153}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also refactor GN args and macros about "enable_mojo_media" to support running
MojoMediaApplication in the browser process or the utility process.
BUG=479935
TEST=Media plays with MojoMediaApplication running in the utility process.
Review URL: https://codereview.chromium.org/1201363002
Cr-Commit-Position: refs/heads/master@{#336104}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch was mostly autogenerated with
https://codereview.chromium.org/1010073002/.
BUG=465354
TBR=nick@chromium.org
Committed: https://crrev.com/422456f9d53f0bf936a64f21a1463fd0abd3df84
Cr-Commit-Position: refs/heads/master@{#333081}
Review URL: https://codereview.chromium.org/1159623009
Cr-Commit-Position: refs/heads/master@{#333112}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 422456f9d53f0bf936a64f21a1463fd0abd3df84.
TBR=zea@chromium.org
BUG=465354
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/1170623003
Cr-Commit-Position: refs/heads/master@{#333086}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch was mostly autogenerated with
https://codereview.chromium.org/1010073002/.
BUG=465354
TBR=nick@chromium.org
Review URL: https://codereview.chromium.org/1159623009
Cr-Commit-Position: refs/heads/master@{#333081}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 5d18b2493af36d77d3c491854e3d56f46f5f747c.
Reason: Broke iOS build: http://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/27843
NOTRY=true
NOTREECHECKS=true
BUG=465354
TBR=sievers@chromium.org,nick@chromium.org
Review URL: https://codereview.chromium.org/1153503009
Cr-Commit-Position: refs/heads/master@{#333047}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch was mostly autogenerated with
https://codereview.chromium.org/1010073002/.
BUG=465354
TBR=nick@chromium.org
Review URL: https://codereview.chromium.org/1159623009
Cr-Commit-Position: refs/heads/master@{#333043}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Per in-person discussions, as long as we have a fork of mojo shell we'll fork the interfaces.
I'll do third_party/mojo/src/mojo/public/java/application in a followup.
TBR=tsepez
Review URL: https://codereview.chromium.org/1139123006
Cr-Commit-Position: refs/heads/master@{#329965}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The globally-set TaskRunner doesn't work because some child
thread can run inside the browser process even when other
child threads run in their own processes.
For example, there is a configuration where GpuChildThread
being in-process while RenderThreadImpl isn't.
It is troublesome once non-renderer child processes adopt
ChannelMojo.
This CL eliminates the global in-process IO task runner
and instead, passes an IO runnner for each thread
if it is configured as an in-process mode.
Note that InProcessChildThreadParams object is introuced to
clarify that the parameter is for in-process mode.
This is a spin-off from https://codereview.chromium.org/960693003/.
R=jamesr@chromium.org, rockot@chromium.org, jam@chromium.org
BUG=377980
Review URL: https://codereview.chromium.org/987693005
Cr-Commit-Position: refs/heads/master@{#321047}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
| |
A single utility process is used for all proxy resolvers. If the process is
killed, a new one will be created.
BUG=11746
Review URL: https://codereview.chromium.org/940813003
Cr-Commit-Position: refs/heads/master@{#320430}
|