396132 - Minotaur needs to check new Content Handling preferences for default handlers

Status: RESOLVED FIXED

(Testing Graveyard :: Minotaur, defect)

Description

[cmtalbert]

It is entirely feasible that future partner builds or L10N specific builds might use a different default than the "standard" default for the content/protocol handling pieces.  Minotaur should be updated to check these defaults and compare them during its verification process.

Comment 2

[cmtalbert]

Attachment: Patch r1

I can't find a good way to break that regexp into multiple lines so that it fits the 80 character limit.  If you know of one, I'm all ears.  I've tried everything I can think of except using two different regexp's and concatenating the results (which seems like way too much work to simply satisfy the line length requirement.)

This outputs the default settings for the protocol handlers and it also outputs the places smartBookmark version which seems to be desired from bug 421483 comment 42 (https://bugzilla.mozilla.org/show_bug.cgi?id=421483#c42)

Comment 3

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 314914 [details] [diff] [review]
Patch r1

I guess you could have multiple regexp objects. E.g., grabDist = /app\.distributor/;
grabPartner = /app\.partner/

and then match on each of them in the matching section below. That would be somewhat cumbersome though it would add to the understandability of each pref you're looking for. I dunno.

Comment 4

[Axel Hecht [:Pike]]

You could do a

var reStrings = ['(app\.distributor)',
                 '(app\.partner)',...];
var grab = new RegExp(reStrings.join('|'));

Comment 5

[cmtalbert]

I took Axel's suggestion and checked it in on trunk.
--> FIXED

Comment 6

[(not currently active) Ted Mielczarek]

Mass moving Minotaur bugs to Testing : Minotaur. Filter on MinotaurMassMove to ignore.