Comment on attachment 497112 Use EXTEND_PAD on X >- if (fastExtendPad) { >- aPattern->SetExtend(gfxPattern::EXTEND_PAD); >- aPattern->SetFilter(aDefaultFilter); >- } else >-#endif >- aPattern->SetFilter(gfxPattern::FILTER_FAST); >+ >+ if (!isDownscale && !fastExtendPad) { >+ aPattern->SetFilter(gfxPattern::FILTER_FAST); > } >+
Looks like an else block with SetFilter(aDefaultFilter) is needed. Without this patch, that currently only happens when !isDownscale, but I expect it should also be done when !isDownscale, as on other platforms. Otherwise, this approach of using EXTEND_PAD when supported by cairo seems sensible to me. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/263661 Title: Black Line appears next to gif and png images To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/263661/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs