Closed Bug 811850 Opened 8 years ago Closed 8 years ago

Fix Azure recording to work with 64-bit systems

Categories

(Core :: Graphics, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: bas.schouten, Unassigned)

Details

Attachments

(1 file)

We want Azure recordings to be completely independent of 64-bit/32-bit.
Attachment #681635 - Flags: review?(jmuizelaar)
Comment on attachment 681635 [details] [diff] [review]
Fix several 64-bit issues for Azure recording

Review of attachment 681635 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/2d/RecordedEvent.cpp
@@ +877,5 @@
>  {
>    RefPtr<PathBuilder> builder = 
>      aTranslator->GetReferenceDrawTarget()->CreatePathBuilder(mFillRule);
>  
> +  for (uint64_t i = 0; i < mPathOps.size(); i++) {

This should remain size_t
Attachment #681635 - Flags: review?(jmuizelaar) → review+
https://hg.mozilla.org/mozilla-central/rev/4789d67c6017
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.