There's a lot of seemingly unnecessary complexity surrounding SurfaceFactory.

In particular, a lot of places create SurfaceFactory lazily on SubmitCompositorFrame such as DelegatedFrameHost, RenderWidgetHostViewGuest, RenderWidgetHostViewChildFrame, and destroy them if a context is lost. This seems unnecessarily complex.

SurfaceFactory can simply call RegisterSurfaceFactoryClient/UnregisterSurfaceFactoryClient internally, eliminating the need for public exposure of these methods on SurfaceManager.