Closed
Bug 1258758
Opened 8 years ago
Closed 8 years ago
Rename GetEffectiveClipRect to GetLocalClipRect
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
RESOLVED
FIXED
mozilla49
People
(Reporter: kats, Assigned: ronoueb)
References
Details
(Keywords: arch, Whiteboard: [gfx-noted])
Attachments
(1 file, 1 obsolete file)
19.20 KB,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
GetEffectiveClipRect returns either the shadow clip rect (for shadow layers) or the regular clip rect (for regular layers). As per the naming convention in bug 1247442, we should rename this to GetLocalClipRect.
Attachment #8747725 -
Flags: review?(bugmail.mozilla)
Reporter | ||
Comment 2•8 years ago
|
||
Comment on attachment 8747725 [details] [diff] [review] rename GetEffectiveClipRect Review of attachment 8747725 [details] [diff] [review]: ----------------------------------------------------------------- Nice, thanks! It looks like there's also a couple of references to GetEffectiveClipRect in widget/gonk/HwcComposer2D.cpp - could you update those as well? That code is only built for Firefox OS which is not a tier-1 platform so technically we don't need to make the change but it's simple enough and it would be nice to not break their build. The other change I'd like you to make is in the commit message - r=kats instead of r=kats@mozilla.com. With those two changes this patch should be ready to land; if you upload a new version I can land it for you.
Attachment #8747725 -
Flags: review?(bugmail.mozilla) → feedback+
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → bd339
Attachment #8747725 -
Attachment is obsolete: true
Attachment #8747742 -
Flags: review?(bugmail.mozilla)
Reporter | ||
Comment 4•8 years ago
|
||
Comment on attachment 8747742 [details] [diff] [review] rename GetEffectiveClipRect 2nd revision Review of attachment 8747742 [details] [diff] [review]: ----------------------------------------------------------------- Perfect, thanks! I'll land this shortly.
Attachment #8747742 -
Flags: review?(bugmail.mozilla) → review+
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a411d67a6e47
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•