fix(reviewer): address review comments on server binding, permissions, immersive mode, and intent extras

Author: dkhawk

ApiDemos/project/java-app/src/main/java/com/example/mapdemo/SamplesBaseActivity.java

@@ -86,6 +86,9 @@ public void onWindowFocusChanged(boolean hasFocus) {
     }
 
     private void applyImmersiveStickyMode() {
+        if (!isReviewerMode()) {
+            return;
+        }
         WindowInsetsControllerCompat insetsController =
                 WindowCompat.getInsetsController(getWindow(), getWindow().getDecorView());
         insetsController.setSystemBarsBehavior(

ApiDemos/project/kotlin-app/src/androidTest/java/com/example/kotlindemos/BasicMapReviewerUiTest.kt

@@ -37,7 +37,12 @@ class BasicMapReviewerUiTest {
 
     @Test
     fun testBasicMapHasReviewerToolbarActions() {
-        val scenario = ActivityScenario.launch(BasicMapDemoActivity::class.java)
+        val context = InstrumentationRegistry.getInstrumentation().targetContext
+        val intent = Intent(context, BasicMapDemoActivity::class.java).apply {
+            putExtra("extra_sample_id", "com.example.kotlindemos.BasicMapDemoActivity")
+            putExtra("extra_is_reviewer_mode", true)
+        }
+        val scenario = ActivityScenario.launch<BasicMapDemoActivity>(intent)
 
         scenario.onActivity { activity ->
             val toolbar = activity.findViewById<MaterialToolbar>(R.id.top_bar)

ApiDemos/project/kotlin-app/src/main/java/com/example/kotlindemos/SamplesBaseActivity.kt

@@ -75,6 +75,7 @@ open class SamplesBaseActivity : AppCompatActivity() {
     }
 
     private fun applyImmersiveStickyMode() {
+        if (!isReviewerMode) return
         val insetsController = WindowCompat.getInsetsController(window, window.decorView)
         insetsController.systemBarsBehavior =
             WindowInsetsControllerCompat.BEHAVIOR_SHOW_TRANSIENT_BARS_BY_SWIPE

ApiDemos/project/reviewer/src/main/res/xml/file_paths.xml

@@ -17,7 +17,4 @@
 
 <paths xmlns:android="http://schemas.android.com/apk/res/android">
     <external-files-path name="reports" path="reports" />
-    <external-files-path name="external_files" path="." />
-    <files-path name="internal_files" path="." />
-    <cache-path name="internal_cache" path="." />
 </paths>

test/server.py

@@ -341,16 +341,17 @@ def update_markdown_task_list(db_data):
 
 if __name__ == "__main__":
     parser = argparse.ArgumentParser(description="Interactive API Server for Maps SDK Review Dashboard.")
+    parser.add_argument("--host", type=str, default="127.0.0.1", help="Host interface to bind to (default: 127.0.0.1)")
     parser.add_argument("--port", type=int, default=8888, help="Port to bind the HTTP server to (default: 8888)")
     parser.add_argument("--java-home", "--jdk-home", dest="java_home", type=str, default="/usr/lib/jvm/java-21-openjdk-amd64", help="Path to JDK to use when executing Gradle build/test tasks")
     args = parser.parse_args()
 
     PORT = args.port
     CUSTOM_JAVA_HOME = args.java_home
 
-    server_address = ("0.0.0.0", PORT)
+    server_address = (args.host, PORT)
     httpd = http.server.ThreadingHTTPServer(server_address, CatalogRequestHandler)
-    print(f"🚀 Interactive Catalog API Server running on port {PORT} (JDK Home: {CUSTOM_JAVA_HOME})...")
+    print(f"🚀 Interactive Catalog API Server running on http://{args.host}:{PORT} (JDK Home: {CUSTOM_JAVA_HOME})...")
     try:
         httpd.serve_forever()
     except KeyboardInterrupt: