Closed
Bug 589613
Opened 14 years ago
Closed 14 years ago
[SeaMonkey 2.1, mochitest-browser-chrome] browser_Services.js | Services.search exists
Categories
(Core :: General, defect)
Core
General
Tracking
()
VERIFIED
FIXED
mozilla2.0b7
People
(Reporter: kairo, Assigned: iannbugzilla)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure, Whiteboard: [sm-perma])
Attachments
(1 file, 2 obsolete files)
1.73 KB,
patch
|
Gavin
:
review+
Callek
:
review+
|
Details | Diff | Splinter Review |
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1282499502.1282502435.6363.gz TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/content/tests/browser/browser_Services.js | Services.search exists TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/content/tests/browser/browser_Services.js | Services.search is an nsIBrowserSearchService The test for Services.search should only be executed when OpenSearch is built.
![]() |
Reporter | |
Updated•14 years ago
|
Blocks: SmTestFail
Comment 1•14 years ago
|
||
Unexpected, bug 553815 was supposed to fix (all the issue of) this test...
Depends on: 553815
![]() |
Reporter | |
Comment 2•14 years ago
|
||
My guess is that the test for the interface in the test now always succeeds as the components are registered differently.
Comment 3•14 years ago
|
||
> My guess is that the test for the interface in the test now always succeeds as > the components are registered differently. yea, that would be the case... I'll have to dive back into this(In reply to comment #2)
Comment 4•14 years ago
|
||
Fwiw, http://brasstacks.mozilla.com/topfails/test/SeaMonkey?name=chrome://mochikit/content/browser/toolkit/content/tests/browser/browser_Services.js "2010-05-27 13:34": when this test was fixed previously. "2010-08-14 10:19": when it started failing again.
The build based on http://hg.mozilla.org/mozilla-central/rev/7b9c03096aa1 and http://hg.mozilla.org/comm-central/rev/6e5632e4d2f0 fails The build based on http://hg.mozilla.org/mozilla-central/rev/67eac5028fd4 and http://hg.mozilla.org/comm-central/rev/e05dc1486d50 does not fail Which gives http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=67eac5028fd4&tochange=7b9c03096aa1 and http://hg.mozilla.org/comm-central/pushloghtml?fromchange=e05dc1486d50&tochange=6e5632e4d2f0 I'm guessing that it will be a m-c change that will have caused the issue, I guess it is http://hg.mozilla.org/mozilla-central/rev/5761fb5b379f is the culprit so cc'ing gavin.
I'm guessing some of the new code should have some ifdefs round it?
![]() |
Reporter | |
Comment 7•14 years ago
|
||
Hah, well, he made the IDL to be built even when the toolkit search service is not built - there you have it.
Patch to ifdef out toolkit search code if toolkit search is not enabled
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #472040 -
Flags: review?(gavin.sharp)
Comment 9•14 years ago
|
||
Comment on attachment 472040 [details] [diff] [review] ifdef bits out patch v0.1 >diff --git a/netwerk/base/public/Makefile.in b/netwerk/base/public/Makefile.in >- nsIRedirectResultListener.idl \ >- nsIBrowserSearchService.idl \ >- $(NULL) >+ nsIRedirectResultListener.idl >+ >+ifdef MOZ_TOOLKIT_SEARCH >+XPIDLSRCS += nsIBrowserSearchService.idl >+endif >+ >+XPIDLSRCS += $(NULL) This doens't look right - AFAIK $(NULL)'s only purpose is to serve as a dummy end marker to ensure that all entries end with "\", so that adding/removing entries doesn't require editing two lines. So you should just leave the $(NULL) at the end of the other block - as in e.g. http://mxr.mozilla.org/mozilla-central/source/embedding/browser/webBrowser/Makefile.in#75 . r=me with that fixed.
Attachment #472040 -
Flags: review?(gavin.sharp) → review+
Component: Testing Infrastructure → General
Product: SeaMonkey → Core
QA Contact: testing-infrastructure → general
Target Milestone: --- → mozilla2.0b6
Assignee | ||
Comment 10•14 years ago
|
||
Changes since v0.1: * Fixed Makefile.in as suggested by reviewer. Requesting a= for simple, low risk patch that fixes an issue with moved code (and helps remove an orange on SM tests)
Attachment #472040 -
Attachment is obsolete: true
Attachment #472053 -
Flags: review+
Attachment #472053 -
Flags: approval2.0?
Comment 11•14 years ago
|
||
Comment on attachment 472053 [details] [diff] [review] Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Backed out: Comment 14] Hrmn actually this should have implicit a+ as "NPOTB" (since Firefox builds with BrowserSearchService of course)
Updated•14 years ago
|
Attachment #472053 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 12•14 years ago
|
||
Comment on attachment 472053 [details] [diff] [review] Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Backed out: Comment 14] http://hg.mozilla.org/mozilla-central/rev/2a216165e361
Attachment #472053 -
Attachment description: Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a → Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Checked in: Comment 12]
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 13•14 years ago
|
||
V.Fixed, per http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284130726.1284133745.30739.gz Linux comm-central-trunk debug test mochitest-other on 2010/09/10 07:58:46
Severity: normal → major
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Comment 14•14 years ago
|
||
Backed out because this caused bug 595509 (MOZ_TOOLKIT_SEARCH isn't AC_DEFINED). https://hg.mozilla.org/mozilla-central/rev/84baf90b040c
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Comment 15•14 years ago
|
||
IanN, on Bug 595509 (attachment 474379 [details] [diff] [review]) I provided a fix. Wish it was taken instead of a backout though. But lets get this (back) in!
Comment 16•14 years ago
|
||
err home network fail :/ the fix was adding DEFINES += -DMOZ_TOOLKIT_SEARCH in the makefile at docshell/base/ conditional on ifdef MOZ_TOOLKIT_SEARCH I can attach the fix here later.
Attachment #472053 -
Attachment description: Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Checked in: Comment 12] → Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Checked in: Comment 12] [Backed out: Comment 14]
Assignee | ||
Comment 17•14 years ago
|
||
As MOZ_TOOLKIT_SEARCH is not AC_DEFINED need to add it to the Makefile.in Used test from bug 595509 to check without any patch applied (PASS), with old patch applied (FAIL), with this patch applied (PASS).
Attachment #474432 -
Flags: review?(gavin.sharp)
Comment 18•14 years ago
|
||
Comment on attachment 474432 [details] [diff] [review] Only build search code when MOZ_TOOLKIT_SEARCH is fully defined patch v0.2 [Checked in: Comment 20] This Makefile change is what my "empty patch" was intended to be in the other bug.
Attachment #474432 -
Flags: review+
Updated•14 years ago
|
Attachment #474432 -
Flags: review?(gavin.sharp) → review+
Attachment #474432 -
Flags: approval2.0?
![]() |
Reporter | |
Comment 19•14 years ago
|
||
The ideal fix would be to get in bug 410613 on the SeaMonkey side :P
Assignee | ||
Comment 20•14 years ago
|
||
Comment on attachment 474432 [details] [diff] [review] Only build search code when MOZ_TOOLKIT_SEARCH is fully defined patch v0.2 [Checked in: Comment 20] http://hg.mozilla.org/mozilla-central/rev/da8dd922ba0c a=NPOTB
Attachment #474432 -
Attachment description: Only build search code when MOZ_TOOLKIT_SEARCH is fully defined patch v0.2 → Only build search code when MOZ_TOOLKIT_SEARCH is fully defined patch v0.2 [Checked in: Comment 20]
Attachment #474432 -
Flags: approval2.0?
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Comment 21•14 years ago
|
||
V.Fixed, per http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284821028.1284823562.22253.gz OS X 10.5 comm-central-trunk debug test mochitest-other on 2010/09/18 07:43:48
Status: RESOLVED → VERIFIED
Updated•14 years ago
|
Attachment #472053 -
Attachment description: Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Checked in: Comment 12] [Backed out: Comment 14] → Only build search code when MOZ_TOOLKIT_SEARCH is enabled patch v0.1a [Backed out: Comment 14]
Attachment #472053 -
Attachment is obsolete: true
Updated•12 years ago
|
Keywords: intermittent-failure
Updated•12 years ago
|
Whiteboard: [sm-perma][orange] → [sm-perma]
You need to log in
before you can comment on or make changes to this bug.
Description
•