881609 - Call gfxPlatform::InitLayersAccelerationPrefs eagerly from gfxPlatform::Init rather than lazily every time we check a pref

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Nick Cameron [:nrc]]

I'm not 100% sure we want to do this (could it affect startup time?), but given we will always check the layers prefs it seems like a win to only call the init method once rather than every time and returning every time but the first. I guess we should assert the prefs have actually been initialised in the methods, but that is debug only. (Note we already do this for sDrawLayerBorders).

Comment 1

[Milan Sreckovic [:milan] (needinfo for best results)]

You're right, it's safe, but suboptimal, especially if that function doesn't get inlined.  I left it that way because it wasn't clear to me gfxPlatform::Init() will always get called before we call these pref getting functions from nsBaseWidget.cpp.  I'm not saying it won't, just that I couldn't find the code that was making sure that happens.

Comment 2

[Milan Sreckovic [:milan] (needinfo for best results)]

Attachment: MozReview Request: Bug 881609: Call InitLayersAccelerationPrefs only once. r?nrc

Review commit: https://reviewboard.mozilla.org/r/36869/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/36869/

Comment 3

[Milan Sreckovic [:milan] (needinfo for best results)]

The code has changed enough that we can satisfy "don't call it each time" as well as "what if Init doesn't get called".  Or come close enough (two calls total, instead of one + per preference check)

Comment 4

[Nick Cameron [:nrc]]

Sorry it took me a while to get to this - I've been on vacation for the past three weeks.

Comment 5

[Nick Cameron [:nrc]]

Comment on attachment 8724110 [details]
MozReview Request: Bug 881609: Call InitLayersAccelerationPrefs only once. r?nrc

https://reviewboard.mozilla.org/r/36869/#review36313

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2ba62e2e02b0

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2ba62e2e02b0