[google_maps_flutter_web] Omit styles when cloudMapId is set

packages/google_maps_flutter/google_maps_flutter_web/CHANGELOG.md

@@ -1,5 +1,6 @@
 ## NEXT
 
+* Fixes a bug where using `cloudMapId` for cloud-based styling would fail if the `style` property was also present.
 * Updates minimum supported SDK version to Flutter 3.29/Dart 3.7.
 
 ## 0.5.12+2

packages/google_maps_flutter/google_maps_flutter_web/example/integration_test/cloud_map_styles_test.dart

@@ -0,0 +1,131 @@
+// Copyright 2013 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import 'dart:async';
+import 'package:flutter/widgets.dart'
+    show Directionality, SizedBox, TextDirection;
+import 'package:flutter_test/flutter_test.dart';
+import 'package:google_maps/google_maps.dart' as gmaps;
+import 'package:google_maps_flutter_platform_interface/google_maps_flutter_platform_interface.dart'
+    show
+        CameraPosition,
+        LatLng,
+        MapConfiguration,
+        MapEvent,
+        MapWidgetConfiguration;
+import 'package:google_maps_flutter_web/google_maps_flutter_web.dart'
+    show GoogleMapController;
+import 'package:integration_test/integration_test.dart';
+
+void main() {
+  IntegrationTestWidgetsFlutterBinding.ensureInitialized();
+
+  MapWidgetConfiguration cfg() => const MapWidgetConfiguration(
+    initialCameraPosition: CameraPosition(target: LatLng(0, 0), zoom: 1),
+    textDirection: TextDirection.ltr,
+  );
+
+  testWidgets('cloudMapId present => mapId set & styles omitted', (
+    WidgetTester tester,
+  ) async {
+    const MapConfiguration testMapConfig = MapConfiguration(
+      cloudMapId: 'test-cloud-map-id',
+    );
+
+    await tester.pumpWidget(
+      const Directionality(textDirection: TextDirection.ltr, child: SizedBox()),
+    );
+
+    final StreamController<MapEvent<Object?>> stream =
+        StreamController<MapEvent<Object?>>();
+    addTearDown(() {
+      // Stream is closed by controller.dispose()
+    });
+
+    gmaps.MapOptions? captured;
+
+    final GoogleMapController controller = GoogleMapController(
+      mapId: 1, // Internal controller ID
+      streamController: stream,
+      widgetConfiguration: cfg(),
+      mapConfiguration: testMapConfig, // cloudMapId is set here
+    );
+
+    controller.debugSetOverrides(
+      setOptions: (gmaps.MapOptions options) {
+        captured = options;
+      },
+    );
+
+    final List<gmaps.MapTypeStyle> styles = <gmaps.MapTypeStyle>[
+      gmaps.MapTypeStyle()
+        ..featureType = 'road'
+        ..elementType = 'geometry',
+    ];
+
+    controller.updateStyles(styles);
+
+    await tester.pump();
+
+    expect(captured, isNotNull);
+    expect(captured!.mapId, testMapConfig.cloudMapId);
+    expect(
+      captured!.styles == null || captured!.styles!.isEmpty,
+      isTrue,
+      reason: 'When cloudMapId is set, styles must not be applied.',
+    );
+
+    controller.dispose();
+  });
+
+  testWidgets('no cloudMapId => styles applied', (WidgetTester tester) async {
+    await tester.pumpWidget(
+      const Directionality(textDirection: TextDirection.ltr, child: SizedBox()),
+    );
+
+    final StreamController<MapEvent<Object?>> stream =
+        StreamController<MapEvent<Object?>>();
+    addTearDown(() {
+      // Stream is closed by controller.dispose()
+    });
+
+    gmaps.MapOptions? captured;
+    final GoogleMapController controller = GoogleMapController(
+      mapId: 2, // Internal controller ID
+      streamController: stream,
+      widgetConfiguration: cfg(),
+    );
+
+    controller.debugSetOverrides(
+      setOptions: (gmaps.MapOptions options) {
+        captured = options;
+      },
+    );
+
+    final List<gmaps.MapTypeStyle> styles = <gmaps.MapTypeStyle>[
+      gmaps.MapTypeStyle()
+        ..featureType = 'poi'
+        ..elementType = 'labels',
+    ];
+
+    controller.updateStyles(styles);
+
+    await tester.pump();
+
+    expect(captured, isNotNull);
+    expect(
+      captured!.mapId,
+      anyOf(isNull, isEmpty),
+      reason: 'mapId should be empty/null when no Cloud Map is used.',
+    );
+    expect(captured!.styles, isNotNull);
+    expect(
+      captured!.styles,
+      isNotEmpty,
+      reason: 'When cloudMapId is null, styles should be applied.',
+    );
+
+    controller.dispose();
+  });
+}

packages/google_maps_flutter/google_maps_flutter_web/lib/src/convert.dart

@@ -99,15 +99,17 @@ gmaps.MapOptions _configurationAndStyleToGmapsOptions(
     options.rotateControl = configuration.fortyFiveDegreeImageryEnabled;
   }
 
+  // If using cloud map, do not set options.styles
+  if (configuration.cloudMapId == null) {
+    options.styles = styles;
+  }
+
   // These don't have any configuration entries, but they seem to be off in the
   // native maps.
   options.mapTypeControl = false;
   options.fullscreenControl = false;
   options.streetViewControl = false;
 
-  // See updateMapConfiguration for why this is not using configuration.style.
-  options.styles = styles;
-
   options.mapId = configuration.cloudMapId;
 
   return options;