Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(133)

Issue 64130045: Fix paths to LayoutTests after the Blink merge.

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 8 months ago by eseidel
Modified:
11 years, 8 months ago
Reviewers:
M-A
CC:
swarming-eng_googlegroups.com, csharp+cc_chromium.org, vadimsh+cc_chromium.org
Base URL:
https://chromium.googlesource.com/external/swarming.client.git@master
Visibility:
Public.

Description

Fix paths to LayoutTests after the Blink merge. BUG=345464

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M isolate.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
eseidel
Odd that this uploaded to appspot instead of codereview.chromium.org This can't be landed until after ...
11 years, 8 months ago (2014-02-20 23:39:37 UTC) #1
M-A
FTR, the upstream repo is: https://code.google.com/p/swarming/source/checkout?repo=client So blink moved already? (I must admit I'm a ...
11 years, 8 months ago (2014-02-21 00:03:10 UTC) #2
eseidel
Nope. Blink has not merged into Chromium yet, but it's coming. I'm preparing the demo ...
11 years, 8 months ago (2014-02-21 00:09:22 UTC) #3
M-A
11 years, 8 months ago (2014-02-21 00:57:35 UTC) #4
Ok thanks, I don't think this line it still technically needed, it's been a long time since LayoutTests is optionally checked out so I'll simply remove this line. It happens I can probably remove the other one too, I'll look into it. Thanks for the warning!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b