1372794 - Remove the expensive do_GetService() call from nsFormFillController::StopControllingInput()

Status: RESOLVED FIXED

(Toolkit :: Form Manager, enhancement)

Description

[(no longer active)]

This shows up under the react-redux Speedometer test.  The form fill controller gets called from <https://searchfox.org/mozilla-central/rev/d67ef71097da4d1aa344c9d9c672e49a7228e765/toolkit/components/satchel/nsFormFillController.cpp#907> and then we get to this do_GetService() call which shows up in the profile: <https://searchfox.org/mozilla-central/rev/d67ef71097da4d1aa344c9d9c672e49a7228e765/toolkit/components/satchel/nsFormFillController.cpp#1369>

I couldn't measure up a total speedup in points in the total Speedometer benchmark with this fixed, but it's still worth fixing.

Comment 1

[(no longer active)]

Attachment: Cache the form autocomplete service instead of repeatedly getting it in the form fill controller

Comment 2

[Matthew N. [:MattN]]

Comment on attachment 8877411 [details] [diff] [review]
Cache the form autocomplete service instead of repeatedly getting it in the form fill controller

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

Thanks

Comment 3

[Matthew N. [:MattN]]

Comment on attachment 8877411 [details] [diff] [review]
Cache the form autocomplete service instead of repeatedly getting it in the form fill controller

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

::: toolkit/components/satchel/nsFormFillController.cpp
@@ +10,2 @@
>  #include "mozilla/ErrorResult.h"
> +#include "mozilla/Unused.h"

Though is Unused.h used in this patch? I'm not seeing it used but I'm also not an expert in our C++.

Comment 4

[(no longer active)]

(In reply to Matthew N. [:MattN] (PM if requests are blocking you) from comment #3)
> ::: toolkit/components/satchel/nsFormFillController.cpp
> @@ +10,2 @@
> >  #include "mozilla/ErrorResult.h"
> > +#include "mozilla/Unused.h"
> 
> Though is Unused.h used in this patch? I'm not seeing it used but I'm also
> not an expert in our C++.

No you're right, I used it in a previous version and then removed the usage and forgot to remove the #include!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b084196d6c8
Cache the form autocomplete service instead of repeatedly getting it in the form fill controller; r=MattN

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/1b084196d6c8