Specify dependences correctly when building mac tools for iOS.

Specify dependences correctly when building mac tools for iOS.

Several auxiliary tools used when building Chrome for iOS are generated via the
following two-step pattern: (1) compiling the executable with ninja, and (2)
copying the executable into a location that is shared with other projects.
Previously, these two steps were specified as actions in the same target.
However, as the ordering of multiple actions in one target is defined only by
inputs and outputs and the compilation action had no inputs/outputs set, the
copying action wasn't properly depending on the compilation action.  

As it's challenging to set correct inputs for the ninja build, this CL
separates these actions into two targets in order to be able to specify that
the second action should not run until the first action finishes.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=164743

[qsr, 2012-10-29 10:42:39]

I don't like that the dependency to protoc is not the same of ios and non-ios.
Could you instead modify protoby.gyp to put the protoc target in a if (!ios) {}
else {} block and have protoc be the right external dependency, as you did for
iossim for example.

[blundell, 2012-10-29 16:10:11]

On 2012/10/29 10:42:39, qsr wrote:
> I don't like that the dependency to protoc is not the same of ios and non-ios.
> Could you instead modify protoby.gyp to put the protoc target in a if (!ios)
{}
> else {} block and have protoc be the right external dependency, as you did for
> iossim for example.

Done. PTAL.

[qsr, 2012-10-29 16:41:26]

lgtm

[blundell, 2012-10-29 16:51:10]

+rohitrao for iossim OWNERS.
+mark for breakpad OWNERS.

[Mark Mentovai, 2012-10-29 18:33:53]

LGTM for breakpad OWNERS approval.

[rohitrao (ping after 24h), 2012-10-29 18:35:25]

LGTM for iossim.

[commit-bot: I haz the power, 2012-10-29 18:56:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/blundell@chromium.org/11301003/27002

[commit-bot: I haz the power, 2012-10-29 21:36:37]

Change committed as 164743