Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 769514 - Kill the Mac bitmap fallback code in nsSVGOuterSVGFrame
: Kill the Mac bitmap fallback code in nsSVGOuterSVGFrame
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: All Mac OS X
: -- normal (vote)
: mozilla16
Assigned To: Jonathan Watt [:jwatt]
: Jet Villegas (:jet)
Depends on: 772313 732819
Blocks: 614732
  Show dependency treegraph
Reported: 2012-06-28 18:24 PDT by Jonathan Watt [:jwatt]
Modified: 2012-07-09 18:29 PDT (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (5.35 KB, patch)
2012-06-28 18:27 PDT, Jonathan Watt [:jwatt]
roc: review+
Details | Diff | Splinter Review

Description Jonathan Watt [:jwatt] 2012-06-28 18:24:50 PDT
Roc tells me that with Layers we shouldn't be drawing direct to the window, so it seems like the bitmap fallback code in nsSVGOuterSVGFrame is useless and should just be removed.
Comment 1 Jonathan Watt [:jwatt] 2012-06-28 18:27:09 PDT
Created attachment 637750 [details] [diff] [review]
Comment 2 Jonathan Watt [:jwatt] 2012-06-28 18:38:26 PDT
Err, ignore the nsSVGGlyphFrame.cpp change.
Comment 3 Jonathan Watt [:jwatt] 2012-06-28 18:57:45 PDT
Just for record keeping, this was previously tried in bug 411334.
Comment 4 Robert O'Callahan (:roc) (email my personal email if necessary) 2012-06-28 19:15:47 PDT
Comment on attachment 637750 [details] [diff] [review]

Review of attachment 637750 [details] [diff] [review]:

R+ with that

::: layout/svg/base/src/nsSVGOuterSVGFrame.cpp
@@ +492,5 @@
>  #endif
>    aContext->PushState();
> +  frame->Paint(aBuilder, aContext, mVisibleRect, ToReferenceFrame());
> +  aContext->PopState();

we shouldn't need this PushState/PopState pair. Try removing it.
Comment 6 Ryan VanderMeulen [:RyanVM] 2012-07-05 17:19:39 PDT

Note You need to log in before you can comment on or make changes to this bug.