Updates to Goto Position to work with the new camera system (#1212)

* add support for 'GoTo Position' to work with the new camera enabled

* small updates to move away from more legacy calls

* minor formatting change

* ensure values are converted from degrees to radians

* revert change to use Enumerate viewport call

* remove formatting changes from SandboxIntegration

Code/Sandbox/Editor/GameEngine.h

@@ -32,24 +32,6 @@ struct IInitializeUIInfo;
 #include <AzCore/Interface/Interface.h>
 #include <AzCore/Math/Vector3.h>
 
-namespace AzToolsFramework
-{
-    //! Operates the Editor's camera
-    class IEditorCameraController
-    {
-    public:
-        AZ_RTTI(IEditorCameraController, "{AEF60D3E-10A1-4161-9379-F68C69A5959C}");
-
-        IEditorCameraController() = default;
-        IEditorCameraController(IEditorCameraController&&) = delete;
-        IEditorCameraController& operator=(IEditorCameraController&&) = delete;
-        virtual ~IEditorCameraController() = default;
-
-        virtual void SetCurrentViewPosition([[maybe_unused]] const AZ::Vector3& position) {}
-        virtual void SetCurrentViewRotation([[maybe_unused]] const AZ::Vector3& rotation) {}
-    };
-}
-
 class ThreadedOnErrorHandler : public QObject
 {
     Q_OBJECT
@@ -68,7 +50,6 @@ AZ_PUSH_DISABLE_DLL_EXPORT_BASECLASS_WARNING
 //! This class serves as a high-level wrapper for CryEngine game.
 class SANDBOX_API CGameEngine
     : public IEditorNotifyListener
-    , private AzToolsFramework::IEditorCameraController
 {
 AZ_POP_DISABLE_DLL_EXPORT_BASECLASS_WARNING
 public:
@@ -153,11 +134,6 @@ public:
     }
 
 private:
-    AZ_PUSH_DISABLE_WARNING(4273, "-Wunknown-warning-option")
-    void SetCurrentViewPosition(const AZ::Vector3& position) override;
-    void SetCurrentViewRotation(const AZ::Vector3& rotation) override;
-    AZ_POP_DISABLE_OVERRIDE_WARNING
-
     void SetGameMode(bool inGame);
     void SwitchToInGame();
     void SwitchToInEditor();