Reland of Simplify import-w3c-tests: support exactly one argument.

Reland of Simplify import-w3c-tests: support exactly one argument. (patchset #1 id:1 of https://codereview.chromium.org/2048953002/ )

Reason for revert:
Making a reland CL to try to fix the unit test failure.

Original issue's description:
> Revert of Simplify import-w3c-tests: support exactly one argument. (patchset #7 id:120001 of https://codereview.chromium.org/1998413002/ )
>
> Reason for revert:
> This seems to have caused consecutive webkit python test failures. Probably there should be an easy fix but let me revert this for now.
>
> {
>   "failures": [
>     "webkitpy/w3c/test_importer_unittest/TestImporterTest/test_does_not_import_reftest"
>   ],
>   "valid": true
> }
>
> Original issue's description:
> > Simplify import-w3c-tests: support exactly one argument.
> >
> > Specifically, take the source repo path (top of repo) as the required argument, and don't support importing single subdirectories.
> >
> > Reason:
> >
> >  1. Using the current directory as the top of repo may be a bit confusing.
> >
> >  2. In the future, I believe we want some kind of auto-updater that updates everything at once and we won't need to support importing individual directories. At present, I believe the only thing that uses import-w3c-tests is update-w3c-deps which, which doesn't currently have an option for updating just one subdirectory.
> >
> > Committed: https://crrev.com/868f94e75ad4b86b2a8a6c15b694cde508af91ee
> > Cr-Commit-Position: refs/heads/master@{#398437}
>
> TBR=dpranke@chromium.org,ojan@chromium.org,qyearsley@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
>
> Committed: https://crrev.com/c34176442a2f64bfec4960a72a92d93865863ee9
> Cr-Commit-Position: refs/heads/master@{#398473}

TBR=dpranke@chromium.org,ojan@chromium.org,kinuko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/6fe70e36410ade1c2aaf45eeb0eeaa81c8004717
Cr-Commit-Position: refs/heads/master@{#399782}

[qyearsley, 2016-06-08 16:02:28]

The CQ bit was checked by qyearsley@chromium.org

[qyearsley, 2016-06-08 16:02:28]

Created Reland of Simplify import-w3c-tests: support exactly one argument.

[qyearsley, 2016-06-08 16:02:37]

The CQ bit was unchecked by qyearsley@chromium.org

[Dirk Pranke, 2016-06-08 16:06:21]

what's the change from the original patch, and do we know how this got past the
CQ?

[qyearsley, 2016-06-08 16:13:51]

On 2016/06/08 at 16:06:21, dpranke wrote:
> what's the change from the original patch, and do we know how this got past
the CQ?

The fix is changing the newly-added test "test_does_not_import_reftest".

This CL conflicted with https://codereview.chromium.org/2045013002, which added
that test method and landed at approximately the same time -- I believe both
went through the CQ at the same time, but combined they cause a test failure.

Note, upon re-reading, I see that test_does_not_import_reftest and
test_does_not_import_owner_files are the same, so they can be combined.

[qyearsley, 2016-06-13 23:09:31]

On 2016/06/08 at 16:13:51, qyearsley wrote:
> On 2016/06/08 at 16:06:21, dpranke wrote:
> > what's the change from the original patch, and do we know how this got past
the CQ?
> 
> The fix is changing the newly-added test "test_does_not_import_reftest".
> 
> This CL conflicted with https://codereview.chromium.org/2045013002, which
added that test method and landed at approximately the same time -- I believe
both went through the CQ at the same time, but combined they cause a test
failure.
> 
> Note, upon re-reading, I see that test_does_not_import_reftest and
test_does_not_import_owner_files are the same, so they can be combined.

This should be ready for review, although I noticed in the recent discussion on
blink-dev
(https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/iMQwsUiW0xM)
Christian Biesinger noted that individual directories could be updated
separately by owners of that directory.

I still think eventually we may have some auto-updating process where all of the
imported W3C tests are updated at once, but meanwhile, it might still be a good
idea to provide an option to just update only one directory -- so it might be
good to keep dir_to_import as an option.

[Dirk Pranke, 2016-06-14 21:01:26]

lgtm. 

If we get to a point where we need to import the repos independently we can
worry about it then.

[qyearsley, 2016-06-14 21:10:12]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-06-14 21:10:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049803002/70001

[commit-bot: I haz the power, 2016-06-14 21:40:46]

Committed patchset #2 (id:70001)

[commit-bot: I haz the power, 2016-06-14 21:40:52]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-14 21:42:14]

Description was changed from

==========
Reland of Simplify import-w3c-tests: support exactly one argument. (patchset #1
id:1 of https://codereview.chromium.org/2048953002/ )

Reason for revert:
Making a reland CL to try to fix the unit test failure.

Original issue's description:
> Revert of Simplify import-w3c-tests: support exactly one argument. (patchset
#7 id:120001 of https://codereview.chromium.org/1998413002/ )
> 
> Reason for revert:
> This seems to have caused consecutive webkit python test failures. Probably
there should be an easy fix but let me revert this for now.
> 
> {
>   "failures": [
>    
"webkitpy/w3c/test_importer_unittest/TestImporterTest/test_does_not_import_reftest"
>   ],
>   "valid": true
> }
> 
> Original issue's description:
> > Simplify import-w3c-tests: support exactly one argument.
> >
> > Specifically, take the source repo path (top of repo) as the required
argument, and don't support importing single subdirectories.
> >
> > Reason:
> >
> >  1. Using the current directory as the top of repo may be a bit confusing.
> >
> >  2. In the future, I believe we want some kind of auto-updater that updates
everything at once and we won't need to support importing individual
directories. At present, I believe the only thing that uses import-w3c-tests is
update-w3c-deps which, which doesn't currently have an option for updating just
one subdirectory.
> >
> > Committed: https://crrev.com/868f94e75ad4b86b2a8a6c15b694cde508af91ee
> > Cr-Commit-Position: refs/heads/master@{#398437}
> 
> TBR=dpranke@chromium.org,ojan@chromium.org,qyearsley@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> 
> Committed: https://crrev.com/c34176442a2f64bfec4960a72a92d93865863ee9
> Cr-Commit-Position: refs/heads/master@{#398473}

TBR=dpranke@chromium.org,ojan@chromium.org,kinuko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Reland of Simplify import-w3c-tests: support exactly one argument. (patchset #1
id:1 of https://codereview.chromium.org/2048953002/ )

Reason for revert:
Making a reland CL to try to fix the unit test failure.

Original issue's description:
> Revert of Simplify import-w3c-tests: support exactly one argument. (patchset
#7 id:120001 of https://codereview.chromium.org/1998413002/ )
>
> Reason for revert:
> This seems to have caused consecutive webkit python test failures. Probably
there should be an easy fix but let me revert this for now.
>
> {
>   "failures": [
>    
"webkitpy/w3c/test_importer_unittest/TestImporterTest/test_does_not_import_reftest"
>   ],
>   "valid": true
> }
>
> Original issue's description:
> > Simplify import-w3c-tests: support exactly one argument.
> >
> > Specifically, take the source repo path (top of repo) as the required
argument, and don't support importing single subdirectories.
> >
> > Reason:
> >
> >  1. Using the current directory as the top of repo may be a bit confusing.
> >
> >  2. In the future, I believe we want some kind of auto-updater that updates
everything at once and we won't need to support importing individual
directories. At present, I believe the only thing that uses import-w3c-tests is
update-w3c-deps which, which doesn't currently have an option for updating just
one subdirectory.
> >
> > Committed: https://crrev.com/868f94e75ad4b86b2a8a6c15b694cde508af91ee
> > Cr-Commit-Position: refs/heads/master@{#398437}
>
> TBR=dpranke@chromium.org,ojan@chromium.org,qyearsley@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
>
> Committed: https://crrev.com/c34176442a2f64bfec4960a72a92d93865863ee9
> Cr-Commit-Position: refs/heads/master@{#398473}

TBR=dpranke@chromium.org,ojan@chromium.org,kinuko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/6fe70e36410ade1c2aaf45eeb0eeaa81c8004717
Cr-Commit-Position: refs/heads/master@{#399782}
==========

[commit-bot: I haz the power, 2016-06-14 21:42:15]

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