[hdx] Fix free camera setter.

In SetViewAndProjectionMatrix, we were ignoring any clipping planes that were set. This resulted in repeated thrashing on the clipping planes state when SetClippingPlanes was called leading to spurious scene updates causing renders to restart and never converge.

(Internal change: 2380790)

Author: rajabala

pxr/imaging/hdx/freeCameraPrimDataSource.cpp

@@ -298,7 +298,7 @@ HdxFreeCameraPrimDataSource::SetWindowPolicy(
     }
     static const HdDataSourceLocator locator =
         HdCameraSchema::GetDefaultLocator()
-            .Append(_tokens->windowPolicy);                    
+            .Append(_tokens->windowPolicy);
     dirtyLocators->insert(locator);
 }
 
@@ -308,10 +308,11 @@ HdxFreeCameraPrimDataSource::SetViewAndProjectionMatrix(
     const GfMatrix4d& projectionMatrix,
     HdDataSourceLocatorSet * dirtyLocators)
 {
-    SetCamera(
-        _CameraFromViewAndProjectionMatrix(
-            viewMatrix, projectionMatrix),
-        dirtyLocators);
+    GfCamera cam = _CameraFromViewAndProjectionMatrix(
+        viewMatrix, projectionMatrix);
+    cam.SetClippingPlanes(_info->camera.GetClippingPlanes());
+
+    SetCamera(cam, dirtyLocators);
 }
 
 void

pxr/imaging/hdx/taskControllerSceneIndex.cpp

@@ -2398,7 +2398,8 @@ SetFreeCameraClipPlanes(const std::vector<GfVec4d> &clippingPlanes)
     }
 
     HdDataSourceLocatorSet locators;
-    ds->SetClippingPlanes({clippingPlanes.begin(), clippingPlanes.end()});
+    ds->SetClippingPlanes(
+        {clippingPlanes.begin(), clippingPlanes.end()}, &locators);
 
     if (locators.IsEmpty()) {
         return;