The default bug view has changed. See this FAQ.

Taskbar Tab preview in Windows 7's (Aero enabled) do not show up in latest Nightlies

VERIFIED FIXED in mozilla7

Status

()

Core
Graphics
--
major
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Optimizer, Assigned: bas)

Tracking

Trunk
mozilla7
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110626 Firefox/7.0a1
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110626 Firefox/7.0a1

Since the Azure build Nightly, The Tab preview have stopped working. I only get spinning wheel (with grey background) in the rectangle area meant for tab preview.

Reproducible: Always

Steps to Reproduce:
1.Have Windows 7 aero enabled and latest Nightly
2.Hover over Nightly icon in Taskbar till you get tab preview displayed


Actual Results:  
N preview is shown in any individual tab area

Expected Results:  
Either current tab's preview or all the tab's preview should be visible (depending on the usr;s settings)
(Reporter)

Updated

6 years ago
Version: unspecified → Trunk
(Assignee)

Comment 1

6 years ago
Tab previews are not enabled by default are they?
Assignee: nobody → bas.schouten
(Reporter)

Comment 2

6 years ago
No . 
Edit : This is only reproducible when we have the setting "Show tab preview in windows taskbar" switched on

Comment 3

6 years ago
Confirmed under condition of browser.taskbar.previews.enable = true,

Regression window(m-c)
Works:
http://hg.mozilla.org/mozilla-central/rev/450e4d9ea2d5
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110624 Firefox/7.0a1 ID:20110625025936
Fails:
http://hg.mozilla.org/mozilla-central/rev/48ad4ffc4230
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110625 Firefox/7.0a1 ID:20110625030821
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=450e4d9ea2d5&tochange=48ad4ffc4230


Regression window(m-i)
Works:
http://hg.mozilla.org/integration/mozilla-inbound/rev/c65f1fb0449d
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110624 Firefox/7.0a1 ID:20110624094307
Fails:
http://hg.mozilla.org/integration/mozilla-inbound/rev/46c3d095bc75
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110624 Firefox/7.0a1 ID:20110624104626
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=c65f1fb0449d&tochange=46c3d095bc75
Suspected bug: Bug 651858

Comment 4

6 years ago
FYI: When I disabled the pref gfx.canvas.azure.enabled, my taskbar tab previews displayed as expected.

Also, the pref browser.taskbar.previews.enable is disabled by default (I have enabled it), though, my experience has been only on very rare occasions that tabs previews do not display as expected. Restarting the browser corrects it.

Please let me know what, if any, additional information may be needed for triaging this. Thanks.
(Reporter)

Comment 5

6 years ago
Please someone confirm this bug and change its status from unconfirmend to something else like addigned ?
(Assignee)

Comment 6

6 years ago
Created attachment 542488 [details] [diff] [review]
Allow Taskbar Previews to explicitly create a thebes canvas

This is a shot at a patch for this. The problem is InitializeWithSurface doesn't work on an Azure canvas. I added a CID for a thebes canvas specifically so taskbar previews can use that.
Attachment #542488 - Flags: feedback?(roc)
Comment on attachment 542488 [details] [diff] [review]
Allow Taskbar Previews to explicitly create a thebes canvas

Review of attachment 542488 [details] [diff] [review]:
-----------------------------------------------------------------

This looks fine, but how are we going to solve this long term? Can we implement InitWithSurface?
Attachment #542488 - Flags: feedback?(roc) → feedback+
(Assignee)

Comment 8

6 years ago
(In reply to comment #7)
> Comment on attachment 542488 [details] [diff] [review] [review]
> Allow Taskbar Previews to explicitly create a thebes canvas
> 
> Review of attachment 542488 [details] [diff] [review] [review]:
> -----------------------------------------------------------------
> 
> This looks fine, but how are we going to solve this long term? Can we
> implement InitWithSurface?

Once we have a cairo Azure backend, I'd say yes. Right now, no, there's simply no way to know how we can draw to whatever is passed in (if it's not a D2D surface) without doing a load of copying, readback and copying back. (i.e., create D2D Azure, draw to it, do a CreateSurfaceFromDrawTarget, and then draw back to the surface in this case, where it's a windows surface). Since we really need to draw to a windows surface for the taskbar previews this seems like the best solution for now.
(Assignee)

Comment 9

6 years ago
Comment on attachment 542488 [details] [diff] [review]
Allow Taskbar Previews to explicitly create a thebes canvas

Requesting review then.
Attachment #542488 - Flags: review?(roc)
Comment on attachment 542488 [details] [diff] [review]
Allow Taskbar Previews to explicitly create a thebes canvas

Review of attachment 542488 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #542488 - Flags: review?(roc) → review+
(Assignee)

Comment 11

6 years ago
Landed on mozilla-inbound:

http://hg.mozilla.org/integration/mozilla-inbound/rev/4eb4d1b81e9b
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
This makes us support canvas.getContext("2dthebes") for content scripts. That's not acceptable.
We could call it moz-thebes and register it on the wiki page.

Or we could just reject it inside ::GetContext().
http://hg.mozilla.org/mozilla-central/rev/4eb4d1b81e9b
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7

Comment 15

6 years ago
I am seeing the tab previews as expected.

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110701 Firefox/7.0a1 ID:20110701030757

C-set: beceddeee3f4

Thanks.
Landed
http://hg.mozilla.org/mozilla-central/rev/a0b12f9bb58d
to address my comments in time for Firefox 7, with pending-r=roc based on your comments.
That looks great, thanks.
Build identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0) Gecko/20100101 Firefox/7.0

Unable to reproduce on latest 7(beta 4) build. Taskbar preview is displayed correctly.
(Reporter)

Comment 19

6 years ago
Isn't this bug Resolved Fixed ?
Please do not comment more.
Changing status to VERIFIED-FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.