Potential oversight/bug in FrontEnd:DoHoverFocusUpdate function

function FrontEnd:DoHoverFocusUpdate(manual_update)
    if self.tracking_mouse and not self.focus_locked then
		if manual_update then
			--something has been manually moved, so we want to update the entities under the mouse and re-evaluate the hover/focus state
			TheInput:UpdateEntitiesUnderMouse()
		end
	    local entitiesundermouse = TheInput:GetAllEntitiesUnderMouse() -- HERE
        local hover_inst = entitiesundermouse[1] -- HERE
        if hover_inst and hover_inst.widget then
            hover_inst.widget:SetFocus()
        elseif #self.screenstack > 0 then
            self.screenstack[#self.screenstack]:SetFocus()
        end
    end
end

TheInput:GetAllEntitiesUnderMouse simply returns the table with entities under the mouse and grabs the first one without filtering, so a mouse-through one might be used here.

Wouldn't it make more sense if Input had a function like:

-- name could also be "GetWorldOrHUDEntityUnderMouse", but that might be too long
function Input:GetEntityUnderMouse()
    return self.mouse_enabled and
        self.hoverinst ~= nil and
        self.hoverinst.entity:IsValid() and
        self.hoverinst.entity:IsVisible() and
        -- here there would normally be a Transform check to distinguish between World and HUD entities
        self.hoverinst or nil
end

And the FrontEnd function was changed to:

function FrontEnd:DoHoverFocusUpdate(manual_update)
    if self.tracking_mouse and not self.focus_locked then
        if manual_update then
            --something has been manually moved, so we want to update the entities under the mouse and re-evaluate the hover/focus state -- vanilla comment
            TheInput:UpdateEntitiesUnderMouse()
        end
        local hover_inst = TheInput:GetEntityUnderMouse()
        if hover_inst and hover_inst.widget then
            hover_inst.widget:SetFocus()
        elseif #self.screenstack > 0 then
            self.screenstack[#self.screenstack]:SetFocus()
        end
    end
end