Move find_depot_tools.py to build/

Move find_depot_tools.py to build/

build/vs_toolchain.py requires find_depot_tools.py, but it's not
available to a project that gets build/ via DEPS and does not also
get all of tools/ via DEPS.  The native_client repo needs to use
build/vs_toolchain.py, but it does not copy chromium/src/tools/ and
it should not have to duplicate find_depot_tools.py just to use
build/vs_toolchain.py.

The native_client repo also needs a standalone script to print out
the depot_tools directory location, and find_depot_tools.py can
serve that purpose too with some trivial additions.

BUG=512869
R=ben@chromium.org, dbeam@chromium.org, dpranke@chromium.org, jbudorick@chromium.org, klundberg@chromium.org, phajdan.jr@chromium.org, sky@chromium.org, tbreisacher@chromium.org

Committed: https://crrev.com/b729cfa517895b9ea99c98a8b23f1a811772ba5b
Cr-Commit-Position: refs/heads/master@{#356141}

[Dan Beam, 2015-10-23 20:57:41]

"many reviewers means little reviewing" - ancient proverb

[Roland McGrath, 2015-10-23 20:58:41]

On 2015/10/23 20:57:41, Dan Beam wrote:
> "many reviewers means little reviewing" - ancient proverb

Many OWNERS files means no way to avoid many reviewers.

[Roland McGrath, 2015-10-23 21:03:59]

OK, I guess I was being stupid about the OWNERS files.
So I really only need Dirk plus one person from chrome/OWNERS to cover
everything.
ben?

[Dirk Pranke, 2015-10-23 22:07:53]

this basically lgtm . 

I have one minor nit and I think there's one change that's just a mistake
(noted).

https://codereview.chromium.org/1418513007/diff/1/build/find_depot_tools.py
File build/find_depot_tools.py (right):

https://codereview.chromium.org/1418513007/diff/1/build/find_depot_tools.py#n...
build/find_depot_tools.py:10: This can also be used as a standlone script to
print out the depot_tools
nit: "standalone"

https://codereview.chromium.org/1418513007/diff/1/chrome/browser/web_dev_styl...
File chrome/browser/web_dev_style/resource_checker_test.py (right):

https://codereview.chromium.org/1418513007/diff/1/chrome/browser/web_dev_styl...
chrome/browser/web_dev_style/resource_checker_test.py:6: import find_depot_tools
this won't actually work, right? build isn't in the path until line 15.

[Roland McGrath, 2015-10-23 22:55:45]

https://codereview.chromium.org/1418513007/diff/1/build/find_depot_tools.py
File build/find_depot_tools.py (right):

https://codereview.chromium.org/1418513007/diff/1/build/find_depot_tools.py#n...
build/find_depot_tools.py:10: This can also be used as a standlone script to
print out the depot_tools
On 2015/10/23 22:07:53, Dirk Pranke wrote:
> nit: "standalone"

Done.

https://codereview.chromium.org/1418513007/diff/1/chrome/browser/web_dev_styl...
File chrome/browser/web_dev_style/resource_checker_test.py (right):

https://codereview.chromium.org/1418513007/diff/1/chrome/browser/web_dev_styl...
chrome/browser/web_dev_style/resource_checker_test.py:6: import find_depot_tools
On 2015/10/23 22:07:53, Dirk Pranke wrote:
> this won't actually work, right? build isn't in the path until line 15.

Right.  This was a mistake left from a previous (mistaken) iteration.

[Roland McGrath, 2015-10-24 08:02:36]

The CQ bit was checked by mcgrathr@chromium.org

[Roland McGrath, 2015-10-24 08:02:36]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1418513007/#ps20001
(title: "review fixes")

[commit-bot: I haz the power, 2015-10-24 08:02:41]

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

[commit-bot: I haz the power, 2015-10-24 08:09:08]

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

[commit-bot: I haz the power, 2015-10-24 08:09:09]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Dan Beam, 2015-10-24 18:27:58]

web_dev_style lgtm if it helps

[Paweł Hajdan Jr., 2015-10-26 12:55:03]

LGTM

[Roland McGrath, 2015-10-26 18:51:42]

The CQ bit was checked by mcgrathr@chromium.org

[commit-bot: I haz the power, 2015-10-26 18:53:07]

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

[commit-bot: I haz the power, 2015-10-26 19:05:23]

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

[commit-bot: I haz the power, 2015-10-26 19:05:24]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ben Goodger (Google), 2015-10-26 20:28:12]

mojo lgtm

[sky, 2015-10-26 20:57:49]

LGTM

[Roland McGrath, 2015-10-26 21:07:18]

The CQ bit was checked by mcgrathr@chromium.org

[commit-bot: I haz the power, 2015-10-26 21:08:16]

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

[commit-bot: I haz the power, 2015-10-26 22:07:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-26 22:08:37]

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

[bcwhite, 2015-10-29 13:56:54]

Be careful moving this file.  There are three files with the same name but
somewhat different internal naming.  Moving the file means that a different
version of this name can be imported depending upon what the search path happens
to be.

This change breaks gyp_chromium.py under CygWin.  Working on a fix now...

[tfarina, 2015-11-01 01:26:24]

Did this break on WebRTC?

I got this:

    import find_depot_tools
ImportError: No module named find_depot_tools