Roll DEPS to class-dump 3.5

Roll DEPS to class-dump 3.5

Removes some build warnings.

Moves the class-dump repo to a github auto-mirror instead of a mirror
that needs to be manually updated.

This also requires that the class-dump.gyp file move from the
third-party repo into the main source tree. Note that the file
structure of the class-dump source changed so the .gyp had to be
updated. For comparison, the old .class-dump.gyp can be found at
https://chromium.googlesource.com/chromium/deps/class-dump/+/master/class-dump.gyp

BUG=318160
Committed: https://crrev.com/1c80791ce7c29a29fc816d89d1a5fa2ac6503b3e
Cr-Commit-Position: refs/heads/master@{#330798}

[lliabraa, 2015-04-27 13:05:24]

lliabraa@chromium.org changed reviewers:
+ justincohen@chromium.org

[lliabraa, 2015-04-27 13:10:17]

The CQ bit was checked by lliabraa@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-27 13:10:21]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1106993002/1

[justincohen, 2015-04-27 13:56:10]

In description: s/to ta github/to the github

LGTM with nits.

https://codereview.chromium.org/1106993002/diff/1/testing/iossim/class-dump.gyp
File testing/iossim/class-dump.gyp (right):

https://codereview.chromium.org/1106993002/diff/1/testing/iossim/class-dump.g...
testing/iossim/class-dump.gyp:1: # Copyright (c) 2012 The Chromium Authors. All
rights reserved.
2015

[lliabraa, 2015-04-27 13:58:23]

The CQ bit was checked by lliabraa@chromium.org

[lliabraa, 2015-04-27 13:58:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
justincohen@chromium.org
Link to the patchset: https://codereview.chromium.org/1106993002/#ps20001
(title: "fix copyright")

[commit-bot: I haz the power, 2015-04-27 13:58:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1106993002/20001

[commit-bot: I haz the power, 2015-04-27 13:58:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-27 13:58:52]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lliabraa, 2015-04-27 14:02:25]

lliabraa@chromium.org changed reviewers:
+ rohitrao@chromium.org

[lliabraa, 2015-04-27 14:02:26]

+rohitrao for OWNERS

[rohitrao (ping after 24h), 2015-04-27 14:11:14]

LGTM

The more standard pattern would be to put the DEPS repo into a
iossim/third_party/class-dump/src subdirectory, which leaves room for a
iossim/third_party/class-dump/class-dump.gyp file.

That would be preferable, but I'm not sure if gclient will be able to handle
that sort of switch.

https://codereview.chromium.org/1106993002/diff/20001/testing/iossim/iossim.gyp
File testing/iossim/iossim.gyp (left):

https://codereview.chromium.org/1106993002/diff/20001/testing/iossim/iossim.g...
testing/iossim/iossim.gyp:86:
'third_party/class-dump/class-dump.gyp:class-dump#host',
Should we also delete the old file?

[rohitrao (ping after 24h), 2015-04-27 14:11:51]

https://codereview.chromium.org/1106993002/diff/20001/testing/iossim/iossim.gyp
File testing/iossim/iossim.gyp (left):

https://codereview.chromium.org/1106993002/diff/20001/testing/iossim/iossim.g...
testing/iossim/iossim.gyp:86:
'third_party/class-dump/class-dump.gyp:class-dump#host',
On 2015/04/27 14:11:14, rohitrao wrote:
> Should we also delete the old file?

Oh nevermind, just realized this was in the DEPS repo.

[lliabraa, 2015-04-27 14:47:34]

The CQ bit was checked by lliabraa@chromium.org to run a CQ dry run

[lliabraa, 2015-04-27 14:47:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
justincohen@chromium.org, rohitrao@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1106993002/#ps40001
(title: "moved checkout to testing/iossim/third_party/class-dump/src")

[commit-bot: I haz the power, 2015-04-27 14:47:44]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1106993002/40001

[lliabraa, 2015-04-27 14:49:37]

PTAL - I moved the checkout location based on Rohit's feedback and included the
README.chromium et al. that was previously brought in from the manual chromium
mirror.

[commit-bot: I haz the power, 2015-04-27 14:50:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-27 14:51:00]

Dry run: Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[lliabraa, 2015-05-20 18:41:02]

The CQ bit was checked by lliabraa@chromium.org to run a CQ dry run

[lliabraa, 2015-05-20 18:41:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
justincohen@chromium.org, rohitrao@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1106993002/#ps60001
(title: "revert to patchset1 due to gclient limitations")

[lliabraa, 2015-05-20 18:41:09]

The CQ bit was unchecked by lliabraa@chromium.org

[lliabraa, 2015-05-20 18:41:13]

The CQ bit was checked by lliabraa@chromium.org

[commit-bot: I haz the power, 2015-05-20 18:42:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1106993002/60001

[commit-bot: I haz the power, 2015-05-20 20:54:48]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-20 20:55:36]

Patchset 4 (id:??) landed as
https://crrev.com/1c80791ce7c29a29fc816d89d1a5fa2ac6503b3e
Cr-Commit-Position: refs/heads/master@{#330798}

[sgurun-gerrit only, 2015-05-20 21:16:20]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1147163003/ by sgurun@chromium.org.

The reason for reverting is: Breaking
https://build.chromium.org/p/chromium.mac/builders/iOS_Device/builds/7076

.