Closed Bug 612392 Opened 9 years ago Closed 9 years ago

Starting to type a url causes hangs

Categories

(Core :: Graphics, defect)

x86
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Tracking Status
fennec 2.0b4+ ---

People

(Reporter: jrmuizel, Assigned: mwu)

References

Details

Attachments

(1 file, 2 obsolete files)

Clicking in the awesome bar causes the browser to hang with opengl layers
Normally our draw logs:

I/Gecko   ( 9235): >> OnDraw
I/Gecko   ( 9235): nsWindow[0x506ab080]::DrawTo no covering child, drawing this

Before the hang we get:

I/Gecko   ( 9235): >> OnDraw

Without the DrawTo
Attached patch Workaround (obsolete) — Splinter Review
Assignee: nobody → mwu
Are both hunks of the patch needed? I applied this hunk:

             if (!doSyncDraw) {
+                if (!m2DMode)
+                    return;
+


and it didn't seem to fix the problem.
tracking-fennec: --- → ?
(In reply to comment #3)
> Are both hunks of the patch needed? I applied this hunk:
> 
>              if (!doSyncDraw) {
> +                if (!m2DMode)
> +                    return;
> +
> 
> 
> and it didn't seem to fix the problem.

Just the first hunk. It worked for me on my N1.
Attached patch Avoid 2D draws when doing 3D (obsolete) — Splinter Review
This lets us keep track of what drawing mode we're in and disable 2D drawing stuff while doing gl drawing.
Attachment #491019 - Attachment is obsolete: true
Attachment #491980 - Flags: review?(vladimir)
Clearing blocking since this blocks a blocking bug, so we already have blocking.
tracking-fennec: ? → ---
I'm still seeing some problems on my Galaxy S. I'll see if I can figure out what is going on.
tracking-fennec: --- → 2.0b4+
(In reply to comment #7)
> I'm still seeing some problems on my Galaxy S. I'll see if I can figure out
> what is going on.

This patch WFM on the galaxy S.
Rebased and ready to land.
Attachment #491980 - Attachment is obsolete: true
http://hg.mozilla.org/mozilla-central/rev/318d60678767
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.