Fix re-render conditions in BackgroundReplacementProvider and logging (#985)

Author: ziyiz-amzn

CHANGELOG.md

@@ -16,9 +16,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Change to use loggers from `LoggerProvider` as default when initialize background blur/replacement processor.
+- Change to not log the entire stringified `options` object in background blur/replacement processor.
 
 ### Fixed
 
+- Fix conditions where `BackgroundReplacementProvider` needs to be re-rendered.
+
 ## [3.9.0] - 2024-04-12
 
 ### Added

src/providers/BackgroundBlurProvider/index.tsx

@@ -110,7 +110,11 @@ export const BackgroundBlurProvider: FC<React.PropsWithChildren<Props>> = ({
     logger.info(
       `Initializing background blur processor with, spec: ${JSON.stringify(
         spec
-      )}, options: ${JSON.stringify(options)}`
+      )}, options: { blurStrength: ${
+        options?.blurStrength
+      }, filterCPUUtilization: ${
+        options?.filterCPUUtilization
+      }, reportingPeriodMillis: ${options?.reportingPeriodMillis} }`
     );
 
     try {

src/providers/BackgroundReplacementProvider/index.tsx

@@ -82,8 +82,7 @@ export const BackgroundReplacementProvider: FC<
         return true;
       }
       if (
-        prev?.imageBlob?.size === next?.imageBlob?.size ||
-        prev?.filterCPUUtilization === next?.filterCPUUtilization ||
+        prev?.filterCPUUtilization !== next?.filterCPUUtilization ||
         prev?.logger !== next?.logger ||
         prev?.reportingPeriodMillis !== next?.reportingPeriodMillis
       ) {
@@ -111,7 +110,11 @@ export const BackgroundReplacementProvider: FC<
     logger.info(
       `Initializing background replacement processor with, ${JSON.stringify(
         spec
-      )}, ${JSON.stringify(options)}`
+      )}, options: { filterCPUUtilization: ${
+        options?.filterCPUUtilization
+      }, reportingPeriodMillis: ${
+        options?.reportingPeriodMillis
+      }}`
     );
 
     try {

tst/providers/BackgroundBlurProvider/index.test.tsx

@@ -63,7 +63,7 @@ describe.only('BackgroundBlurProvider', () => {
     expect(loggerInfoMock).toHaveBeenCalledWith(
       `Initializing background blur processor with, spec: ${JSON.stringify(
         undefined
-      )}, options: ${JSON.stringify(blurOptions)}`
+      )}, options: { blurStrength: ${blurOptions.blurStrength}, filterCPUUtilization: ${blurOptions.filterCPUUtilization}, reportingPeriodMillis: ${blurOptions.reportingPeriodMillis} }`
     );
     expect(loggerInfoMock).toHaveBeenCalledWith(
       'Specs or options were changed. Destroying and re-initializing background blur processor.'