Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Renaming "legacy" paths and DDC modules to "DDC" paths and modules. #2360

Merged
merged 6 commits into from
Feb 2, 2024
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions dwds/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
## 23.4.0-wip

- Adding tests for constants in DDC after a hot restart - [#2349](https://github.com/dart-lang/webdev/pull/2349)
- Renaming `dart_library.js` to `ddc_module_loader.js` to match SDK naming changes - [#2360](https://github.com/dart-lang/webdev/pull/2360)

## 23.3.0

Expand Down
6 changes: 3 additions & 3 deletions dwds/lib/dwds.dart
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ export 'src/handlers/dev_handler.dart' show AppConnectionException;
export 'src/handlers/socket_connections.dart';
export 'src/loaders/build_runner_require.dart'
show BuildRunnerRequireStrategyProvider;
export 'src/loaders/frontend_server_legacy.dart'
show FrontendServerLegacyStrategyProvider;
export 'src/loaders/ddc.dart' show DdcStrategy;
export 'src/loaders/frontend_server_ddc.dart'
show FrontendServerDdcStrategyProvider;
export 'src/loaders/frontend_server_require.dart'
show FrontendServerRequireStrategyProvider;
export 'src/loaders/legacy.dart' show LegacyStrategy;
export 'src/loaders/require.dart' show RequireStrategy;
export 'src/loaders/strategy.dart'
show LoadStrategy, ReloadConfiguration, BuildSettings;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ var baseUrl = (function () {
}());
''';

/// A load strategy for the legacy module system.
class LegacyStrategy extends LoadStrategy {
/// A load strategy for the DDC module system.
class DdcStrategy extends LoadStrategy {
@override
final ReloadConfiguration reloadConfiguration;

Expand Down Expand Up @@ -129,7 +129,7 @@ class LegacyStrategy extends LoadStrategy {

final BuildSettings _buildSettings;

LegacyStrategy(
DdcStrategy(
this.reloadConfiguration,
this._moduleProvider,
this._digestsProvider,
Expand All @@ -152,26 +152,26 @@ class LegacyStrategy extends LoadStrategy {
};

@override
String get id => 'legacy';
String get id => 'ddc';

@override
String get moduleFormat => 'ddc';

@override
String get loadLibrariesModule => 'dart_library.ddk.js';
String get loadLibrariesModule => 'ddc_module_loader.ddk.js';

@override
String get loadModuleSnippet => 'dart_library.import';

@override
Future<String> bootstrapFor(String entrypoint) async =>
await _legacyLoaderSetup(entrypoint);
await _ddcLoaderSetup(entrypoint);

@override
String loadClientSnippet(String clientScript) =>
'window.\$dartLoader.forceLoadModule("$clientScript");\n';

Future<String> _legacyLoaderSetup(String entrypoint) async {
Future<String> _ddcLoaderSetup(String entrypoint) async {
final metadataProvider = metadataProviderFor(entrypoint);
final modulePaths = await _moduleProvider(metadataProvider);
final scripts = <Map<String, String?>>[];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,22 +3,22 @@
// found in the LICENSE file.

import 'package:dwds/src/debugging/metadata/provider.dart';
import 'package:dwds/src/loaders/legacy.dart';
import 'package:dwds/src/loaders/ddc.dart';
import 'package:dwds/src/loaders/strategy.dart';
import 'package:dwds/src/readers/asset_reader.dart';
import 'package:dwds/src/services/expression_compiler.dart';
import 'package:path/path.dart' as p;

/// Provides a [LegacyStrategy] suitable for use with Frontend Server.
class FrontendServerLegacyStrategyProvider {
/// Provides a [DdcStrategy] suitable for use with Frontend Server.
class FrontendServerDdcStrategyProvider {
final ReloadConfiguration _configuration;
final AssetReader _assetReader;
final PackageUriMapper _packageUriMapper;
final Future<Map<String, String>> Function() _digestsProvider;
final String _basePath;
final BuildSettings _buildSettings;

late final LegacyStrategy _legacyStrategy = LegacyStrategy(
late final DdcStrategy _ddcStrategy = DdcStrategy(
_configuration,
_moduleProvider,
(_) => _digestsProvider(),
Expand All @@ -33,15 +33,15 @@ class FrontendServerLegacyStrategyProvider {
null,
);

FrontendServerLegacyStrategyProvider(
FrontendServerDdcStrategyProvider(
this._configuration,
this._assetReader,
this._packageUriMapper,
this._digestsProvider,
this._buildSettings,
) : _basePath = _assetReader.basePath;

LegacyStrategy get strategy => _legacyStrategy;
DdcStrategy get strategy => _ddcStrategy;

String _removeBasePath(String path) {
if (_basePath.isEmpty) return path;
Expand Down
4 changes: 2 additions & 2 deletions dwds/test/fixtures/context.dart
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import 'package:dwds/src/connections/app_connection.dart';
import 'package:dwds/src/connections/debug_connection.dart';
import 'package:dwds/src/debugging/webkit_debugger.dart';
import 'package:dwds/src/loaders/build_runner_require.dart';
import 'package:dwds/src/loaders/frontend_server_legacy.dart';
import 'package:dwds/src/loaders/frontend_server_ddc.dart';
import 'package:dwds/src/loaders/frontend_server_require.dart';
import 'package:dwds/src/loaders/strategy.dart';
import 'package:dwds/src/readers/proxy_server_asset_reader.dart';
Expand Down Expand Up @@ -343,7 +343,7 @@ class TestContext {
() async => {},
buildSettings,
).strategy,
ModuleFormat.ddc => FrontendServerLegacyStrategyProvider(
ModuleFormat.ddc => FrontendServerDdcStrategyProvider(
testSettings.reloadConfiguration,
assetReader,
packageUriMapper,
Expand Down
6 changes: 3 additions & 3 deletions dwds/web/client.dart
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ import 'package:uuid/uuid.dart';
import 'package:web_socket_channel/web_socket_channel.dart';

import 'promise.dart';
import 'reloader/legacy_restarter.dart';
import 'reloader/ddc_restarter.dart';
import 'reloader/manager.dart';
import 'reloader/require_restarter.dart';
import 'reloader/restarter.dart';
Expand All @@ -54,8 +54,8 @@ Future<void>? main() {
Restarter restarter;
if (dartModuleStrategy == 'require-js') {
restarter = await RequireRestarter.create();
} else if (dartModuleStrategy == 'legacy') {
restarter = LegacyRestarter();
} else if (dartModuleStrategy == 'ddc' || dartModuleStrategy == 'legacy') {
restarter = DdcRestarter();
} else {
throw StateError('Unknown module strategy: $dartModuleStrategy');
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import 'dart:js';

import 'restarter.dart';

class LegacyRestarter implements Restarter {
class DdcRestarter implements Restarter {
@override
Future<bool> restart({String? runId}) async {
final dartLibrary = context['dart_library'] as JsObject;
Expand Down
Loading