Closed Bug 667298 Opened 11 years ago Closed 11 years ago

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

Categories

(Core :: Graphics, defect)

x86_64
Windows 7
defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla7

People

(Reporter: Optimizer, Assigned: bas.schouten)

Details

Attachments

(1 file)

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)
Version: unspecified → Trunk
Tab previews are not enabled by default are they?
Assignee: nobody → bas.schouten
No . 
Edit : This is only reproducible when we have the setting "Show tab preview in windows taskbar" switched on
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
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.
Please someone confirm this bug and change its status from unconfirmend to something else like addigned ?
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+
(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.
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+
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
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
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.
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.
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.