The default bug view has changed. See this FAQ.

[MacOSX] mochitest-chrome: the 3 layout/base/tests/test_printpreview*.xul need to report 'todo'

VERIFIED FIXED in mozilla1.9.3a5

Status

()

Core
Printing: Output
VERIFIED FIXED
8 years ago
7 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Depends on: 1 bug)

Trunk
mozilla1.9.3a5
All
Mac OS X
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(status1.9.2 .5-fixed, status1.9.1 .10-fixed)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
{
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242092605.1242096280.17513.gz&fulltext=1
OS X 10.5.2 mozilla-central unit test on 2009/05/11 18:43:25

*** 43086 INFO Running /tests/layout/base/tests/test_bug396024.html...
*** 43087 INFO TEST-KNOWN-FAIL | /tests/layout/base/tests/test_bug396024.html | [SimpleTest.report()] No checks actually run.
}

See
http://hg.mozilla.org/mozilla-central/rev/b3d077ffddbd
(Assignee)

Comment 1

8 years ago
Same with
{
*** 43515 INFO Running /tests/layout/base/tests/test_bug482976.html...
*** 43516 INFO TEST-KNOWN-FAIL | /tests/layout/base/tests/test_bug482976.html | [SimpleTest.report()] No checks actually run.
}
Depends on: 482976
Summary: [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html needs to report success → [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html and /test_bug482976.html need to report success
(Assignee)

Comment 2

8 years ago
Same with
{
*** 43526 INFO Running /tests/layout/base/tests/test_printpreview.html...
*** 43527 INFO TEST-KNOWN-FAIL | /tests/layout/base/tests/test_printpreview.html | [SimpleTest.report()] No checks actually run.
}
blamed to restricted access bugs: bug 425265, bug 424377.
Summary: [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html and /test_bug482976.html need to report success → [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html, /test_bug482976.html and test_printpreview.html need to report success
The Mac platform doesn't have the print preview code necessary for it to work. That might be a bug, I think I've seen a bug floating around about that. So I guess it might need to report todo perhaps.
(Assignee)

Comment 4

8 years ago
(In reply to comment #3)
> That might be a bug, I think I've seen a bug floating around about that.

Bug 347417 ;-)

> So I guess it might need to report todo perhaps.

Agreed.
Depends on: 347417
Hardware: x86 → All
Summary: [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html, /test_bug482976.html and test_printpreview.html need to report success → [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html, /test_bug482976.html and test_printpreview.html need to report 'todo'
(In reply to comment #4)
> (In reply to comment #3)
> > That might be a bug, I think I've seen a bug floating around about that.
> 
> Bug 347417 ;-)

I'm not sure it's that bug. That bug seems to talk about a print preview menu-item, it doesn't seem to explicitly mention about the desire to have webBrowserPrint.printPreview implemented on the Mac.
(Assignee)

Comment 6

8 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1242134886.1242140574.17608.gz&fulltext=1
OS X 10.5.2 mozilla-1.9.1 unit test on 2009/05/12 06:28:06
(Assignee)

Updated

8 years ago
No longer blocks: 483407
Depends on: 483407
(Assignee)

Updated

7 years ago
Summary: [MacOSX] mochitest-plain: layout/base/tests/test_bug396024.html, /test_bug482976.html and test_printpreview.html need to report 'todo' → [MacOSX] mochitest-chrome: layout/base/tests/test_bug396024.html, /test_bug482976.html and test_printpreview.html need to report 'todo'
(Assignee)

Updated

7 years ago
Summary: [MacOSX] mochitest-chrome: layout/base/tests/test_bug396024.html, /test_bug482976.html and test_printpreview.html need to report 'todo' → [MacOSX] mochitest-chrome: the 3 layout/base/tests/test_printpreview*.xul need to report 'todo'
(Assignee)

Comment 7

7 years ago
Created attachment 438666 [details] [diff] [review]
(Av1) Remove useless |var printerEnumerator|, Use a todo()
[Checkin: See comment 8 & 11]
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #438666 - Flags: review?(roc)
Attachment #438666 - Flags: review?(roc) → review+
(Assignee)

Comment 8

7 years ago
Comment on attachment 438666 [details] [diff] [review]
(Av1) Remove useless |var printerEnumerator|, Use a todo()
[Checkin: See comment 8 & 11]


http://hg.mozilla.org/mozilla-central/rev/058b1d064f88
Attachment #438666 - Attachment description: (Av1) Remove useless |var printerEnumerator|, Use a todo() → (Av1) Remove useless |var printerEnumerator|, Use a todo() [Checkin: Comment 8]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
(Assignee)

Comment 9

7 years ago
(In reply to comment #8)

+
http://hg.mozilla.org/mozilla-central/rev/22df6c8c783a
(Bv1) Add missed |var todo = window.opener.wrappedJSObject.todo;|.
(Assignee)

Updated

7 years ago
Blocks: 396024, 482976
No longer depends on: 396024, 482976
(Assignee)

Updated

7 years ago
Blocks: 556686
(Assignee)

Comment 10

7 years ago
V.Fixed per tinderboxes.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 11

7 years ago
Comment on attachment 438666 [details] [diff] [review]
(Av1) Remove useless |var printerEnumerator|, Use a todo()
[Checkin: See comment 8 & 11]



http://hg.mozilla.org/releases/mozilla-1.9.2/rev/9196d70fd5e5
including
(In reply to comment #9)
> http://hg.mozilla.org/mozilla-central/rev/22df6c8c783a
> (Bv1) Add missed |var todo = window.opener.wrappedJSObject.todo;|.
Attachment #438666 - Attachment description: (Av1) Remove useless |var printerEnumerator|, Use a todo() [Checkin: Comment 8] → (Av1) Remove useless |var printerEnumerator|, Use a todo() [Checkin: See comment 8 & 11]
(Assignee)

Updated

7 years ago
status1.9.2: --- → .5-fixed
(Assignee)

Comment 12

7 years ago
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/3eab3c816d4e
(Cv1-191) Remove useless |var printerEnumerator|, Use a todo().
status1.9.1: --- → .10-fixed
You need to log in before you can comment on or make changes to this bug.