The default bug view has changed. See this FAQ.

[mochitest-browser-chrome] numerous browser_searching.js failures on SeaMonkey

RESOLVED FIXED in seamonkey2.1a3

Status

SeaMonkey
UI Design
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Robert Kaiser, Assigned: Robert Kaiser)

Tracking

(Blocks: 1 bug)

unspecified
seamonkey2.1a3
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
After bug 558287 landed browser_searching.js, it produces numerous failures on SeaMonkey. I suspect that adding the prefs it added in firefox.js on our side will fix those as well.
(Assignee)

Comment 1

7 years ago
Preliminary testing doesn't look that good with just those prefs added. Someone needs to take a deeper look into this, either me or preferably someone else.
Do you refer to the *test* file that was added? This bug report is quite vague, I thought it referred to an actual functionality breakage.
(Assignee)

Comment 3

7 years ago
yes, browser_searching.js is a test file and it produces about 80 or so mochitest-browser-chrome test failures.
(Assignee)

Updated

7 years ago
Blocks: 452942
(Assignee)

Updated

7 years ago
Summary: numerous browser_searching.js failures on SeaMonkey → [mochitest-browser-chrome] numerous browser_searching.js failures on SeaMonkey
(Assignee)

Comment 4

7 years ago
Created attachment 459523 [details] [diff] [review]
fix the prefs (not the test) [Checkin: Comment 6]

This patch just fixes the prefs. It doesn't really help the test, I guess it has clever design that hardwires them to some value anyhow and uses local mochi.test URLs instead of querying the real AMO service anyhow.
Still, we should set those in our build in any case, so I guess I should better attach them and get them reviewed, right? ;-)
Attachment #459523 - Flags: review?(bugspam.Callek)
Comment on attachment 459523 [details] [diff] [review]
fix the prefs (not the test) [Checkin: Comment 6]

seems to work:

https://services.addons.mozilla.org/en-US/firefox/api/1/search/chatzilla/all/2/windows/2.0
and
https://services.addons.mozilla.org/en-US/seamonkey/api/1/search/chatzilla/all/2/windows/2.0

return the same data... (granted I used a pretty contrived search term and an app version we won't see on our nightlies, but still.
Attachment #459523 - Flags: review?(bugspam.Callek) → review+

Comment 6

7 years ago
Comment on attachment 459523 [details] [diff] [review]
fix the prefs (not the test) [Checkin: Comment 6]

http://hg.mozilla.org/comm-central/rev/91d9bc2eeaa3
Lets see where this takes things to start with
Attachment #459523 - Attachment description: fix the prefs (not the test) → fix the prefs (not the test) [Checkin: Comment 6]
(Assignee)

Comment 7

7 years ago
Looks like bug 566593 actually fixed that, but I'll mark this as mine for the patch that was checked in here.
Assignee: nobody → kairo
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Depends on: 566593
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a3
You need to log in before you can comment on or make changes to this bug.