Closed
Bug 868181
Opened 12 years ago
Closed 12 years ago
ASSERTION: Invalid layer manager (LAYERS_CLIENT) when loading any page on Android
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
RESOLVED
FIXED
mozilla23
Tracking | Status | |
---|---|---|
firefox22 | --- | unaffected |
firefox23 | --- | fixed |
People
(Reporter: cpeterson, Assigned: mattwoodrow)
Details
(Keywords: regression)
Attachments
(1 file)
1.12 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
I believe this is a mozilla-central regression from 5/19 or 5/20.
Android's nsWindow::DrawTo() function expects LAYERS_BASIC or LAYERS_OPENGL, but gets LAYERS_CLIENT. So DrawTo() logs errors and then skips mWidgetListener->PaintWindow().
I/Gecko (16250): Invalid layer manager: BackendType=5
I/Gecko (16250): ###!!! ASSERTION: Invalid layer manager: 'Error', file /Users/cpeterson/Code/mozilla/central/widget/android/nsWindow.cpp, line 999
I/Gecko (16250): Invalid layer manager: BackendType=5
I/Gecko (16250): ###!!! ASSERTION: Invalid layer manager: 'Error', file /Users/cpeterson/Code/mozilla/central/widget/android/nsWindow.cpp, line 999
I/Gecko (16250): Invalid layer manager: BackendType=5
I/Gecko (16250): ###!!! ASSERTION: Invalid layer manager: 'Error', file /Users/cpeterson/Code/mozilla/central/widget/android/nsWindow.cpp, line 999
I/Gecko (16250): Invalid layer manager: BackendType=5
I/Gecko (16250): ###!!! ASSERTION: Invalid layer manager: 'Error', file /Users/cpeterson/Code/mozilla/central/widget/android/nsWindow.cpp, line 999
Reporter | ||
Updated•12 years ago
|
Summary: ASSERTION: Invalid layer manager: 'Error' when loading any page on Android → ASSERTION: Invalid layer manager (LAYERS_CLIENT) when loading any page on Android
Assignee | ||
Comment 2•12 years ago
|
||
This is a regression from my changes, I'll fix it.
Assignee: nobody → matt.woodrow
Flags: needinfo?(nical.bugzilla)
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #745739 -
Flags: review?(roc)
Assignee | ||
Comment 4•12 years ago
|
||
Do we actually need the code paths for painting in-process any more?
Afaik, we only needed those for xul-fennec which isn't supported any more.
If that's true, then we can just assert that we've got LAYERS_CLIENT, and remove the others, as well as the gfxContext* paramters (It looks like we only ever use a dummy surface anyway).
Attachment #745739 -
Flags: review?(roc) → review+
(In reply to Matt Woodrow (:mattwoodrow) from comment #4)
> Do we actually need the code paths for painting in-process any more?
>
> Afaik, we only needed those for xul-fennec which isn't supported any more.
Yeah, I think we don't.
Assignee | ||
Comment 6•12 years ago
|
||
Comment 7•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Updated•12 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•