| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/874663004
Cr-Commit-Position: refs/heads/master@{#318378}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I don't think any other config will work at the moment.
Note that this is also totally untested, but should at least compile :).
R=brettw@chromium.org
BUG=432959
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
Review URL: https://codereview.chromium.org/954383002
Cr-Commit-Position: refs/heads/master@{#318329}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The variants of postTask and postDelayedTask without location
information aren't needed anymore.
BUG=450977
TBR=jamesr@chromium.org
Review URL: https://codereview.chromium.org/911103002
Cr-Commit-Position: refs/heads/master@{#318289}
|
|
|
|
|
|
|
|
|
|
| |
This method has been removed from WebMimeRegistry.
BUG=385874
Review URL: https://codereview.chromium.org/937323003
Cr-Commit-Position: refs/heads/master@{#318273}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should get most of NaCl and the PPAPI stuff needed
for NaCl building and linking. There is more work to be
done to get some of the test binaries working (and
probably fill out parts of the NaCl SDK) and possibly
some pnacl support work remaining as well.
NaCl is still disabled by default (set enable_nacl=true
to change). Enabling nacl is still mostly untested and
likely doesn't work at all :).
R=ncbray@chromium.org, brettw@chromium.org
BUG=432959
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
Review URL: https://codereview.chromium.org/877553008
Cr-Commit-Position: refs/heads/master@{#318180}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/959873003
Cr-Commit-Position: refs/heads/master@{#318125}
|
|
|
|
|
|
|
|
|
| |
//build/module_args/mojo.gni is the canonical place for setting information
that might differ between repos, such as the location of the Mojo SDK.
Review URL: https://codereview.chromium.org/956573002
Cr-Commit-Position: refs/heads/master@{#318015}
|
|
|
|
|
|
|
|
|
|
|
| |
The new Clang plugin is more picky.
BUG=444089, 459607
TBR=hans,aa
Review URL: https://codereview.chromium.org/954113003
Cr-Commit-Position: refs/heads/master@{#317971}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix build error with new Clang/plugin:
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_dbg/builds/164/steps/gn/logs/stdio
BUG=444089, 459607
NOTRY=true
Review URL: https://codereview.chromium.org/955783002
Cr-Commit-Position: refs/heads/master@{#317898}
|
|
|
|
|
|
|
|
|
|
|
| |
Per dalecurtis@ suggestion on
https://codereview.chromium.org/883353012/
BUG=457959
Review URL: https://codereview.chromium.org/941633004
Cr-Commit-Position: refs/heads/master@{#317876}
|
|
|
|
|
|
|
|
|
|
|
|
| |
These files declare/define mojo TypeConverters, so the file names
should reflect that.
R=jamesr@chromium.org,msw@chromium.org
BUG=457280
Review URL: https://codereview.chromium.org/950573002
Cr-Commit-Position: refs/heads/master@{#317619}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The latest GN binaries rename cpu_arch to current_cpu and/or target_cpu
as appropriate, build_cpu_arch to host_cpu, and os to current_os and target_os as appropriate.
R=brettw@chromium.org
TBR=ddorwin@chromium.org
EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
BUG=344767
Review URL: https://codereview.chromium.org/913373002
Cr-Commit-Position: refs/heads/master@{#317223}
|
|
|
|
|
|
|
|
|
|
|
|
| |
path.
Also makes jpeg color space enum naming more correct.
BUG=333619
Review URL: https://codereview.chromium.org/819713005
Cr-Commit-Position: refs/heads/master@{#317147}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Android binaries were relying on -Wl,--exclude-libs=ALL to prevent
unnecessary symbols from being exported in the final shared libraries
built in Chromium, however several changes were submitted that actually
disabled this linker flag in virtually all binaries, and thus it was not
really being used and many symbols were exported by mistake.
Remove --exclude-libs=ALL entirely from the Android build, as well as
all the places which were previously having to unset it, and instead
exclude the specific libraries which incorporate visible symbols that we
don't want: the NDK libraries which we statically link into the binary,
and a number of components of chromium which contain assembly code which
has not yet been fixed to set symbol visibilities correctly.
This reduces the number of exported symbols in the shared libraries to
only the ones we intended to export, which saves ~120KB binary size.
The GN build does not build the same set of things as static libraries,
so the list is slightly different there and several more symbols are
left in the final binaries which shouldn't be there, but are not pulled
in via a static library and so no --exclude-libs flag will ever fix it;
the symbol visibilities will need to be fixed at source.
BUG=448386
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/806533003
Cr-Commit-Position: refs/heads/master@{#317027}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
RendererScheduler.
In order to implement long idle times, the RendererScheduler needs to
know when the compositor has stoped drawing frames. To enable this, add
a BeginMainFrameNotExpectedSoon signal which is sent when the cc
scheduler stops requesting BeginFrames from the Browser.
Design Doc: https://docs.google.com/a/chromium.org/document/d/1yBlUdYW8VTIfB-DqhvQqUeP0kf-Ap1W4cao2yQq58Do/edit
BUG=455713
TBR=danakj@chromium.org, torne@chromium.org, aa@chromium.org
Review URL: https://codereview.chromium.org/904633004
Cr-Commit-Position: refs/heads/master@{#316804}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change reenables the Mojo test in Chrome and uses the Mojo IRT interface
inside Mojo calls.
It touches a file in the Mojo repo, which corresponds to the following
change there:
https://codereview.chromium.org/918243002/
That change is necessary for the mojo irt interface to be queried from C++
code.
BUG=454466
Review URL: https://codereview.chromium.org/923593002
Cr-Commit-Position: refs/heads/master@{#316694}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes geometry_type_converters.h independent of skia. A target
can use geometry_type_converters.h without picking up a dependency on
skia inadvertently.
Create transform_converters.h for the converters to and from
gfx::Transform, which is implemented using skia's SkMatrix44.
R=jamesr@chromium.org,msw@chromium.org
BUG=457280
Review URL: https://codereview.chromium.org/911793002
Cr-Commit-Position: refs/heads/master@{#316252}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Added LibraryLoader.get() to instance LibraryLoader signleton.
- Added LibraryProcessType to indicate the process it is loaded in.
- Changed BrowserStartupController to accept libraryProcessType parameter.
- Added ChromeBrowserStartupController for helping to create BrowserStartupController for Chrome.
TBR=mnaganov, ben, mark
BUG=454870
Review URL: https://codereview.chromium.org/897683003
Cr-Commit-Position: refs/heads/master@{#316170}
|
|
|
|
|
|
|
|
|
|
| |
If SetupLock() is called on the context provider, it can be used from multiple threads if the lock returned from GetLock() is held around every call into the context.
BUG=454500
Review URL: https://codereview.chromium.org/918793003
Cr-Commit-Position: refs/heads/master@{#316108}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch pulls in two fixes to mojo's WebThread implementation from
Chromium:
1. Race condition fix for deleting posted tasks.
2. Add postTask/postDelayedTask variants that take the posting location
as a parameter.
BUG=450977
Review URL: https://codereview.chromium.org/911073002
Cr-Commit-Position: refs/heads/master@{#315962}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
this is all broken and has been for a while. Since then, 2012 to 2013
other, sytem-specific systems have been used.
Note that there is a v8 test that is using the stats table. Hoping to sort this now.
BUG=149742
Review URL: https://codereview.chromium.org/899753002
Cr-Commit-Position: refs/heads/master@{#315890}
|
|
|
|
|
|
|
|
|
|
| |
Recent changes in the Mojo repo have added an IRT interface for Mojo support; this change rolls that (and other changes) into chrome to enable IRT support there as well.
BUG=454466
Review URL: https://codereview.chromium.org/874383003
Cr-Commit-Position: refs/heads/master@{#315786}
|
|
|
|
|
|
|
|
|
|
|
| |
This file is no longer used outside of Chromium.
BUG=450356
R=blundell@chromium.org
Review URL: https://codereview.chromium.org/908853002
Cr-Commit-Position: refs/heads/master@{#315516}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The following NaCl-related defines are now provided by
native_client/src/include/build_config.h instead of the build system:
NACL_WINDOWS
NACL_LINUX
NACL_OSX
NACL_ANDROID
NACL_BUILD_ARCH
NACL_BUILD_SUBARCH
This change removes these definitions from gyp(i) files, as they no longer need
to be provided by the build system.
BUG=440012
Review URL: https://codereview.chromium.org/912503003
Cr-Commit-Position: refs/heads/master@{#315429}
|
|
|
|
|
|
|
|
|
| |
BUG=450356
R=blundell@chromium.org, jamesr@chromium.org
Review URL: https://codereview.chromium.org/904443003
Cr-Commit-Position: refs/heads/master@{#315278}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
[Taken over from blundell's https://codereview.chromium.org/901843003/,
PS3.]
Adapt to de-clienting of Surface.
Have nacl_listener pass a SimplePlatformSupport instance to
mojo::embedder::Init to satisfy the DCHECK introduced in
https://codereview.chromium.org/898623002/diff/40001/mojo/edk/embedder/embedder.cc.
[Additional fix backported from
https://codereview.chromium.org/898423002/.]
Fix mojo::embedder::CreateChannel(): the evaluation order between the
channel_info->channel_id and channel_info.release() is undefined.
[Additional "fix" backported from
https://codereview.chromium.org/903933006/.]
Add an "unnecessary" .Pass() to try to placate MSVS.
[Additional fix:]
surfaces_service.mojom.h -> surfaces.mojom.h
R=jamesr@chromium.org
NOPRESUBMIT=true
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win8_chromium_gn_dbg,win8_chromium_gn_rel
Review URL: https://codereview.chromium.org/904103003
Cr-Commit-Position: refs/heads/master@{#315161}
|
|
|
|
|
|
|
|
|
|
|
|
| |
We have users of html_viewer that do not use the mojo provided services, including the surface service. This change: https://codereview.chromium.org/869343004/ broke them
I think it's right to check this here. Another option would be to do it in layer_tree_host.cc
BUG=
Review URL: https://codereview.chromium.org/902293002
Cr-Commit-Position: refs/heads/master@{#315157}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit c2e11810044fe5ff1c1b8588103671925a5b74fe.
TBR=viettrungluu@chromium.org
NOTRY=true
NOTREECHECKS=true
NOPRESUBMIT=true
BUG=
Review URL: https://codereview.chromium.org/877993004
Cr-Commit-Position: refs/heads/master@{#315134}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
[Taken over from blundell's https://codereview.chromium.org/901843003/,
PS3.]
Adapt to de-clienting of Surface.
Have nacl_listener pass a SimplePlatformSupport instance to
mojo::embedder::Init to satisfy the DCHECK introduced in
https://codereview.chromium.org/898623002/diff/40001/mojo/edk/embedder/embedder.cc.
[Additional fix backported from
https://codereview.chromium.org/898423002/.]
Fix mojo::embedder::CreateChannel(): the evaluation order between the
channel_info->channel_id and channel_info.release() is undefined.
TBR=blundell@chromium.org
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/904103003
Cr-Commit-Position: refs/heads/master@{#315128}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves generated code for Mojo support in NaCl from mojo/monacl to
mojo/nacl. This makes the target directory more consistent across GYP and GN
builds.
An upcoming change for IRT support will generate a header, which will need a
consistent path (relative to include_dirs) across build systems.
An earlier version of this change failed to update
mojo/mojo_nacl_untrusted.gyp, causing clean builds to fail. I've confirmed
that this change builds correctly with a clean build.
BUG=454466
TEST=clobber, then build
Review URL: https://codereview.chromium.org/881933004
Cr-Commit-Position: refs/heads/master@{#314811}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now that fractional scroll offset can pass through cc main
thread and Blink, we can remove the scroll_delta flooring hack
and send the full fractional scroll_delta from impl to main.
Note that this will break the old pinch viewport code path (
which is to be removed soon) since it still only expects integer
scroll offset.
BUG=414283
Review URL: https://codereview.chromium.org/870363005
Cr-Commit-Position: refs/heads/master@{#314626}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This file was deprecated by corresponding apptests and was deleted in
https://codereview.chromium.org/722943004 .
It was accidentally ressurected when moving the network service
source-of-truth to Chromium - we carried over modifications and new
files, but didn't carry over deletions of whole files.
BUG=450356
Review URL: https://codereview.chromium.org/894093003
Cr-Commit-Position: refs/heads/master@{#314601}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/896243003/)
Reason for revert:
Caused build failures on mac bots. Not sure why it failed there but not on the CQ.
Original issue's description:
> Mojo+NaCl: Move generated code to mojo/nacl.
>
> This change moves generated code for Mojo support in NaCl from mojo/monacl to
> mojo/nacl. This makes the target directory more consistent across GYP and GN
> builds.
>
> An upcoming change for IRT support will generate a header, which will need a
> consistent path (relative to include_dirs) across build systems.
>
> BUG=454466
>
> Committed: https://crrev.com/99cdffdb116cf608630600bbbf66a37fcf16695e
> Cr-Commit-Position: refs/heads/master@{#314591}
TBR=ncbray@chromium.org,jamesr@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=454466
Review URL: https://codereview.chromium.org/900983002
Cr-Commit-Position: refs/heads/master@{#314597}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change moves generated code for Mojo support in NaCl from mojo/monacl to
mojo/nacl. This makes the target directory more consistent across GYP and GN
builds.
An upcoming change for IRT support will generate a header, which will need a
consistent path (relative to include_dirs) across build systems.
BUG=454466
Review URL: https://codereview.chromium.org/896243003
Cr-Commit-Position: refs/heads/master@{#314591}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes plumbing for InstrumentWillBeginFrame(), InstrumentDidBeginFrame(),
InstrumentDidCancelFrame() and InstrumentWillComposite() between CC and
WebDevToolsAgent and replaces it with trace events emitted directly from CC.
Note all related methods of LayerTreeHostClient still remain, as they're used
in tests.
TBR=jamesr
(for mojo/)
BUG=
Review URL: https://codereview.chromium.org/892203002
Cr-Commit-Position: refs/heads/master@{#314571}
|
|
|
|
|
|
|
|
|
|
| |
This CL generalizes upload_network_service.py to support uploading other
services (currently, html_viewer). html_viewer.mojo is not zipped before being
uploaded so that it can be accessed dynamically by mojo_shell.
Review URL: https://codereview.chromium.org/879233006
Cr-Commit-Position: refs/heads/master@{#314533}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Video textures can be larger than the visible size of the video.
Because they have bilinear filtering turned on for them, if the video is
scaled, then the this means that the part of the texture outside of the
video itself can be sampled, causing odd visible artifacts like
hairlines on the side of the image.
To fix this, pass in a clamp value of half a pixel (because of bilinear
filtering) inside of the visible rect and clamp all texture coordinates
to this maximum value in the fragment shader.
BUG=429640
Review URL: https://codereview.chromium.org/881963002
Cr-Commit-Position: refs/heads/master@{#314200}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch makes the Chromium apptest runner resilient to missing apps
(e.g. tracing.mojo) polluting the output of mojo_shell that it tries to
parse.
BUG=450356
TBR=jamesr@chromium.org
Review URL: https://codereview.chromium.org/892933002
Cr-Commit-Position: refs/heads/master@{#314134}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also pass MediaPermission to ProxyDecryptor, in preparation for triggering
user prompt from ProxyDecryptor for prefixed EME implementation.
TBR=jam@chromium.org,aa@chromium.org
BUG=446263
Review URL: https://codereview.chromium.org/887873003
Cr-Commit-Position: refs/heads/master@{#314082}
|
|
|
|
|
|
|
|
| |
BUG=451321
Review URL: https://codereview.chromium.org/880613005
Cr-Commit-Position: refs/heads/master@{#313984}
|
|
|
|
|
|
|
|
|
|
|
| |
This CL moves services code that is brought in from the Mojo repo and currently
living in //mojo/services to instead live in //third_party/mojo_services/src.
It fixes up include paths, buildfile references, and DEPS entries to account
for the new state of the world.
Review URL: https://codereview.chromium.org/861683003
Cr-Commit-Position: refs/heads/master@{#313876}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
mojo::AsyncWaiter was added to mojo/public/cpp/environment in
https://github.com/domokit/mojo/commit/2b6c27fbbe9b704a77e977615b945f364bf520ca
and was rolled into chromium. However, the chromium environment
implementation is on mojo/environment, which is maintained independently
of the version in the mojo repo. This CL copies the mojo repo's
mojo/environment into the chromium repo so that mojo::AsyncWaiter is
usable there.
Review URL: https://codereview.chromium.org/883583005
Cr-Commit-Position: refs/heads/master@{#313859}
|
|
|
|
|
|
|
|
|
|
| |
I'm planning to remove this interface in favor of connecting directly
to mojo.Surface in the mojo repo, this updates html_viewer so it will
still work after the removal.
Review URL: https://codereview.chromium.org/869343004
Cr-Commit-Position: refs/heads/master@{#313858}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch copies apptest_runner from Mojo, stripping it down to parts that are
needed to run network service apptests on Linux. This allows us to run them in
a Chromium checkout.
BUG=450356
Review URL: https://codereview.chromium.org/874273002
Cr-Commit-Position: refs/heads/master@{#313834}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update HTMLViewer for de-clienting of Shell.
Update ServiceRegistryTest.java for de-clienting of MathCalculator.
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/883843002
Cr-Commit-Position: refs/heads/master@{#313729}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Launching Mojo apps on Android is slow because we keep the network cache in
memory, which means we lose it every time we launch MojoShell. After this CL,
we use a disk cache on Android. Note: We still use a memory cache on desktop
because we want to be able to run many instances of mojo_shell in parallel on
desktop for testing.
R=jamesr@chromium.org
BUG=452226
Review URL: https://codereview.chromium.org/883073003
Cr-Commit-Position: refs/heads/master@{#313642}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Remove old versions of createChildFrame.
* Add browser tests.
Related CLs:
Chromium part 1: https://codereview.chromium.org/837283003/
Blink part 1: https://codereview.chromium.org/793493003/
Blink part 2: https://codereview.chromium.org/838903002/
BUG=426512
Review URL: https://codereview.chromium.org/797813006
Cr-Commit-Position: refs/heads/master@{#313384}
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TBR=aa@chromium.org for html_viewer
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/866263004
Cr-Commit-Position: refs/heads/master@{#313314}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
MediaPermission is the interface for media code to check/request media related
permissions. MediaPermissionDispatcher is a MediaPermission implementation in
content/.
Plan of follow-up CL:
- Use MediaPermission in ProxyDecryptor and remove current protected media
identifier info bar code in the browser process (on Android and ChromeOS).
BUG=446263
TEST=Tested on Android with encrypted-media-requestmediakeysystemaccess.html
layout test by calling MediaPermission methods manually in
webencryptedmediaclient_impl.cc.
Review URL: https://codereview.chromium.org/871663003
Cr-Commit-Position: refs/heads/master@{#313303}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This carries over a fix by qsr@ for the mojo helper handle_watcher
landed in the mojo repo in https://codereview.chromium.org/781753004.
The fix allows the network service apptests not to crash when built
against the Chromium copy of mojo/common.
BUG=450356
Review URL: https://codereview.chromium.org/878513003
Cr-Commit-Position: refs/heads/master@{#313261}
|