| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Define a global assert_no_deps on iOS (in ios/build/config.gni) and
use it in targets that have been fixed to work on iOS to prevent
further regression.
BUG=297668
Review URL: https://codereview.chromium.org/1812823002
Cr-Commit-Position: refs/heads/master@{#382286}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Chrome on iOS is aligning with the rest of Chrome and no longer uses
"ios_extra_version_path". Remove it from version_info components and
remove the "gclient" hook that generated ios/build/util/CANARY_VERSION.
BUG=591419
Review URL: https://codereview.chromium.org/1784783002
Cr-Commit-Position: refs/heads/master@{#380394}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I forgot that the GN arg is goma_dir, not gomadir, and
needed to be quoted as a string. Also, this CL enables
goma for the ios_rel_device_gn bot as well.
TBR=smut@chromium.org
BUG=517217
Review URL: https://codereview.chromium.org/1717873003 .
Cr-Commit-Position: refs/heads/master@{#376620}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Okay, "additional_compile_targets" only has an effect
if we're using ninja and either on tryservers or are using
goma. These bots aren't tryservers, but should be using goma, and
aren't. Fix that.
TBR=smut@chromium.org
BUG=517216
Review URL: https://codereview.chromium.org/1713123003 .
Cr-Commit-Position: refs/heads/master@{#376614}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For some reason, the GN iOS waterfall bots don't seem to be
defaulting to a target of "All" like the trybots are.
Hopefully the src-side override in this CL will fix things.
TBR=smut@chromium.org
BUG=517216
Review URL: https://codereview.chromium.org/1717923002 .
Cr-Commit-Position: refs/heads/master@{#376613}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Port code from tools/gyp/pylib/gyp/mac_tool.py to silence libtool
warning to ios/build/packaging/link_dependencies.py.
Add dependency on ios/build/packaging/link_dependencies.py for
target that invoke it so that they are rebuilt when the script
is updated.
BUG=None
Review URL: https://codereview.chromium.org/1670613002
Cr-Commit-Position: refs/heads/master@{#373784}
|
|
|
|
|
|
|
|
| |
BUG=570783
Review URL: https://codereview.chromium.org/1536643003
Cr-Commit-Position: refs/heads/master@{#365952}
|
|
|
|
|
|
|
|
|
| |
BUG=565519
TBR=baxley,dpranke
Review URL: https://codereview.chromium.org/1528143004
Cr-Commit-Position: refs/heads/master@{#365704}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#1 id:1 of https://codereview.chromium.org/1518393002/ )
Reason for revert:
Re-landing the original change since the build-side issue has been fixed.
Original issue's description:
> Revert of Enable goma on iOS CQ bots and matching waterfall bots. (patchset #7 id:180001 of https://codereview.chromium.org/1515933003/ )
>
> Reason for revert:
> Uncaught exception on iOS_Device ninja builder.
>
> https://build.chromium.org/p/chromium.mac/builders/iOS_Device_%28ninja%29/builds/32818
>
> Original issue's description:
> > Enable goma on iOS CQ bots and matching waterfall bots.
> >
> > By flipping the existing bots to use MB to invoke GYP instead
> > of invoking GYP via `gclient runhooks`, we can enable goma
> > solely by src-side changes.
> >
> > R=smut@google.com, phajdan.jr@chromium.org
> > BUG=564374
> >
> > Committed: https://crrev.com/63ca22d6bb7feb029a766f49fa8ee9bcbcfc7d76
> > Cr-Commit-Position: refs/heads/master@{#364866}
>
> TBR=phajdan.jr@chromium.org,smut@google.com,dpranke@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564374
>
> Committed: https://crrev.com/8fb34db92b286b24e39fba2943fe2679a1b5cdce
> Cr-Commit-Position: refs/heads/master@{#364974}
TBR=phajdan.jr@chromium.org,smut@google.com,perkj@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=564374
Review URL: https://codereview.chromium.org/1520253004
Cr-Commit-Position: refs/heads/master@{#365109}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#7 id:180001 of https://codereview.chromium.org/1515933003/ )
Reason for revert:
Uncaught exception on iOS_Device ninja builder.
https://build.chromium.org/p/chromium.mac/builders/iOS_Device_%28ninja%29/builds/32818
Original issue's description:
> Enable goma on iOS CQ bots and matching waterfall bots.
>
> By flipping the existing bots to use MB to invoke GYP instead
> of invoking GYP via `gclient runhooks`, we can enable goma
> solely by src-side changes.
>
> R=smut@google.com, phajdan.jr@chromium.org
> BUG=564374
>
> Committed: https://crrev.com/63ca22d6bb7feb029a766f49fa8ee9bcbcfc7d76
> Cr-Commit-Position: refs/heads/master@{#364866}
TBR=phajdan.jr@chromium.org,smut@google.com,dpranke@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=564374
Review URL: https://codereview.chromium.org/1518393002
Cr-Commit-Position: refs/heads/master@{#364974}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By flipping the existing bots to use MB to invoke GYP instead
of invoking GYP via `gclient runhooks`, we can enable goma
solely by src-side changes.
R=smut@google.com, phajdan.jr@chromium.org
BUG=564374
Review URL: https://codereview.chromium.org/1515933003
Cr-Commit-Position: refs/heads/master@{#364866}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With these changes, the ios trybots should start working.
Part of this change makes MB aware of the bot config files
in //ios/build/bots, so that we don't have to configure
the iOS bots in two different places.
See the updated MB docs for details.
R=smut@google.com, sdefresne@chromium.org, stuartmorgan@chromium.org
BUG=517216
Review URL: https://codereview.chromium.org/1411183010
Cr-Commit-Position: refs/heads/master@{#360688}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since these tests require the bring-up of the HttpServer, added
a new target ios_web_inttests with equivalent test fixtures.
Moved http_server to this new suite and enabled the unittest
since
app transport security was disabled in
https://codereview.chromium.org/1306733002
Added tests for Cookies, LocalStorage.
Also factored out the StringResponseProvider from
http_server_inttest
BUG=520583, 516600
Review URL: https://codereview.chromium.org/1446523002
Cr-Commit-Position: refs/heads/master@{#359780}
|
|
|
|
|
|
|
|
|
|
| |
As per https://code.google.com/p/chromium/issues/detail?id=552099#c20
BUG=552099
Review URL: https://codereview.chromium.org/1432823002
Cr-Commit-Position: refs/heads/master@{#358437}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Chrome on iOS no longer depends on the content startup and instead uses
ios/web startup (http://crrev.com/1406983008). Cleanup files thare are
no longer necessary from //content targets.
Stop building content_unittests on iOS (as it depends on BrowserThread)
now that none of the used functionality are testable on iOS and remove
it from the list of tests to run on the bots.
BUG=547725
Review URL: https://codereview.chromium.org/1427983002
Cr-Commit-Position: refs/heads/master@{#357635}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Xcode 7.0 has been publicly released and iOS downstream requires it to build
so use the iOS 9.0 SDK with Xcode 7.0 for the bots upstream.
Run the tests on all the currently supported iOS version 7.1, 8.3 and 9.0.
BUG=516606
Review URL: https://codereview.chromium.org/1369243002
Cr-Commit-Position: refs/heads/master@{#351178}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On iOS the patch element of a Chrome version comes from a different file
generated at "gclient runhooks" time. Upstream the code responsible for
the generation of that file.
Add the hook to DEPS and mark the file as ignored in global .gitignore.
BUG=512750
Review URL: https://codereview.chromium.org/1247983004
Cr-Commit-Position: refs/heads/master@{#340060}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
//ios unit tests will soon required access to packed resources (both theme
and strings) so add steps required to repack data.
Those steps only pack what is required by //ios unit tests and thus pack to
repack_ios to avoid collision with downstream packing steps. Eventually the
upstream and downstream repack steps will be merged.
This forks some steps from //chrome since //ios cannot depends on //chrome
and those steps are too specific to Chrome to be moved higher in //build.
BUG=429756,475514
Review URL: https://codereview.chromium.org/1164983004
Cr-Commit-Position: refs/heads/master@{#333229}
|
|
|
|
|
|
|
|
|
|
|
| |
Run the unit tests on all version of the OS supported. Currently this include
iOS 7 and iOS 8 (i.e. SDK 7.1 and SDK 8.3 for simulators in Xcode).
BUG=493729
Review URL: https://codereview.chromium.org/1143133005
Cr-Commit-Position: refs/heads/master@{#332188}
|
|
|
|
|
|
|
|
|
|
|
| |
This code is copied directly from the downstream iOS repo, except for the
changes to ios/ios.gyp.
BUG=483209
Review URL: https://codereview.chromium.org/1125293004
Cr-Commit-Position: refs/heads/master@{#330952}
|
|
|
|
|
|
|
|
|
|
| |
This matches downstream, where we build a universal binary, and then run tests on the 64-bit iPhone 5s simulator and the 32-bit iPad Retina simulator.
BUG=490189
Review URL: https://codereview.chromium.org/1150753002
Cr-Commit-Position: refs/heads/master@{#330915}
|
|
|
|
|
|
|
|
| |
BUG=452726
Review URL: https://codereview.chromium.org/1146243003
Cr-Commit-Position: refs/heads/master@{#330821}
|
|
|
|
|
|
|
|
| |
BUG=475219
Review URL: https://codereview.chromium.org/1143933002
Cr-Commit-Position: refs/heads/master@{#330781}
|
|
|
|
|
|
|
|
| |
BUG=452726
Review URL: https://codereview.chromium.org/1145723003
Cr-Commit-Position: refs/heads/master@{#330767}
|
|
|
|
|
|
|
|
| |
BUG=485435
Review URL: https://codereview.chromium.org/1123283009
Cr-Commit-Position: refs/heads/master@{#329748}
|
|
|
|
|
|
|
|
| |
BUG=441392
Review URL: https://codereview.chromium.org/1089903003
Cr-Commit-Position: refs/heads/master@{#329237}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add all the unit tests for code that is compiled and used on iOS so
that they are run on iOS bots (waterfall and CQ).
BUG=475512
Committed: https://crrev.com/e4c308a2736e5c80e35b5f9ebaf6f2d31dcd0523
Cr-Commit-Position: refs/heads/master@{#324624}
Review URL: https://codereview.chromium.org/1077733002
Cr-Commit-Position: refs/heads/master@{#324653}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1077733002/)
Reason for revert:
I believe this CL is causing the IOS Simulator (DBG) bot to fail on gcm_unit_tests:
gcm_unit_tests (iPhone 5s iOS 8.1) gcm_unit_tests (iPhone 5s iOS 8.1) AppNotFoundError: App does not exist: /Volumes/data/b/build/slave/iOS_Simulator__dbg_/build/src/xcodebuild/Debug-iphonesimulator/gcm_unit_tests.app.
Example builder:
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/25320
Original issue's description:
> Update the list of tests to run on iOS bots
>
> Add all the unit tests for code that is compiled and used on iOS so
> that they are run on iOS bots (waterfall and CQ).
>
> BUG=475512
>
> Committed: https://crrev.com/e4c308a2736e5c80e35b5f9ebaf6f2d31dcd0523
> Cr-Commit-Position: refs/heads/master@{#324624}
TBR=smut@google.com,droger@chromium.org,sdefresne@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=475512
Review URL: https://codereview.chromium.org/1074163003
Cr-Commit-Position: refs/heads/master@{#324635}
|
|
|
|
|
|
|
|
|
|
|
| |
Add all the unit tests for code that is compiled and used on iOS so
that they are run on iOS bots (waterfall and CQ).
BUG=475512
Review URL: https://codereview.chromium.org/1077733002
Cr-Commit-Position: refs/heads/master@{#324624}
|
|
|
|
|
|
|
|
| |
BUG=471413
Review URL: https://codereview.chromium.org/1046033002
Cr-Commit-Position: refs/heads/master@{#323068}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/995903002
Cr-Commit-Position: refs/heads/master@{#320064}
|
|
|
|
|
|
|
|
| |
BUG=440857
Review URL: https://codereview.chromium.org/963683007
Cr-Commit-Position: refs/heads/master@{#319920}
|
|
|
|
|
|
|
|
|
|
|
| |
Per the explanation in the bug, ui/ios should not have been created in
the first place; this code belongs in the ui/gfx subcomponent.
BUG=446643
Review URL: https://codereview.chromium.org/983413004
Cr-Commit-Position: refs/heads/master@{#319720}
|
|
|
|
|
|
|
|
| |
BUG=440857
Review URL: https://codereview.chromium.org/963683006
Cr-Commit-Position: refs/heads/master@{#319123}
|
|
|
|
|
|
|
|
|
|
|
| |
ios_unittests name does not reflect the real purpose of the target.
Enable the tests on the try bots.
BUG=462150
Review URL: https://codereview.chromium.org/951383002
Cr-Commit-Position: refs/heads/master@{#318428}
|
|
|
|
|
|
|
|
|
|
| |
These correspond to the bots turned down in https://codereview.chromium.org/957153002.
BUG=457659
Review URL: https://codereview.chromium.org/960063002
Cr-Commit-Position: refs/heads/master@{#318279}
|
|
|
|
|
|
|
|
| |
BUG=460086
Review URL: https://codereview.chromium.org/959543007
Cr-Commit-Position: refs/heads/master@{#318259}
|
|
|
|
|
|
|
|
| |
BUG=440789
Review URL: https://codereview.chromium.org/950903006
Cr-Commit-Position: refs/heads/master@{#318258}
|
|
|
|
|
|
|
|
|
|
| |
They only have 5.1 installed
BUG=460930
Review URL: https://codereview.chromium.org/947273002
Cr-Commit-Position: refs/heads/master@{#317636}
|
|
|
|
|
|
|
|
| |
BUG=452193
Review URL: https://codereview.chromium.org/941033002
Cr-Commit-Position: refs/heads/master@{#317351}
|
|
|
|
|
|
|
|
|
|
| |
Note that target_subarch=both is already set on the tryserver and Mac main waterfalls.
BUG=458398
Review URL: https://codereview.chromium.org/926443002
Cr-Commit-Position: refs/heads/master@{#316910}
|
|
|
|
|
|
|
|
|
|
| |
Because there is no signing identity on upstream bots. Omitting this variable is causing at least device bots to assume it should be 1, might as well be explicit on all bots.
BUG=452193
Review URL: https://codereview.chromium.org/918213004
Cr-Commit-Position: refs/heads/master@{#316215}
|
|
|
|
|
|
|
|
|
|
| |
BUG=452193
R=stip
TBR=phajdan.jr
Review URL: https://codereview.chromium.org/916643002
Cr-Commit-Position: refs/heads/master@{#315866}
|
|
This CL upstreams the iOS translate client code, and the iOS resources
infrastructure.
TBR=asvitkine
Review URL: https://codereview.chromium.org/809333003
Cr-Commit-Position: refs/heads/master@{#310513}
|