Last Comment Bug 719136 - [Azure] Scrolling Wolfram Alpha causes crash
: [Azure] Scrolling Wolfram Alpha causes crash
Status: RESOLVED FIXED
: crash
Product: Core
Classification: Components
Component: Graphics (show other bugs)
: Trunk
: x86_64 Windows 7
: -- critical (vote)
: mozilla12
Assigned To: Bas Schouten (:bas.schouten)
:
: Milan Sreckovic [:milan]
Mentors:
Depends on:
Blocks: 715768
  Show dependency treegraph
 
Reported: 2012-01-18 10:11 PST by Leman Bennett [Omega]
Modified: 2012-05-20 13:41 PDT (History)
7 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Don't attempt to upload when there's nothing to upload (1.07 KB, patch)
2012-01-18 11:05 PST, Bas Schouten (:bas.schouten)
jmuizelaar: review+
Details | Diff | Splinter Review

Description Leman Bennett [Omega] 2012-01-18 10:11:33 PST
With Azure-Thebes wrapper enabled: Visit Wolfram Alpha and input search terms. Any will do. Then, scroll the page.

For example: http://www.wolframalpha.com/input/?i=x^2%2By^2%3D1

I used autoscroll, but should be reproducable without autoscroll.

Breakpad IDs:
https://crash-stats.mozilla.com/report/index/bp-1216c1c5-048e-4688-86d7-cf65f2120118
https://crash-stats.mozilla.com/report/index/bp-a88e1f30-ee32-4f49-a7fd-240ba2120118
https://crash-stats.mozilla.com/report/index/bp-1611c9de-3d6e-4960-8b01-fdd042120118
Comment 1 Bas Schouten (:bas.schouten) 2012-01-18 10:52:07 PST
This is what I was able to reproduce, it has a more useful stack:

https://crash-stats.mozilla.com/report/index/bp-9986771d-34ea-4532-9bf6-af9ef2120118
Comment 2 Bas Schouten (:bas.schouten) 2012-01-18 11:05:33 PST
Created attachment 589578 [details] [diff] [review]
Don't attempt to upload when there's nothing to upload

This brings to attention the fact we actually need another bug for getting the right upload region for an edge/corner for the extend modes case. For now I think we should get this fix in, we don't generally ever need the other functionality for content drawing, just canvas.
Comment 3 Jeff Muizelaar [:jrmuizel] 2012-01-18 11:09:43 PST
Test case?
Comment 4 Bas Schouten (:bas.schouten) 2012-01-18 11:40:58 PST
(In reply to Jeff Muizelaar [:jrmuizel] from comment #3)
> Test case?

I'm pretty sure this is some scrolling based edge-case. Normally we don't try to draw when something is outside the surface. I had to scroll a bunch to get this to occur. Maybe there's a way to reproduce it in a simple environment, but I don't know what it is.
Comment 5 Jeff Muizelaar [:jrmuizel] 2012-01-19 07:50:33 PST
Would this be easier to test if we had an azure test suite?
Comment 6 Bas Schouten (:bas.schouten) 2012-01-19 08:56:45 PST
Yes it would.
Comment 7 Theliel 2012-01-24 16:58:53 PST
Similar Crash (with nVidia in my case: nvwgf2um.dll@0x281db0)scrolling Wolfram and this one:

http://www.overclock.net/t/149879/howto-nvidia-bios-flashing/490


Azure-Thebes wrapper enabled, (Crash only with Direct3D 10 layers, Direct3D 9 layers dont crash)

https://crash-stats.mozilla.com/report/index/bp-9511e8be-8d80-4ae3-834e-9e9ab2120124
https://crash-stats.mozilla.com/report/index/bp-9ba17c6a-3757-4546-8dea-f740d2120125
Comment 8 Jeff Muizelaar [:jrmuizel] 2012-01-25 13:05:01 PST
(In reply to Bas Schouten (:bas) from comment #6)
> Yes it would.

Would you be willing to write the Azure test even if we don't have a framework for running it. I'd like us to start having an Azure test suite for testing the lower level semantics of the backends and for being able to run tests without having to rebuild the browser.
Comment 9 Bas Schouten (:bas.schouten) 2012-01-30 21:12:51 PST
(In reply to Jeff Muizelaar [:jrmuizel] from comment #8)
> (In reply to Bas Schouten (:bas) from comment #6)
> > Yes it would.
> 
> Would you be willing to write the Azure test even if we don't have a
> framework for running it. I'd like us to start having an Azure test suite
> for testing the lower level semantics of the backends and for being able to
> run tests without having to rebuild the browser.

I want to do that, I actually want to do the framework very, very soon. I'll make this the first test in there.
Comment 10 Bas Schouten (:bas.schouten) 2012-01-30 21:17:57 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/c645bbfc7475
Comment 11 Ed Morley [:emorley] 2012-01-31 06:49:35 PST
https://hg.mozilla.org/mozilla-central/rev/c645bbfc7475

Note You need to log in before you can comment on or make changes to this bug.