Closed
Bug 500910
Opened 15 years ago
Closed 15 years ago
GTK test plugin should take XShapes into account when computing clip region
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: roc, Assigned: roc)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
When computing the plugin's effective clip region, the test plugin should consider any shape regions set on ancestor windows. We need this so that when we use shapes to clip plugins instead of ancestor widgets, tests still pass.
Assignee | ||
Updated•15 years ago
|
Assignee | ||
Updated•15 years ago
|
Attachment #385562 -
Flags: review?(mozbugz)
Updated•15 years ago
|
Attachment #385562 -
Flags: review?(mozbugz) → review+
Comment 1•15 years ago
|
||
Comment on attachment 385562 [details] [diff] [review]
fix
-EXTRA_DSO_LDOPTS += $(MOZ_GTK2_LIBS) $(XLDFLAGS) $(XLIBS)
+EXTRA_DSO_LDOPTS += $(MOZ_GTK2_LIBS) $(XLDFLAGS) $(XLIBS) -lXext
$(XEXT_LIBS) is available for this.
Assignee | ||
Comment 2•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•15 years ago
|
||
I backed this out due to test failures.
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1247207098.1247211633.9357.gz#err0
Assignee | ||
Updated•15 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 4•15 years ago
|
||
There was a critical missing gdk_intersect_region(region, windowRgn). I'm not sure how that happened, but the tests pass with this version of the patch.
Attachment #388055 -
Flags: review+
Assignee | ||
Updated•15 years ago
|
Whiteboard: [needs landing]
Assignee | ||
Updated•15 years ago
|
Attachment #385562 -
Attachment is obsolete: true
Assignee | ||
Comment 5•15 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Whiteboard: [needs landing]
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•