Skip to content

Commit bdabc4f

Browse files
LanguageDetector: Remove AILanguageDetectorFactory
Remove AILanguageDetectorFactory and expose create and availability statically on LanguageDetector. This is a followup change to CLs crrev.com/c/6402873, crrev.com/c/6402775. Note: This change is a cherry-pick of crrev.com/c/6397965. Some changes have been made as part of rebasing + fixing previous CQ test failures. PS #1 represents the cherry-picked change prior to rebase + minor edits. Fixed: 402165734 Change-Id: Id384ebe31ccfe9e0efd65dd7c890cd66875a9ed8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6403274 Reviewed-by: Mike Wasserman <[email protected]> Commit-Queue: Christine Hollingsworth <[email protected]> Cr-Commit-Position: refs/heads/main@{#1439000}
1 parent d803d63 commit bdabc4f

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

ai/language_detection/availability-detached-crash.tentative.https.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<body>
66
<iframe src="about:blank"></iframe>
77
<script>
8-
const factory = frames[0].ai.languageDetector;
8+
const factory = frames[0].LanguageDetector;
99
const detector = factory.create();
1010
document.getElementsByTagName("iframe")[0].remove();
1111
detector.availability();

ai/language_detection/detector.https.tentative.any.js

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@
66

77
promise_test(async t => {
88
// Language detection is available after call to `create()`.
9-
await ai.languageDetector.create();
10-
const availability = await ai.languageDetector.availability();
9+
await LanguageDetector.create();
10+
const availability = await LanguageDetector.availability();
1111
assert_equals(availability, 'available');
1212
}, 'Simple LanguageDetector.availability() call');
1313

1414
promise_test(async t => {
15-
const detector = await ai.languageDetector.create();
15+
const detector = await LanguageDetector.create();
1616
const results = await detector.detect('this string is in English');
1717
// "en" should be highest confidence.
1818
assert_equals(results[0].detectedLanguage, 'en');
@@ -26,37 +26,37 @@ promise_test(async t => {
2626
const controller = new AbortController();
2727
controller.abort();
2828

29-
const createPromise = ai.languageDetector.create({signal: controller.signal});
29+
const createPromise = LanguageDetector.create({signal: controller.signal});
3030

3131
await promise_rejects_dom(t, 'AbortError', createPromise);
32-
}, 'AILanguageDetectorFactory.create() call with an aborted signal.');
32+
}, 'LanguageDetector.create() call with an aborted signal.');
3333

3434
promise_test(async t => {
3535
await testAbortPromise(t, signal => {
36-
return ai.languageDetector.create({signal});
36+
return LanguageDetector.create({signal});
3737
});
38-
}, 'Aborting AILanguageDetectorFactory.create().');
38+
}, 'Aborting LanguageDetector.create().');
3939

4040
promise_test(async t => {
4141
const controller = new AbortController();
4242
controller.abort();
4343

44-
const detector = await ai.languageDetector.create();
44+
const detector = await LanguageDetector.create();
4545
const detectPromise =
4646
detector.detect('this string is in English', {signal: controller.signal});
4747

4848
await promise_rejects_dom(t, 'AbortError', detectPromise);
4949
}, 'LanguageDetector.detect() call with an aborted signal.');
5050

5151
promise_test(async t => {
52-
const detector = await ai.languageDetector.create();
52+
const detector = await LanguageDetector.create();
5353
await testAbortPromise(t, signal => {
5454
return detector.detect('this string is in English', {signal});
5555
});
5656
}, 'Aborting LanguageDetector.detect().');
5757

5858
promise_test(async t => {
59-
const detector = await ai.languageDetector.create();
59+
const detector = await LanguageDetector.create();
6060

6161
const text = 'this string is in English';
6262
const inputUsage = await detector.measureInputUsage(text);
@@ -77,15 +77,15 @@ promise_test(async t => {
7777
const controller = new AbortController();
7878
controller.abort();
7979

80-
const detector = await ai.languageDetector.create();
80+
const detector = await LanguageDetector.create();
8181
const measureInputUsagePromise =
8282
detector.measureInputUsage('hello', {signal: controller.signal});
8383

8484
await promise_rejects_dom(t, 'AbortError', measureInputUsagePromise);
8585
}, 'Translator.measureInputUsage() call with an aborted signal.');
8686

8787
promise_test(async t => {
88-
const detector = await ai.languageDetector.create();
88+
const detector = await LanguageDetector.create();
8989
await testAbortPromise(t, signal => {
9090
return detector.measureInputUsage('hello', {signal});
9191
});

0 commit comments

Comments
 (0)