| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
Switches gtests to explicitly passing path to install script rather than
--incremental-install + heuristics.
BUG=575975
Review URL: https://codereview.chromium.org/1680233002
Cr-Commit-Position: refs/heads/master@{#374699}
|
|
|
|
|
|
|
|
| |
BUG=476719
Review URL: https://codereview.chromium.org/1571803002
Cr-Commit-Position: refs/heads/master@{#368599}
|
|
|
|
|
|
|
|
| |
BUG=476719
Review URL: https://codereview.chromium.org/1516363002
Cr-Commit-Position: refs/heads/master@{#364762}
|
|
|
|
|
|
|
|
|
|
|
| |
The default depot_tools pylintrc disables some checks that we can
reasonably enable in build/android/. This CL does so.
BUG=
Review URL: https://codereview.chromium.org/1315743004
Cr-Commit-Position: refs/heads/master@{#347720}
|
|
|
|
|
|
|
|
| |
BUG=476719
Review URL: https://codereview.chromium.org/1314313004
Cr-Commit-Position: refs/heads/master@{#347187}
|
|
|
|
|
|
|
|
|
|
| |
This is a reland of https://codereview.chromium.org/1281923003
BUG=517709
Review URL: https://codereview.chromium.org/1294713008
Cr-Commit-Position: refs/heads/master@{#345779}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#5 id:80001 of https://codereview.chromium.org/1281923003/ )
Reason for revert:
broke the perf bots, will fix and reland.
Original issue's description:
> [Android] Add --blacklist-file as a command-line option.
>
> BUG=517709
>
> Committed: https://crrev.com/9874769cf24b5870d9bb7c22fe5312b1cf8bcd5e
> Cr-Commit-Position: refs/heads/master@{#344125}
TBR=eakuefner@chromium.org,mikecase@chromium.org,nednguyen@google.com,rnephew@chromium.org,sullivan@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=517709
Review URL: https://codereview.chromium.org/1292053006
Cr-Commit-Position: refs/heads/master@{#344141}
|
|
|
|
|
|
|
|
| |
BUG=517709
Review URL: https://codereview.chromium.org/1281923003
Cr-Commit-Position: refs/heads/master@{#344125}
|
|
|
|
|
|
|
|
|
|
| |
(RELAND)
BUG=267773,482367,482406
Review URL: https://codereview.chromium.org/1112843002
Cr-Commit-Position: refs/heads/master@{#327501}
|
|
|
|
|
|
|
|
|
|
| |
Leaving out RestartAdbd this time. I'll address that separately.
BUG=267773
Review URL: https://codereview.chromium.org/1078223002
Cr-Commit-Position: refs/heads/master@{#324904}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #2 id:20001 of https://codereview.chromium.org/1050883002/)
Reason for revert:
Broke the perf bots again.
Original issue's description:
> [Android] Reland cleanup of old_interface in build/android/pylib.
>
> Original CL: https://codereview.chromium.org/1047703002/
>
> BUG=267773
TBR=klundberg@chromium.org,perezju@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=267773
Review URL: https://codereview.chromium.org/1059443005
Cr-Commit-Position: refs/heads/master@{#323573}
|
|
|
|
|
|
|
|
|
|
| |
Original CL: https://codereview.chromium.org/1047703002/
BUG=267773
Review URL: https://codereview.chromium.org/1050883002
Cr-Commit-Position: refs/heads/master@{#323518}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:40001 of https://codereview.chromium.org/1047703002/)
Reason for revert:
This make sharded perf tests fail everywhere, example:
http://build.chromium.org/p/chromium.perf/builders/Android%20Nexus4%20Perf/builds/2797
Original issue's description:
> [Android] Clean up old_interface in build/android/.
>
> BUG=267773
>
> Committed: https://crrev.com/32b3e358017c863318fa7cca3a3c41bb5d056c24
> Cr-Commit-Position: refs/heads/master@{#323135}
TBR=perezju@chromium.org,klundberg@chromium.org,jbudorick@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=267773
Review URL: https://codereview.chromium.org/1049993003
Cr-Commit-Position: refs/heads/master@{#323202}
|
|
|
|
|
|
|
|
| |
BUG=267773
Review URL: https://codereview.chromium.org/1047703002
Cr-Commit-Position: refs/heads/master@{#323135}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ResourceFetcherTests are running well even on Android now except
ResourceFetcher404.
I confirmed that the payload of http response of ResourceFetcher404
included error logs of the test server as well, which was causing
the failure.
To enable ResourceFetcher404, I changed sys.stderr to sys.stdout on
Android.
BUG=248796
Review URL: https://codereview.chromium.org/986233002
Cr-Commit-Position: refs/heads/master@{#319996}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL makes a couple of small modifications to enable instrumentation
tests of managed sign ins:
- added the policy_testserver as another type of host-driver test server
- added /test/ping to the policy_testserver for liveness checks
- added a configuration for the policy_testserver for the Android tests
- added a UserInfo blob for the managed sign in tests
- added BookmarksBridge.bookmarkAllUserNodesRemoved() to observe the
wiping of the bookmarks on managed sign out
- refactored the SigninManager to show the managed sign in alert dialog
as a separated fragment. This doesn't change the UI in any way,
but makes testing easier by giving a tag to the dialog fragment.
BUG=436068
Review URL: https://codereview.chromium.org/755583004
Cr-Commit-Position: refs/heads/master@{#306084}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/696923002
Cr-Commit-Position: refs/heads/master@{#302494}
|
|
|
|
|
|
|
|
| |
BUG=267773
Review URL: https://codereview.chromium.org/404993004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285097 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Chrome for Android instrumentation tests (the tests in ChromeTest.apk)
need to start some services to test Chromecast support. This was
originally done within the ChromeTest.apk, but, because of the changes
required for https://code.google.com/p/chromium/issues/detail?id=272790
this is no longer possible. This patch allows the use of a seperate test
support apk.
BUG=384537
Review URL: https://codereview.chromium.org/336373004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279096 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=267773
Review URL: https://codereview.chromium.org/235943002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263484 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
build/android/.
BUG=267773
Review URL: https://codereview.chromium.org/221823011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263055 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=168518
Review URL: https://codereview.chromium.org/194843003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256429 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=280354
Review URL: https://codereview.chromium.org/144183004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252687 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=168518
NOTRY=true
Review URL: https://codereview.chromium.org/163923002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251153 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=168518
Review URL: https://codereview.chromium.org/159853010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250822 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(https://codereview.chromium.org/132463007/)
Reason for revert:
This patch is probably the cause of crbug.com/342539.
Original issue's description:
> Enable presubmit pylint in build/android.
>
> BUG=168518
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=249793
TBR=craigdh@chromium.org,frankf@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=168518
Review URL: https://codereview.chromium.org/153743008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250269 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=168518
Review URL: https://codereview.chromium.org/132463007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249793 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The sync tests failure seems to have to do with port unmapping. Not unmapping
ports let most of the sync tests pass (only 2 failures remain for an unrelated
reason apparently).
Note that port unmapping was added to avoid exceeding the file descriptor limit
in net_unittests since each test server was listening on a different port. The
sync tests use a server always listening on a same port so unmapping the port
is not strictly needed.
BUG=313809
Review URL: https://codereview.chromium.org/106533002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@239467 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rather than (un)mapping the ports on each one of the internal instrumentation
tests, do it higher up at the host driven level.
Ensure the test server is fully setup and tear down.
Ensure the adb port is fully unmapped at the host forwarder level.
Add more log information to the forwarder, and add them to TestResult.log in case of failure.
Kill the device forwarder when unmapping all ports.
BUG=313809
Review URL: https://codereview.chromium.org/59313003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234486 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Use the one defined in constants.py instead.
BUG=
TEST=build/android/test_runner.py instrumentation --test-apk ChromiumTestShellTest --test_data chrome:chrome/test/data/android/device_files --verbose --host-driven-root=chrome/android -f testDisableAndEnableSync --python-only
Review URL: https://codereview.chromium.org/59713002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233057 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
scripts to use it.
BUG=260494
TEST=None
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/23494039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222313 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Consistently causing WebKit Android (Nexus4) to fail provision_devices.
> [android] Adds constants.GetOutDirectory() and converts test scripts to use it.
>
> BUG=260494
> TEST=None
> NOTRY=True
>
> Review URL: https://chromiumcodereview.appspot.com/22903016
TBR=craigdh@chromium.org
Review URL: https://codereview.chromium.org/23926012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222008 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=260494
TEST=None
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/22903016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221736 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Whenever a host driven test need to setup a server as part of a test,
the command line needs of the application to be instrumented needs to
change.
This CL adds a helper class for setting up a test server, and also adds
support for tests to setup additional flags (typically the URL for how
to connect to the test server).
Reverted in: http://crrev.com/220769
Relanding after re-upload to the same patch.
BUG=272584
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220750
Review URL: https://chromiumcodereview.appspot.com/23726004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221122 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
check_perms failing on linux cros bots:
http://chromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Full/builds/24739
http://chromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Full/builds/24739/steps/check_perms/logs/stdio
> Add test_server_setup.py and add support for additional flags.
>
> Whenever a host driven test need to setup a server as part of a test,
> the command line needs of the application to be instrumented needs to
> change.
>
> This CL adds a helper class for setting up a test server, and also adds
> support for tests to setup additional flags (typically the URL for how
> to connect to the test server).
>
> Depends on: https://codereview.chromium.org/23467004/
>
> BUG=272584
> NOTRY=true
>
> Review URL: https://chromiumcodereview.appspot.com/23726004
TBR=nyquist@chromium.org
Review URL: https://codereview.chromium.org/23606011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220769 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Whenever a host driven test need to setup a server as part of a test,
the command line needs of the application to be instrumented needs to
change.
This CL adds a helper class for setting up a test server, and also adds
support for tests to setup additional flags (typically the URL for how
to connect to the test server).
Depends on: https://codereview.chromium.org/23467004/
BUG=272584
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/23726004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220750 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
NOTRY=True
BUG=176323
Review URL: https://chromiumcodereview.appspot.com/22891019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219048 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Host-driven tests should use _RunJavaTestFilters instead now.
NOTRY=True
BUG=268683
Review URL: https://chromiumcodereview.appspot.com/22929017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218751 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This eliminates the need to specify package name and bring it
more in line with normal instrumentation tests.
NOTRY=True
BUG=268683
Review URL: https://chromiumcodereview.appspot.com/23295006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218530 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
NOTRY=True
BUG=176323
Review URL: https://chromiumcodereview.appspot.com/23295007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218453 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=260494
TEST=None
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/22933005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217855 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also renames several files in pylib/host_driven to match the general file
naming scheme.
This change will break existing host-driven tests downstream which are run
though scripts other than test_runner.
NOTRY=True
BUG=176323
Review URL: https://chromiumcodereview.appspot.com/19537004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215944 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Options objects are specific to the test type, and are named tuples, so
they are also immutable.
This is a precursor to the host-driven refactor.
BUG=176323
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/21008004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215054 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The initial attempt caused flakiness (random deadlocks). It turns out that
testserver.py was inheriting undesirable file descriptors when it was spawned
(including file lock file descriptors). Closing the unnecessary file
descriptors between fork() and exec() solves the problem.
Forwarder used to be a pain to setup/tear down across all the various
harnesses.
This CL should hopefully solve these issues by hiding these
implementation details. The host daemon is now killed once the first time that
the Forwarder class is used and the daemon running on the devices is also
killed the first time a port is forwarded for a specific device.
TBR=bulach@chromium.org
BUG=242846
Review URL: https://codereview.chromium.org/20824008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215023 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
based on Md5Sum.
This deprecates the -I switch to instrumention tests.
BUG=260084
TEST=None
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/19799003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213956 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts r212020 which caused some timeouts on the net_unittests every ~15
builds.
Conflicts:
build/android/pylib/base/test_dispatcher.py
build/android/pylib/instrumentation/test_runner.py
chrome/test/chromedriver/run_py_tests.py
TBR=bulach@chromium.org
BUG=262573,242846
Review URL: https://codereview.chromium.org/19844006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212888 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
In addition, cleans up dispatch.
BUG=259063, 259128
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/18770008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212663 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Forwarder used to be a pain to setup/tear down across all the various
harnesses.
This CL should hopefully solve these issues by hiding these
implementation details. The host daemon is now killed once the first time that
the Forwarder class is used and the daemon running on the devices is also
killed the first time a port is forwarded for a specific device.
TBR=bulach@chromium.org
BUG=242846
Review URL: https://codereview.chromium.org/19550004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212582 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This causes flakiness on the Android debug tester bot.
> Move Python setup/tear down logic into Forwarder itself.
>
> Forwarder used to be a pain to setup/tear down across all the various
> harnesses.
> This CL should hopefully solve these issues by hiding these implementation
> details. The host daemon is now killed once the first time that the Forwarder
> class is used and the daemon running on the devices is also killed the first
> time a port is forwarded for a specific device.
>
> BUG=242846
> R=bulach@chromium.org, frankf@chromium.org
>
> Review URL: https://codereview.chromium.org/18086004
TBR=pliard@chromium.org
Review URL: https://codereview.chromium.org/19576005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212054 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Forwarder used to be a pain to setup/tear down across all the various
harnesses.
This CL should hopefully solve these issues by hiding these implementation
details. The host daemon is now killed once the first time that the Forwarder
class is used and the daemon running on the devices is also killed the first
time a port is forwarded for a specific device.
BUG=242846
R=bulach@chromium.org, frankf@chromium.org
Review URL: https://codereview.chromium.org/18086004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212020 0039d316-1c4b-4281-b951-d872f2087c98
|