869143 - XPCSHELL_TEST conversion cleanups

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Joey Armstrong [:joey]]

Remove DISABLED_XPCSHELL_TEST from Makefile.in.
cleanup remaining conversion items {list to be added}

Comment 1

[Joey Armstrong [:joey]]

Attachment: phase2 cleanup for XPCSHELL_TEST conversion.

Comment 2

[Joey Armstrong [:joey]]

Comment on attachment 747999 [details] [diff] [review]
phase2 cleanup for XPCSHELL_TEST conversion.

Phase2 conversion cleanup.  Remove scaffolding (s/XPCSHELL_/DISABLED_XPCSHELL_) and conditionals from the Makefile.in flavor of XPCSHELL_TESTS.

Comment 3

[Michael Shal [:mshal]]

Comment on attachment 747999 [details] [diff] [review]
phase2 cleanup for XPCSHELL_TEST conversion.

>diff --git a/toolkit/components/downloads/test/Makefile.in b/toolkit/components/downloads/test/Makefile.in
>--- a/toolkit/components/downloads/test/Makefile.in
>+++ b/toolkit/components/downloads/test/Makefile.in
>@@ -1,20 +1,15 @@
> #
> # This Source Code Form is subject to the terms of the Mozilla Public
> # License, v. 2.0. If a copy of the MPL was not distributed with this
> # file, You can obtain one at http://mozilla.org/MPL/2.0/.
> 
>-DEPTH   = @DEPTH@
>+DEPTH     = @DEPTH@
> topsrcdir = @top_srcdir@
> srcdir    = @srcdir@
>-VPATH   = @srcdir@
>+VPATH     = @srcdir@
> relativesrcdir = @relativesrcdir@

Please avoid making unrelated whitespace changes to patches like this, since it makes review a bit more difficult. Other than that, looks good.

Comment 4

[Joey Armstrong [:joey]]

Try job: https://tbpl.mozilla.org/?tree=Try&rev=98bddfc4855f

Comment 5

[Joey Armstrong [:joey]]

inbound push:
committed changeset 131815:2bf6dcf8c74a
https://hg.mozilla.org/integration/mozilla-inbound/rev/2bf6dcf8c74a

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/2bf6dcf8c74a