Closed Bug 1478385 Opened 4 years ago Closed 4 years ago
Fix confussion in Get
Cached Widget Border
59 bytes, text/x-review-board-request
No description provided.
Comment on attachment 8994862 [details] Bug 1478385: Fix confusion in GetCachedWidgetBorder. https://reviewboard.mozilla.org/r/259404/#review266518 ::: commit-message-9abe7:5 (Diff revision 1) > +Bug 1478385: Fix confusion in GetCachedWidgetBorder. r?karlt > + > +aWidgetType is a NS_THEME_* constant. What this code is comparing is not. > + > +Is this comparison needed at all? Yes. I expect you are fixing a hard-to-reproduce bug for users with combinations of RTL and LTR pages.
Attachment #8994862 - Flags: review?(karlt) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/a7ad6a70635a Fix confusion in GetCachedWidgetBorder. r=karlt
You need to log in before you can comment on or make changes to this bug.