Add wifi_manager.py and WifiUtils.apk from tradefed to public checkout.

Add wifi_manager.py and WifiUtils.apk from tradefed to public checkout.

BUG=495153

[navabi, 2015-06-18 01:04:57]

navabi@google.com changed reviewers:
+ jbudorick@chromium.org, navabi@google.com, perezju@chromium.org,
pschmidt@google.com

[navabi, 2015-06-18 01:04:58]

https://codereview.chromium.org/1187373002/diff/1/build/android/pylib/device/...
File build/android/pylib/device/wifi_manager.py (right):

https://codereview.chromium.org/1187373002/diff/1/build/android/pylib/device/...
build/android/pylib/device/wifi_manager.py:26: WifiUtil is available at
build/android/apks/WifiUtil.apk, for more
changed this.

https://codereview.chromium.org/1187373002/diff/1/build/android/pylib/device/...
build/android/pylib/device/wifi_manager.py:154: def _WifiUtilMethod(self,
method, args=None, type_fn=None):
and I changed the name of the type_fn argument. It was type so pylint was
complaining.

[perezju, 2015-06-18 09:53:33]

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
File build/android/apks/README.chromium (right):

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
build/android/apks/README.chromium:1: Name: WifiUtil apk
maybe call this "Android APKs", so in the future we can drop here other APKs we
might need?

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
build/android/apks/README.chromium:12: with cherry-pick
e9d4e8bab7fbb696d5f1b8115da35a2ee887bf6c
was this something we built ourselves? we should probably add how to get this
info on the upgrade instructions.

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
build/android/apks/README.chromium:19: machine_setup.html
To save us a few clicks, maybe also add here the full command, e.g.:

$ repo init -u https://android.googlesource.com/platform/manifest -b tradefed

https://codereview.chromium.org/1187373002/diff/1/build/android/pylib/device/...
File build/android/pylib/device/wifi_manager.py (right):

https://codereview.chromium.org/1187373002/diff/1/build/android/pylib/device/...
build/android/pylib/device/wifi_manager.py:154: def _WifiUtilMethod(self,
method, args=None, type_fn=None):
On 2015/06/18 01:04:57, navabi wrote:
> and I changed the name of the type_fn argument. It was type so pylint was
> complaining.

nit: maybe call it "return_as"? looks a bit nicer in the callers.

[jbudorick, 2015-06-18 14:21:48]

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
File build/android/apks/README.chromium (right):

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
build/android/apks/README.chromium:1: Name: WifiUtil apk
I don't think we want this in build/android/. I think it belongs somewhere in
third_party and should only be DEPSed in for android checkouts, similar to how
we handle the android SDK in third_party/android_tools.

[navabi, 2015-06-18 16:52:34]

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
File build/android/apks/README.chromium (right):

https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
build/android/apks/README.chromium:1: Name: WifiUtil apk
On 2015/06/18 14:21:47, jbudorick (ooo til 2015-06-18) wrote:
> I don't think we want this in build/android/. I think it belongs somewhere in
> third_party and should only be DEPSed in for android checkouts, similar to how
> we handle the android SDK in third_party/android_tools.

So for android_tools we've created and maintain a separate git repo that is then
DEPsed. I'm fine putting this in another directory, but I'm reluctant to create
a new repo for a single apk.

One idea is checking it into the android_tools repo, as it is used as a testing
tool on Android. WDYT?

[jbudorick, 2015-06-19 04:00:06]

On 2015/06/18 at 16:52:34, navabi wrote:
>
https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
> File build/android/apks/README.chromium (right):
> 
>
https://codereview.chromium.org/1187373002/diff/1/build/android/apks/README.c...
> build/android/apks/README.chromium:1: Name: WifiUtil apk
> On 2015/06/18 14:21:47, jbudorick (ooo til 2015-06-18) wrote:
> > I don't think we want this in build/android/. I think it belongs somewhere
in
> > third_party and should only be DEPSed in for android checkouts, similar to
how
> > we handle the android SDK in third_party/android_tools.
> 
> So for android_tools we've created and maintain a separate git repo that is
then DEPsed. I'm fine putting this in another directory, but I'm reluctant to
create a new repo for a single apk.
> 
> One idea is checking it into the android_tools repo, as it is used as a
testing tool on Android. WDYT?

I think I'd prefer this to be in a repo separate from android_tools, which seems
limited to the SDK & NDK. I don't think that repo would necessarily have to be
limited to just this APK, though it would start that way.

I can't say I'd be an OWNER of android_tools if it had an OWNERS file, though.