Remove special cases for WPT path rewriting

Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=498120
Committed: https://crrev.com/74c476b03793ce0ba9bfda73d96a8142792f07b4
Cr-Commit-Position: refs/heads/master@{#402858}

[Rick Byers, 2016-06-23 19:44:29]

Description was changed from

==========
Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=614778
==========

to

==========
Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=498120
==========

[Rick Byers, 2016-06-23 19:44:29]

rbyers@chromium.org changed reviewers:
+ tkent@chromium.org

[Rick Byers, 2016-06-23 19:44:39]

tkent@ PTAL

[Rick Byers, 2016-06-23 20:35:36]

On 2016/06/23 19:44:39, Rick Byers wrote:
> tkent@ PTAL

Note that I'll actually get these previously disabled tests running (or disabled
for other reasons) as part of my import in
https://codereview.chromium.org/2086283003/

[Rick Byers, 2016-06-23 20:37:29]

On 2016/06/23 20:35:36, Rick Byers wrote:
> On 2016/06/23 19:44:39, Rick Byers wrote:
> > tkent@ PTAL
> 
> Note that I'll actually get these previously disabled tests running (or
disabled
> for other reasons) as part of my import in
> https://codereview.chromium.org/2086283003/

Actually it's too hard to combine a partially complete import with another
import.  I'll do a follow-up 2nd import in another CL.

[tkent, 2016-06-23 23:01:53]

Now the function is called only if we are running WPT tests. So this change
should be safe.  LGTM.
I prefer updating W3CImportExpectations when we import next time.

[Rick Byers, 2016-06-29 15:51:14]

On 2016/06/23 23:01:53, tkent wrote:
> Now the function is called only if we are running WPT tests. So this change
> should be safe.  LGTM.
> I prefer updating W3CImportExpectations when we import next time.

Ok, I'll do that in a follow-on import.  Thanks.

+avi@ for content/OWNERS

(I wonder if we shouldn't have a layout_test/OWNERS?).

[Rick Byers, 2016-06-29 15:51:27]

rbyers@chromium.org changed reviewers:
+ avi@chromium.org

[Rick Byers, 2016-06-29 15:51:39]

On 2016/06/29 15:51:14, Rick Byers wrote:
> On 2016/06/23 23:01:53, tkent wrote:
> > Now the function is called only if we are running WPT tests. So this change
> > should be safe.  LGTM.
> > I prefer updating W3CImportExpectations when we import next time.
> 
> Ok, I'll do that in a follow-on import.  Thanks.
> 
> +avi@ for content/OWNERS

+avi@ for real this time.

> 
> (I wonder if we shouldn't have a layout_test/OWNERS?).

[Avi (use Gerrit), 2016-06-29 16:15:19]

lgtm

[Rick Byers, 2016-06-29 16:19:47]

The CQ bit was checked by rbyers@chromium.org

[Rick Byers, 2016-06-29 16:19:47]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2090033003/#ps40001
(title: "Undo W3CTestExpectations")

[commit-bot: I haz the power, 2016-06-29 16:20:19]

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, 2016-06-29 17:45:59]

Description was changed from

==========
Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=498120
==========

to

==========
Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=498120
==========

[commit-bot: I haz the power, 2016-06-29 17:46:00]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-29 17:46:37]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-29 17:47:49]

Description was changed from

==========
Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=498120
==========

to

==========
Remove special cases for WPT path rewriting

Now that we rewrite absolute paths only when running WPT tests, there's no
longer any need to try to detect the names of the paths to rewrite.  Instead
just rewrite all absolute paths (that aren't already inside LayoutTests).

BUG=498120

Committed: https://crrev.com/74c476b03793ce0ba9bfda73d96a8142792f07b4
Cr-Commit-Position: refs/heads/master@{#402858}
==========

[commit-bot: I haz the power, 2016-06-29 17:47:50]

Patchset 3 (id:??) landed as
https://crrev.com/74c476b03793ce0ba9bfda73d96a8142792f07b4
Cr-Commit-Position: refs/heads/master@{#402858}