move GetActionContextAsync and its related stuff to abstract place

[infofromca]

Is your feature request related to a problem? Please describe.

under public class SeoMetaSettingsHandler and its related
private readonly IActionContextAccessor _actionContextAccessor;
private readonly IHttpContextAccessor _httpContextAccessor;
private readonly IUrlHelperFactory _urlHelperFactory;

Describe the solution you'd like

move
private readonly IActionContextAccessor _actionContextAccessor;
private readonly IHttpContextAccessor _httpContextAccessor;
private readonly IUrlHelperFactory _urlHelperFactory;
and
GetActionContextAsync
to a topService, because many modules will use the same logic.

[Piedone]

Moving injected dependency to a parent class is only marginally useful since you still have to write the constructor in a child class.

Factoring out this method can be useful:

OrchardCore/src/OrchardCore.Modules/OrchardCore.Seo/Handlers/SeoMetaSettingsHandler.cs

Lines 209 to 223 in 557d6cf

	internal static async Task<ActionContext> GetActionContextAsync(HttpContext httpContext)
	{
	var routeData = new RouteData();
	routeData.Routers.Add(new RouteCollection());
	var actionContext = new ActionContext(httpContext, routeData, new ActionDescriptor());
	var filters = httpContext.RequestServices.GetServices<IAsyncViewActionFilter>();
	foreach (var filter in filters)
	{
	await filter.OnActionExecutionAsync(actionContext);
	}
	return actionContext;
	}

Because it's duplicated here and here. These examples also show that these should be centralized in some kind of static helper.