| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1734243002
Cr-Commit-Position: refs/heads/master@{#377770}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For a time we needed to enable (parts of) the WebRTC GN build
on iOS, because there was an indirect dependency on WebRTC in
order for some components to be able to depend on xmllite.
Those components have since been switched to depend on libxml2,
and so there is no reason for Chromium to build WebRTC on iOS
now (though WebRTC is free to build on iOS on its own).
This CL cleans up the build configs accordingly.
R=brettw@chromium.org, kjellander@chromium.org, sdefresne@chromium.org
BUG=538497
CQ_INCLUDE_TRYBOTS=tryserver.chromium.mac:ios_rel_device_gn,ios_dbg_simulator_gn
Review URL: https://codereview.chromium.org/1718073002
Cr-Commit-Position: refs/heads/master@{#377070}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The standalone (non-Chromium) WebRTC build has some targets
that on the Mac use Objective-C features that require the
targets to be deployed to 10.7 or newer hosts (i.e., they
won't run on 10.6).
Because we share GN //build configurations between the WebRTC
and Chromium project, we need to give WebRTC to toggle the
minimum configuration needed; we do this by creating a new file called
build.gni in //build_overrides/build.gni file, and
making //build/config/mac/mac_sdk.gni import that. This introduces
a dependency from //build to //build_overrides, and means that other
projects that depend on //build will need to create a new file when
this change is rolled in..
R=brettw@chromium.org
BUG=webrtc:5453, 588513
Review URL: https://codereview.chromium.org/1686373002
Cr-Commit-Position: refs/heads/master@{#376856}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This gets 'gn_all' building on iOS so we can start adding tryservers
to the CQ and turn on the waterfall bots.
At the moment WebRTC doesn't quite work on iOS, so we don't build
the components that need to depend on it; we add an 'ios_use_webrtc'
arg so that we can hack around it locally to build the rest of iOS,
but that arg should be deleted and WebRTC fixed.
R=sdefresne@chromium.org, brettw@chromium.org
BUG=459705, webrtc:5213
CQ_EXTRA_TRYBOTS=tryserver.chromium.mac:ios_dbg_simulator_gn,ios_rel_device_gn
Review URL: https://codereview.chromium.org/1467803003
Cr-Commit-Position: refs/heads/master@{#365373}
|
|
|
|
|
|
|
|
|
|
|
| |
This matches GYP.
BUG=545641
R=dpranke@chromium.org
Review URL: https://codereview.chromium.org/1495673002
Cr-Commit-Position: refs/heads/master@{#362804}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This implements the ReadableStream and CountQueuingStrategy classes from the Streams Standard using V8 extras. They are marked experimental. The implementation follows the spec and reference implementation very closely.
The IDL implementation of ReadableStream is no longer exposed to the global scope; it was not usable in any way. But, to avoid any breakage in the interim, we introduce a non-experimental ReadableStreamTempStub.js V8 extra that replaces the IDL implementation with equivalent (non-)functionality.
Further work will be necessary to integrate this implementation with Fetch, replacing Fetch's use of ReadableByteStream, before we can completely remove the existing implementation. The current implementation landscape is explained in the new README.md file in this directory.
The tests are drawn from https://github.com/whatwg/streams/tree/master/reference-implementation/web-platform-tests.
BUG=503491
R=yhirano@chromium.org,jochen@chromium.org
Review URL: https://codereview.chromium.org/1404523005
Cr-Commit-Position: refs/heads/master@{#362201}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
data name
This patch changed all android GN target which use v8 snapshot files
as of it being prepared.
origin patch
https://codereview.chromium.org/1422793004/
This patch also fix package_resource.py issue that asset's
renaming didn't trigger the packaging.
origin patch
https://codereview.chromium.org/1417753004/
Switch Chrome Android using v8 arch specific external data name
For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
This patch also removed the existing v8 startup data in
destination directory by using copy_ex to remove directory
first.
BUG=540788, 547235
TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org
Review URL: https://codereview.chromium.org/1411283009
Cr-Commit-Position: refs/heads/master@{#357577}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
external data name (patchset #9 id:180001 of https://codereview.chromium.org/1422793004/ )
Reason for revert:
Looks like it caused Android GN net_unittests to crash: https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/31886/steps/net_unittests/logs/stdio; suspect it's crashing a ProxyResolverV8 test.
Original issue's description:
> Reland Reland Switch Chrome Android using v8 arch specific external data name
>
> This patch also fix package_resource.py issue that asset's
> renaming didn't trigger the packaging.
>
> origin patch
> https://codereview.chromium.org/1417753004/
>
> Switch Chrome Android using v8 arch specific external data name
>
> For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
> For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
>
> This patch also removed the existing v8 startup data in
> destination directory by using copy_ex to remove directory
> first.
>
> BUG=540788,547235
>
> TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org
>
> Committed: https://crrev.com/02d6496adfb377afc0708e53cd0c8118c0591f99
> Cr-Commit-Position: refs/heads/master@{#357480}
TBR=jbudorick@chromium.org,miguelg@chromium.org,michaelbai@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=540788,547235
Review URL: https://codereview.chromium.org/1432553003
Cr-Commit-Position: refs/heads/master@{#357523}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch also fix package_resource.py issue that asset's
renaming didn't trigger the packaging.
origin patch
https://codereview.chromium.org/1417753004/
Switch Chrome Android using v8 arch specific external data name
For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
This patch also removed the existing v8 startup data in
destination directory by using copy_ex to remove directory
first.
BUG=540788,547235
TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org
Review URL: https://codereview.chromium.org/1422793004
Cr-Commit-Position: refs/heads/master@{#357480}
|
|
|
|
|
|
|
|
|
| |
R=brettw@chromium.org, jam@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1416873013
Cr-Commit-Position: refs/heads/master@{#356707}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
name (patchset #3 id:40001 of https://codereview.chromium.org/1417753004/ )
Reason for revert:
broke gpu bots: https://build.chromium.org/p/chromium.gpu/builders/Android%20Debug%20%28Nexus%206%29/builds/3712
Original issue's description:
> Orignal patch
> https://codereview.chromium.org/1400953002/
>
> Switch Chrome Android using v8 arch specific external data name
>
> For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
> For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
>
> This patch also removed the existing v8 startup data in
> destination directory by using copy_ex to remove directory
> first.
>
> BUG=540788
> TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org
>
> Committed: https://crrev.com/ed31140dee44c06e6398e92e1afb9c047c9137e6
> Cr-Commit-Position: refs/heads/master@{#355843}
TBR=michaelbai@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=540788
Review URL: https://codereview.chromium.org/1421893002
Cr-Commit-Position: refs/heads/master@{#355877}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1400953002/
Switch Chrome Android using v8 arch specific external data name
For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
This patch also removed the existing v8 startup data in
destination directory by using copy_ex to remove directory
first.
BUG=540788
TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org
Review URL: https://codereview.chromium.org/1417753004
Cr-Commit-Position: refs/heads/master@{#355843}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #5 id:80001 of https://codereview.chromium.org/1400953002/ )
Reason for revert:
Looks to have broken various Android Mojo tests:
http://build.chromium.org/p/chromium.linux/buildstatus?builder=Android%20GN&number=31621
http://build.chromium.org/p/chromium.linux/buildstatus?builder=Android%20GN&number=31622
Original issue's description:
> Switch Chrome Android using v8 arch specific external data name
>
> For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
> For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
>
> This patch also removed the existing v8 startup data in
> destination directory by using copy_ex to remove directory
> first.
>
> BUG=540788
>
> Committed: https://crrev.com/979b03a19f63b1cbe02fa3df99707c3f93f7c9cf
> Cr-Commit-Position: refs/heads/master@{#355680}
TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org,jam@chromium.org,michaelbai@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=540788
Review URL: https://codereview.chromium.org/1419673008
Cr-Commit-Position: refs/heads/master@{#355697}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.
This patch also removed the existing v8 startup data in
destination directory by using copy_ex to remove directory
first.
BUG=540788
Review URL: https://codereview.chromium.org/1400953002
Cr-Commit-Position: refs/heads/master@{#355680}
|
|
|
|
|
|
|
|
| |
BUG=510485
Review URL: https://codereview.chromium.org/1414353002
Cr-Commit-Position: refs/heads/master@{#355291}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Provides values for the Chromium build of WebRTC.
Corresponding change in WebRTC:
https://codereview.webrtc.org/1403453003/
BUG=webrtc:5070,chromium:541791
TBR=brettw@chromium.org
Review URL: https://codereview.chromium.org/1406543003
Cr-Commit-Position: refs/heads/master@{#353969}
|
|
The original intent of the build/module_args directory was to
provide a place where different repos could customize settings
as needed depending on their dependencies and needs.
However, given that you can't really yet embed GN by just
deps-ing in //build/config and //build/toolchain, you end up
needing to deps-in all of build, and so there was no good way to
have different settings per-repo in //build/module_args.
This CL changes the approach such that we will have an additional
top-level directory called //build_overrides, which is therefore
separate from //build and can be properly customized.
It is unfortunate that we need to use two top-level directories
for GN, but we don't have a good alternative at this time. Once
we can remove the GYP build and further clean up the structure
and dependencies of //build we will hopefully be able to do better.
R=brettw@chromium.org
BUG=541791
Review URL: https://codereview.chromium.org/1397493004
Cr-Commit-Position: refs/heads/master@{#353690}
|