Upstream helper script to build Chromium on iOS and update instructions.

Upstream helper script to build Chromium on iOS and update instructions.

Upstream some scripts used to simplify the setup of a new Chromium on iOS
checkout and update the instruction to remove obsolete informations, and
add recommendation to use the script.

BUG=None

Committed: https://crrev.com/7f4c25f2b5b8c745b254197282630dbcd7bfa755
Cr-Commit-Position: refs/heads/master@{#419422}

[sdefresne, 2016-09-15 13:47:33]

sdefresne@chromium.org changed reviewers:
+ noyau@chromium.org

[sdefresne, 2016-09-15 13:47:34]

Please take a look.

[sdefresne, 2016-09-19 08:20:06]

Ping?

[noyau (Ping after 24h), 2016-09-19 09:03:30]

lgtm

https://codereview.chromium.org/2343853002/diff/1/docs/ios_build_instructions.md
File docs/ios_build_instructions.md (right):

https://codereview.chromium.org/2343853002/diff/1/docs/ios_build_instructions...
docs/ios_build_instructions.md:17: To checkout the source, use `fetch` command
from [depot\_tools] in a new
s/fetch/fetch ios/ ?

https://codereview.chromium.org/2343853002/diff/1/docs/ios_build_instructions...
docs/ios_build_instructions.md:66: the list of targets and files in the Xcode
solution may be stale.
Can this be automated in runhook?

[sdefresne, 2016-09-19 09:43:21]

On 2016/09/19 09:03:30, noyau wrote:
> lgtm
> 
>
https://codereview.chromium.org/2343853002/diff/1/docs/ios_build_instructions.md
> File docs/ios_build_instructions.md (right):
> 
>
https://codereview.chromium.org/2343853002/diff/1/docs/ios_build_instructions...
> docs/ios_build_instructions.md:17: To checkout the source, use `fetch` command
> from [depot\_tools] in a new
> s/fetch/fetch ios/ ?
> 
>
https://codereview.chromium.org/2343853002/diff/1/docs/ios_build_instructions...
> docs/ios_build_instructions.md:66: the list of targets and files in the Xcode
> solution may be stale.
> Can this be automated in runhook?

No, upstream do not want to automatically call gn from runhooks (on any
platform).

[sdefresne, 2016-09-19 09:48:03]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2016-09-19 09:48:03]

The patchset sent to the CQ was uploaded after l-g-t-m from noyau@chromium.org
Link to the patchset: https://codereview.chromium.org/2343853002/#ps20001
(title: "Fix comment.")

[commit-bot: I haz the power, 2016-09-19 09:48:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-19 09:56:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-19 09:58:40]

Description was changed from

==========
Upstream helper script to build Chromium on iOS and update instructions.

Upstream some scripts used to simplify the setup of a new Chromium on iOS
checkout and update the instruction to remove obsolete informations, and
add recommendation to use the script.

BUG=None
==========

to

==========
Upstream helper script to build Chromium on iOS and update instructions.

Upstream some scripts used to simplify the setup of a new Chromium on iOS
checkout and update the instruction to remove obsolete informations, and
add recommendation to use the script.

BUG=None

Committed: https://crrev.com/7f4c25f2b5b8c745b254197282630dbcd7bfa755
Cr-Commit-Position: refs/heads/master@{#419422}
==========

[commit-bot: I haz the power, 2016-09-19 09:58:41]

Patchset 2 (id:??) landed as
https://crrev.com/7f4c25f2b5b8c745b254197282630dbcd7bfa755
Cr-Commit-Position: refs/heads/master@{#419422}