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

Reply via email to