Adding more browser tests in copy between windows in file manager.

Adding more browser tests in copy between windows in file manager.

BUG=375853
TEST=This is test. Ran tests manually.

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

[hirono, 2014-05-19 04:22:48]

lgtm with a nit. Thank you!

https://codereview.chromium.org/288263003/diff/1/chrome/browser/chromeos/file...
File chrome/browser/chromeos/file_manager/file_manager_browsertest.cc (right):

https://codereview.chromium.org/288263003/diff/1/chrome/browser/chromeos/file...
chrome/browser/chromeos/file_manager/file_manager_browsertest.cc:873:
TestParameter(NOT_IN_GUEST_MODE, "copyBetweenWindowsDriveToLocal"),
nit: Could you please add the tests at the end of the list?
The test names include the index of the test list (e.g.
CopyBetweenWindows/FileManagerBrowserTest/0), so it makes the names in the
previous test logs inconsistent.

[tturchetto1, 2014-05-19 20:15:45]

https://codereview.chromium.org/288263003/diff/1/chrome/browser/chromeos/file...
File chrome/browser/chromeos/file_manager/file_manager_browsertest.cc (right):

https://codereview.chromium.org/288263003/diff/1/chrome/browser/chromeos/file...
chrome/browser/chromeos/file_manager/file_manager_browsertest.cc:873:
TestParameter(NOT_IN_GUEST_MODE, "copyBetweenWindowsDriveToLocal"),
On 2014/05/19 04:22:49, hirono wrote:
> nit: Could you please add the tests at the end of the list?
> The test names include the index of the test list (e.g.
> CopyBetweenWindows/FileManagerBrowserTest/0), so it makes the names in the
> previous test logs inconsistent.

Done.

[tturchetto1, 2014-05-19 20:16:04]

The CQ bit was checked by tturchetto@google.com

[commit-bot: I haz the power, 2014-05-19 20:17:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tturchetto@google.com/288263003/20001

[commit-bot: I haz the power, 2014-05-20 00:08:51]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-05-20 00:13:21]

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

[commit-bot: I haz the power, 2014-05-20 00:13:21]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[satorux1, 2014-05-21 17:57:20]

LGTM based on hirono's. BUG=None is unusual. tturchetto@, could you file a bug
and update the BUG= line?

[tturchetto1, 2014-05-21 18:55:04]

The CQ bit was checked by tturchetto@google.com

[commit-bot: I haz the power, 2014-05-21 19:56:42]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tturchetto@google.com/288263003/20001

[commit-bot: I haz the power, 2014-05-21 20:15:41]

Change committed as 271954