822018 - Remove the global PB specific code in tps.jsm from per-window PB builds

Status: RESOLVED FIXED

(Testing Graveyard :: TPS, defect)

Description

[(no longer active)]

We need to make the global PB specific code in http://mxr.mozilla.org/mozilla-central/source/services/sync/tps/extensions/tps/modules/tps.jsm hidden behind #ifndef MOZ_PER_WINDOW_PRIVATE_BROWSING.  One such code is the SetPrivateBrowsing function.  There are three tests which call that function and they should not be run in such builds, but the problem is that they don't use the usual Makefile.in files, and instead they use all_tests.json.  Therefore, we need to also be able to preprocess all_tests.json somehow too, but I'm not sure how to do that.

Richard, do you have any idea?

Thanks!

Comment 1

[Richard Newman [:rnewman]]

TPS doesn't run on buildbot (yet?), so nothing will go red until services-central has MPWPB set by default. CCing jgriffin, who keeps TPS fed and watered.

Comment 2

[Jonathan Griffin (:jgriffin)]

TPS runs tests that are bundled into a tests.zip package:

http://mxr.mozilla.org/mozilla-central/source/testing/testsuite-targets.mk#416

We could add some pre-processing here, and add some logic to only copy files that end up in the post-processed version, instead of everything as we do now.

Comment 3

[(no longer active)]

Given comment 1, I'm unblocking this for per-window private browsing, but will leave the bug open since this needs to get fixed when we want to run TPS tests on Tinderbox.

Comment 4

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/418277a17cb2 took care of some of this, but that file is still listening for the private-browsing notification, which will never be dispatched in per-window PB builds.

Comment 5

[(no longer active)]

(In reply to :Ehsan Akhgari from comment #4)
> https://hg.mozilla.org/mozilla-central/rev/418277a17cb2 took care of some of
> this, but that file is still listening for the private-browsing
> notification, which will never be dispatched in per-window PB builds.

That said, there's more mentions of "private-browsing" left in that file...

Comment 6

[(no longer active)]

Josh, I would _really_ like to land this before bug 784841 lands.  gps tells me that will happen next week, if not later this week.

Comment 7

[(no longer active)]

(In reply to :Ehsan Akhgari from comment #6)
> Josh, I would _really_ like to land this before bug 784841 lands.  gps tells
> me that will happen next week, if not later this week.

Wrong bug!

Comment 8

[Lina Butler (ex-Mozilla)]

Attachment: Bug 822018 - Remove unused mentions of private browsing from TPS.

Review commit: https://reviewboard.mozilla.org/r/241034/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/241034/

Comment 9

[Thom Chiovoloni [:tcsc] (ex-moco)]

Comment on attachment 8972435 [details]
Bug 822018 - Remove unused mentions of private browsing from TPS.

https://reviewboard.mozilla.org/r/241034/#review246816

LGTM

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fa4f44996ac6
Remove unused mentions of private browsing from TPS. r=tcsc

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/fa4f44996ac6