Increase file length limit for w3c test importer

Increase file path length limit for w3c test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871
Review-Url: https://codereview.chromium.org/2645323003
Cr-Commit-Position: refs/heads/master@{#445546}
Committed: https://chromium.googlesource.com/chromium/src/+/706a4e2dceb1e19d8b74cff5a9cd8b04dc781b1d

[majidvp, 2017-01-23 20:26:10]

majidvp@chromium.org changed reviewers:
+ qyearsley@chromium.org

[majidvp, 2017-01-23 20:26:26]

Description was changed from

==========
Increase file length limit for test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871
==========

to

==========
Increase file length limit for w3c test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871
==========

[qyearsley, 2017-01-23 21:04:44]

lgtm

https://codereview.chromium.org/2645323003/diff/20001/third_party/WebKit/Tool...
File third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_importer.py (right):

https://codereview.chromium.org/2645323003/diff/20001/third_party/WebKit/Tool...
third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_importer.py:47:
MAX_PATH_LENGTH = 140
Let's see, if the path to the Chromium repo were:

  "C:\b\build\slave\Win7_Release__NVIDIA_GeForce_730_\build\src" (60 characters)

then the path length we should allow would be around 260 - 60 = ~200

Of course, that's just assuming that base path; base path may be longer. So 140
sounds good to me :-)

[majidvp, 2017-01-23 21:16:19]

The CQ bit was checked by majidvp@chromium.org

[majidvp, 2017-01-23 21:16:27]

The CQ bit was unchecked by majidvp@chromium.org

[majidvp, 2017-01-23 21:16:38]

Description was changed from

==========
Increase file length limit for w3c test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871
==========

to

==========
Increase file path length limit for w3c test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871
==========

[majidvp, 2017-01-23 21:16:44]

The CQ bit was checked by majidvp@chromium.org

[commit-bot: I haz the power, 2017-01-23 21:17:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-23 23:45:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485206204667170,
"parent_rev": "7aaa35ff0a9ab97fba82c7edcc2c90c186828e10", "commit_rev":
"706a4e2dceb1e19d8b74cff5a9cd8b04dc781b1d"}

[commit-bot: I haz the power, 2017-01-23 23:45:44]

Description was changed from

==========
Increase file path length limit for w3c test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871
==========

to

==========
Increase file path length limit for w3c test importer

Since the original limit was added the base path has been reduced by 15 chars.*
Increasing the limit by this amount allows us to import more tests without
breaking any existing windows bots.

* from 'imported/web-platform-tests' to 'external/wpt'

BUG=609871

Review-Url: https://codereview.chromium.org/2645323003
Cr-Commit-Position: refs/heads/master@{#445546}
Committed:
https://chromium.googlesource.com/chromium/src/+/706a4e2dceb1e19d8b74cff5a9cd...
==========

[commit-bot: I haz the power, 2017-01-23 23:45:48]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/706a4e2dceb1e19d8b74cff5a9cd...