android tests: don't override DIR_MODULE (take 2)

android tests: don't override DIR_MODULE (take 2)

another attempt at https://codereview.chromium.org/408063002/ to try fixing
the iOS Simulator:URLFixerTest failures writing to DIR_TEST_DATA.

R=nyquist@chromium.org
TBR=brettw@chromium.org, nyquist@chromium.org, sky@chromium.org
BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285451

[tim (not reviewing), 2014-07-24 18:48:54]

TBR reviewers; Only change from previous is I'm trying to satisfy iOS Simulator
by picking different test directories to write to in URLFixerTest.

[tim (not reviewing), 2014-07-24 19:48:11]

The CQ bit was checked by tim@chromium.org

[commit-bot: I haz the power, 2014-07-24 19:51:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tim@chromium.org/411253010/1

[nyquist, 2014-07-24 21:08:46]

lgtm

[sky, 2014-07-24 22:15:27]

In the future please upload the original patch (sync'ed to trunk) followed by
the fix. That way it's easier to see the changes. Alternatively reopen the
original issue and add the change to it.

[tim (not reviewing), 2014-07-24 22:33:06]

noted! thanks for the tip.

On Thu, Jul 24, 2014 at 3:15 PM,  <sky@chromium.org> wrote:
> In the future please upload the original patch (sync'ed to trunk) followed
> by
> the fix. That way it's easier to see the changes. Alternatively reopen the
> original issue and add the change to it.
>
> https://codereview.chromium.org/411253010/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2014-07-24 23:04:15]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_rel_swarming on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel_sw...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[tim (not reviewing), 2014-07-25 01:51:55]

On 2014/07/24 23:04:15, I haz the power (commit-bot) wrote:
> FYI, CQ is re-trying this CL (attempt #1).
> The failing builders are:
>   linux_chromium_rel_swarming on tryserver.chromium
>
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel_sw...)
>   mac_chromium_rel on tryserver.chromium
>
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
>   win_chromium_rel on tryserver.chromium
>
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)


Previously failing test is passing, mac_chromium_rel failure seems unrelated:
http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...,
and
also seeing similar failures on update_scripts @ win_gpu_triggered_tests.

[tim (not reviewing), 2014-07-25 01:53:30]

Committed patchset #1 manually as r285451 (presubmit successful).