Revert "Add a screen-ai CIPD package for PDF OCR E2E tests on Linux"
This reverts commit 8c005672e75f619a1f2df8241fe996f9f03cbe7d.
Reason for revert:
The test |PDFOCRIntegrationTest.EnsureScreenAIInitializes| introduced
in this CL consistently fails in Linux MSan Tests.
Please see crbug.com/1513808 for more information.
Original change's description:
> Add a screen-ai CIPD package for PDF OCR E2E tests on Linux
>
> To have E2E tests for PDF OCR on Windows, macOS, and Linux, we need to
> be able to download and use the screen-ai library in those tests.
> Pursuing this goal, we upload the screen-ai library for Linux to CIPD
> and download it from CIPD to //third_party/screen-ai. This downloaded
> library is then loaded in E2E tests (i.e. browser tests).
>
> For more information, visit go/chrome_screen_ai_tests and read the
> Uploading the library files to CIPD section.
>
> AX-Relnotes: n/a.
> Bug: 1443346
> Change-Id: I7d55f9ae970a8e39aafbc4273fae2d654b4af17e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4803348
> Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
> Reviewed-by: John Abd-El-Malek <jam@chromium.org>
> Reviewed-by: Lei Zhang <thestig@chromium.org>
> Commit-Queue: Kyungjun Lee <kyungjunlee@google.com>
> Reviewed-by: David Tseng <dtseng@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1240288}
Bug: 1443346
Change-Id: If2f6d956e634872f9e0816dfd8c1c9bbcbda9e16
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5148588
Commit-Queue: Taiyo Mizuhashi <taiyo@chromium.org>
Owners-Override: Taiyo Mizuhashi <taiyo@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1240444}
diff --git a/DEPS b/DEPS
index 1132e900..38e8b2c 100644
--- a/DEPS
+++ b/DEPS
@@ -274,11 +274,6 @@
# reclient CIPD package version
'reclient_version': 're_client_version:0.125.0.f3883c2-gomaip',
- # screen-ai CIPD package Linux version
- # TODO(b/281483558): Use a tag to download the latest version of screen-ai
- # (e.g. 'version:121.3') and find a way to automate updating //DEPS with it.
- 'screen_ai_linux': 'x8c4xOQj3V2uyBicjNa2YkN71brkj5FZg157RueHF_oC',
-
# The path of the sysroots.json file.
# This is used by vendor builds like Electron.
'sysroots_json_path': 'build/linux/sysroot_scripts/sysroots.json',
@@ -4087,17 +4082,6 @@
'condition': 'checkout_src_internal',
},
- 'src/third_party/screen-ai/linux': {
- 'packages': [
- {
- 'package': 'chromium/third_party/screen-ai/linux',
- 'version': Var('screen_ai_linux'),
- },
- ],
- 'condition': 'checkout_linux',
- 'dep_type': 'cipd',
- },
-
'src/third_party/soda': {
'packages': [
{
diff --git a/chrome/browser/pdf/pdf_extension_accessibility_test.cc b/chrome/browser/pdf/pdf_extension_accessibility_test.cc
index 7e62591..7af8f1b 100644
--- a/chrome/browser/pdf/pdf_extension_accessibility_test.cc
+++ b/chrome/browser/pdf/pdf_extension_accessibility_test.cc
@@ -14,7 +14,6 @@
#include "base/containers/flat_set.h"
#include "base/files/file_util.h"
#include "base/logging.h"
-#include "base/run_loop.h"
#include "base/strings/strcat.h"
#include "base/strings/string_piece.h"
#include "base/strings/string_util.h"
@@ -65,14 +64,6 @@
#include "chrome/browser/renderer_context_menu/pdf_ocr_menu_observer.h"
#endif // BUILDFLAG(ENABLE_SCREEN_AI_SERVICE)
-#if BUILDFLAG(IS_LINUX)
-#include "base/scoped_observation.h"
-#include "chrome/browser/screen_ai/screen_ai_install_state.h"
-#include "chrome/common/pref_names.h"
-#include "components/strings/grit/components_strings.h"
-#include "ui/base/l10n/l10n_util.h"
-#endif // BUILDFLAG(IS_LINUX)
-
namespace {
using ::content::WebContents;
@@ -1173,110 +1164,3 @@
#if BUILDFLAG(ENABLE_SCREEN_AI_SERVICE)
INSTANTIATE_FEATURE_OVERRIDE_TEST_SUITE(PDFExtensionAccessibilityPdfOcrTest);
#endif // BUILDFLAG(ENABLE_SCREEN_AI_SERVICE)
-
-#if BUILDFLAG(IS_LINUX)
-
-class ScreenAIInstallStateObserver
- : public screen_ai::ScreenAIInstallState::Observer {
- public:
- ScreenAIInstallStateObserver() {
- component_ready_observer_.Observe(
- screen_ai::ScreenAIInstallState::GetInstance());
- }
-
- ScreenAIInstallStateObserver(const ScreenAIInstallStateObserver&) = delete;
- ScreenAIInstallStateObserver& operator=(const ScreenAIInstallStateObserver&) =
- delete;
-
- ~ScreenAIInstallStateObserver() override = default;
-
- // screen_ai::ScreenAIInstallState::Observer:
- void StateChanged(screen_ai::ScreenAIInstallState::State state) override {
- if (state == screen_ai::ScreenAIInstallState::State::kReady) {
- is_ready_ = true;
- if (quit_closure_) {
- std::move(quit_closure_).Run();
- }
- }
- }
-
- void WaitForReady() {
- if (is_ready_) {
- return;
- }
-
- base::RunLoop run_loop;
- quit_closure_ = run_loop.QuitClosure();
- run_loop.Run();
- }
-
- private:
- bool is_ready_ = false;
- base::OnceClosure quit_closure_;
- base::ScopedObservation<screen_ai::ScreenAIInstallState,
- screen_ai::ScreenAIInstallState::Observer>
- component_ready_observer_{this};
-};
-
-class PDFOCRIntegrationTest : public PDFExtensionAccessibilityTest {
- public:
- PDFOCRIntegrationTest() = default;
- ~PDFOCRIntegrationTest() override = default;
-
- // PDFExtensionAccessibilityTest:
- void SetUpOnMainThread() override {
- PDFExtensionAccessibilityTest::SetUpOnMainThread();
-
- screen_ai::ScreenAIInstallState::GetInstance()
- ->SetComponentFolderForTesting();
-
- content::BrowserAccessibilityState::GetInstance()->EnableAccessibility();
- EnableScreenReader(true);
- }
-
- void TearDownOnMainThread() override {
- EnableScreenReader(false);
- PDFExtensionAccessibilityTest::TearDownOnMainThread();
- }
-
- protected:
- std::vector<base::test::FeatureRef> GetEnabledFeatures() const override {
- auto enabled = PDFExtensionAccessibilityTest::GetEnabledFeatures();
- enabled.push_back(::features::kPdfOcr);
- enabled.push_back(::features::kScreenAITestMode);
- return enabled;
- }
-
- std::vector<base::test::FeatureRef> GetDisabledFeatures() const override {
- // `PDFExtensionAccessibilityTest` has `::features::kPdfOcr` in a list of
- // disabled features. Now that `::features::kPdfOcr` is used in this test,
- // just return an empty list to exclude the feature from the list.
- return {};
- }
-
- void EnableScreenReader(bool enabled) {
- // Spoof a screen reader.
- if (enabled) {
- content::BrowserAccessibilityState::GetInstance()
- ->AddAccessibilityModeFlags(ui::AXMode::kScreenReader);
- } else {
- content::BrowserAccessibilityState::GetInstance()
- ->RemoveAccessibilityModeFlags(ui::AXMode::kScreenReader);
- }
- }
-};
-
-IN_PROC_BROWSER_TEST_F(PDFOCRIntegrationTest, EnsureScreenAIInitializes) {
- ScreenAIInstallStateObserver observer;
-
- // Turn on PDF OCR by setting its pref to be true.
- browser()->profile()->GetPrefs()->SetBoolean(
- prefs::kAccessibilityPdfOcrAlwaysActive, true);
- EXPECT_TRUE(browser()->profile()->GetPrefs()->GetBoolean(
- prefs::kAccessibilityPdfOcrAlwaysActive));
-
- observer.WaitForReady();
- EXPECT_EQ(screen_ai::ScreenAIInstallState::State::kReady,
- screen_ai::ScreenAIInstallState::GetInstance()->get_state());
-}
-#endif // BUILDFLAG(IS_LINUX)
diff --git a/chrome/browser/screen_ai/screen_ai_install_state.cc b/chrome/browser/screen_ai/screen_ai_install_state.cc
index eead748..45eaabea 100644
--- a/chrome/browser/screen_ai/screen_ai_install_state.cc
+++ b/chrome/browser/screen_ai/screen_ai_install_state.cc
@@ -24,7 +24,6 @@
#if BUILDFLAG(IS_LINUX)
#include "base/cpu.h"
-#include "base/files/file_util.h"
#endif
#if BUILDFLAG(IS_WIN)
@@ -249,18 +248,6 @@
component_binary_path_.clear();
}
-void ScreenAIInstallState::SetComponentFolderForTesting() {
- CHECK_IS_TEST();
-#if BUILDFLAG(IS_LINUX)
- // Set the path to the ScreenAI test files. For more details, see the
- // `screen_ai_test_files` rule in the accessibility_common BUILD file.
- base::FilePath screenai_library_path =
- screen_ai::GetLatestComponentBinaryPath();
- CHECK(base::PathExists(screenai_library_path));
- SetComponentFolder(screenai_library_path.DirName());
-#endif // BUILDFLAG(IS_LINUX)
-}
-
void ScreenAIInstallState::SetStateForTesting(State state) {
state_ = state;
for (ScreenAIInstallState::Observer* observer : observers_) {
diff --git a/chrome/browser/screen_ai/screen_ai_install_state.h b/chrome/browser/screen_ai/screen_ai_install_state.h
index 2af8a31a..933b296 100644
--- a/chrome/browser/screen_ai/screen_ai_install_state.h
+++ b/chrome/browser/screen_ai/screen_ai_install_state.h
@@ -103,7 +103,6 @@
State get_state() { return state_; }
void ResetForTesting();
- void SetComponentFolderForTesting();
void SetStateForTesting(State state);
private:
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index 823b558..ad3dfdb6 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -3097,13 +3097,6 @@
"//pdf/loader",
]
- if (is_linux) {
- # Add a data dependency for pdf_extension_accessibility_test.cc to
- # notify testing builders that this test needs this library, which will
- # need to be downloaded from CIPD as defined in //DEPS.
- data_deps += [ "//third_party/screen-ai:screen_ai_linux" ]
- }
-
if (enable_printing) {
sources += [ "../browser/pdf/pdf_extension_printing_test.cc" ]
diff --git a/components/services/screen_ai/public/cpp/BUILD.gn b/components/services/screen_ai/public/cpp/BUILD.gn
index 21ace58b..240357f0 100644
--- a/components/services/screen_ai/public/cpp/BUILD.gn
+++ b/components/services/screen_ai/public/cpp/BUILD.gn
@@ -11,7 +11,6 @@
deps = [
"//base",
"//components/component_updater",
- "//ui/accessibility:ax_base",
]
configs += [ "//build/config/compiler:wexit_time_destructors" ]
diff --git a/components/services/screen_ai/public/cpp/DEPS b/components/services/screen_ai/public/cpp/DEPS
index 7669085..905953f 100644
--- a/components/services/screen_ai/public/cpp/DEPS
+++ b/components/services/screen_ai/public/cpp/DEPS
@@ -1,9 +1,3 @@
include_rules = [
"+components/component_updater",
]
-
-specific_include_rules = {
- "utilities.cc": [
- "+ui/accessibility/accessibility_features.h",
- ],
-}
diff --git a/components/services/screen_ai/public/cpp/utilities.cc b/components/services/screen_ai/public/cpp/utilities.cc
index 14bb69c..f67e960 100644
--- a/components/services/screen_ai/public/cpp/utilities.cc
+++ b/components/services/screen_ai/public/cpp/utilities.cc
@@ -4,14 +4,12 @@
#include "components/services/screen_ai/public/cpp/utilities.h"
-#include "base/check_is_test.h"
#include "base/files/file_enumerator.h"
#include "base/files/file_path.h"
#include "base/files/file_util.h"
#include "base/path_service.h"
#include "build/build_config.h"
#include "components/component_updater/component_updater_paths.h"
-#include "ui/accessibility/accessibility_features.h"
namespace screen_ai {
@@ -32,24 +30,6 @@
"/run/imageloader/screen-ai/package/root/";
#endif
-#if BUILDFLAG(IS_LINUX)
-constexpr base::FilePath::CharType kScreenAIResourcePathForTests[] =
- FILE_PATH_LITERAL("third_party/screen-ai/linux/resources");
-
-// Get the absolute path of the ScreenAI component for testing.
-base::FilePath GetTestComponentBinaryPath() {
- base::FilePath test_data_dir;
- CHECK(base::PathService::Get(base::DIR_SRC_TEST_DATA_ROOT, &test_data_dir));
-
- base::FilePath screenai_library_path =
- test_data_dir.Append(base::FilePath(kScreenAIResourcePathForTests))
- .Append(kScreenAIComponentBinaryName);
-
- CHECK(base::PathExists(screenai_library_path));
- return screenai_library_path;
-}
-#endif // BUILDFLAG(IS_LINUX)
-
} // namespace
base::FilePath GetRelativeInstallDir() {
@@ -72,13 +52,6 @@
}
base::FilePath GetLatestComponentBinaryPath() {
-#if BUILDFLAG(IS_LINUX)
- if (features::IsScreenAITestModeEnabled()) {
- CHECK_IS_TEST();
- return GetTestComponentBinaryPath();
- }
-#endif // BUILDFLAG(IS_LINUX)
-
base::FilePath latest_version_dir;
#if BUILDFLAG(IS_CHROMEOS)
latest_version_dir = base::FilePath::FromASCII(kScreenAIDlcRootPath);
diff --git a/third_party/.gitignore b/third_party/.gitignore
index 563c82b8..b3a5540 100644
--- a/third_party/.gitignore
+++ b/third_party/.gitignore
@@ -137,7 +137,6 @@
/rust-toolchain-intermediate
/scan-build/src
/scons-2.0.1
-/screen-ai/linux/
/shaka-player/dist/
/siso/siso
/siso/siso.exe
diff --git a/third_party/screen-ai/BUILD.gn b/third_party/screen-ai/BUILD.gn
deleted file mode 100644
index a9c6149..0000000
--- a/third_party/screen-ai/BUILD.gn
+++ /dev/null
@@ -1,11 +0,0 @@
-# Copyright 2023 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-group("screen_ai_linux") {
- # Note that this group depends on data that must be used only for test. If we
- # need to remove/update the testonly variable below, we must go through a
- # security review.
- testonly = true
- data_deps = [ "//third_party/screen-ai/linux:screen_ai_resources" ]
-}
diff --git a/third_party/screen-ai/LICENSE b/third_party/screen-ai/LICENSE
deleted file mode 100644
index 8c0497a..0000000
--- a/third_party/screen-ai/LICENSE
+++ /dev/null
@@ -1,179 +0,0 @@
-- Eigen is licensed for use as follows:
-/*
- Apache License
- Version 2.0, January 2004
- http://www.apache.org/licenses/
-
- TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
-
- 1. Definitions.
-
- "License" shall mean the terms and conditions for use, reproduction,
- and distribution as defined by Sections 1 through 9 of this document.
-
- "Licensor" shall mean the copyright owner or entity authorized by
- the copyright owner that is granting the License.
-
- "Legal Entity" shall mean the union of the acting entity and all
- other entities that control, are controlled by, or are under common
- control with that entity. For the purposes of this definition,
- "control" means (i) the power, direct or indirect, to cause the
- direction or management of such entity, whether by contract or
- otherwise, or (ii) ownership of fifty percent (50%) or more of the
- outstanding shares, or (iii) beneficial ownership of such entity.
-
- "You" (or "Your") shall mean an individual or Legal Entity
- exercising permissions granted by this License.
-
- "Source" form shall mean the preferred form for making modifications,
- including but not limited to software source code, documentation
- source, and configuration files.
-
- "Object" form shall mean any form resulting from mechanical
- transformation or translation of a Source form, including but
- not limited to compiled object code, generated documentation,
- and conversions to other media types.
-
- "Work" shall mean the work of authorship, whether in Source or
- Object form, made available under the License, as indicated by a
- copyright notice that is included in or attached to the work
- (an example is provided in the Appendix below).
-
- "Derivative Works" shall mean any work, whether in Source or Object
- form, that is based on (or derived from) the Work and for which the
- editorial revisions, annotations, elaborations, or other modifications
- represent, as a whole, an original work of authorship. For the purposes
- of this License, Derivative Works shall not include works that remain
- separable from, or merely link (or bind by name) to the interfaces of,
- the Work and Derivative Works thereof.
-
- "Contribution" shall mean any work of authorship, including
- the original version of the Work and any modifications or additions
- to that Work or Derivative Works thereof, that is intentionally
- submitted to Licensor for inclusion in the Work by the copyright owner
- or by an individual or Legal Entity authorized to submit on behalf of
- the copyright owner. For the purposes of this definition, "submitted"
- means any form of electronic, verbal, or written communication sent
- to the Licensor or its representatives, including but not limited to
- communication on electronic mailing lists, source code control systems,
- and issue tracking systems that are managed by, or on behalf of, the
- Licensor for the purpose of discussing and improving the Work, but
- excluding communication that is conspicuously marked or otherwise
- designated in writing by the copyright owner as "Not a Contribution."
-
- "Contributor" shall mean Licensor and any individual or Legal Entity
- on behalf of whom a Contribution has been received by Licensor and
- subsequently incorporated within the Work.
-
- 2. Grant of Copyright License. Subject to the terms and conditions of
- this License, each Contributor hereby grants to You a perpetual,
- worldwide, non-exclusive, no-charge, royalty-free, irrevocable
- copyright license to reproduce, prepare Derivative Works of,
- publicly display, publicly perform, sublicense, and distribute the
- Work and such Derivative Works in Source or Object form.
-
- 3. Grant of Patent License. Subject to the terms and conditions of
- this License, each Contributor hereby grants to You a perpetual,
- worldwide, non-exclusive, no-charge, royalty-free, irrevocable
- (except as stated in this section) patent license to make, have made,
- use, offer to sell, sell, import, and otherwise transfer the Work,
- where such license applies only to those patent claims licensable
- by such Contributor that are necessarily infringed by their
- Contribution(s) alone or by combination of their Contribution(s)
- with the Work to which such Contribution(s) was submitted. If You
- institute patent litigation against any entity (including a
- cross-claim or counterclaim in a lawsuit) alleging that the Work
- or a Contribution incorporated within the Work constitutes direct
- or contributory patent infringement, then any patent licenses
- granted to You under this License for that Work shall terminate
- as of the date such litigation is filed.
-
- 4. Redistribution. You may reproduce and distribute copies of the
- Work or Derivative Works thereof in any medium, with or without
- modifications, and in Source or Object form, provided that You
- meet the following conditions:
-
- (a) You must give any other recipients of the Work or
- Derivative Works a copy of this License; and
-
- (b) You must cause any modified files to carry prominent notices
- stating that You changed the files; and
-
- (c) You must retain, in the Source form of any Derivative Works
- that You distribute, all copyright, patent, trademark, and
- attribution notices from the Source form of the Work,
- excluding those notices that do not pertain to any part of
- the Derivative Works; and
-
- (d) If the Work includes a "NOTICE" text file as part of its
- distribution, then any Derivative Works that You distribute must
- include a readable copy of the attribution notices contained
- within such NOTICE file, excluding those notices that do not
- pertain to any part of the Derivative Works, in at least one
- of the following places: within a NOTICE text file distributed
- as part of the Derivative Works; within the Source form or
- documentation, if provided along with the Derivative Works; or,
- within a display generated by the Derivative Works, if and
- wherever such third-party notices normally appear. The contents
- of the NOTICE file are for informational purposes only and
- do not modify the License. You may add Your own attribution
- notices within Derivative Works that You distribute, alongside
- or as an addendum to the NOTICE text from the Work, provided
- that such additional attribution notices cannot be construed
- as modifying the License.
-
- You may add Your own copyright statement to Your modifications and
- may provide additional or different license terms and conditions
- for use, reproduction, or distribution of Your modifications, or
- for any such Derivative Works as a whole, provided Your use,
- reproduction, and distribution of the Work otherwise complies with
- the conditions stated in this License.
-
- 5. Submission of Contributions. Unless You explicitly state otherwise,
- any Contribution intentionally submitted for inclusion in the Work
- by You to the Licensor shall be under the terms and conditions of
- this License, without any additional terms or conditions.
- Notwithstanding the above, nothing herein shall supersede or modify
- the terms of any separate license agreement you may have executed
- with Licensor regarding such Contributions.
-
- 6. Trademarks. This License does not grant permission to use the trade
- names, trademarks, service marks, or product names of the Licensor,
- except as required for reasonable and customary use in describing the
- origin of the Work and reproducing the content of the NOTICE file.
-
- 7. Disclaimer of Warranty. Unless required by applicable law or
- agreed to in writing, Licensor provides the Work (and each
- Contributor provides its Contributions) on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
- implied, including, without limitation, any warranties or conditions
- of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
- PARTICULAR PURPOSE. You are solely responsible for determining the
- appropriateness of using or redistributing the Work and assume any
- risks associated with Your exercise of permissions under this License.
-
- 8. Limitation of Liability. In no event and under no legal theory,
- whether in tort (including negligence), contract, or otherwise,
- unless required by applicable law (such as deliberate and grossly
- negligent acts) or agreed to in writing, shall any Contributor be
- liable to You for damages, including any direct, indirect, special,
- incidental, or consequential damages of any character arising as a
- result of this License or out of the use or inability to use the
- Work (including but not limited to damages for loss of goodwill,
- work stoppage, computer failure or malfunction, or any and all
- other commercial damages or losses), even if such Contributor
- has been advised of the possibility of such damages.
-
- 9. Accepting Warranty or Additional Liability. While redistributing
- the Work or Derivative Works thereof, You may choose to offer,
- and charge a fee for, acceptance of support, warranty, indemnity,
- or other liability obligations and/or rights consistent with this
- License. However, in accepting such obligations, You may act only
- on Your own behalf and on Your sole responsibility, not on behalf
- of any other Contributor, and only if You agree to indemnify,
- defend, and hold each Contributor harmless for any liability
- incurred by, or claims asserted against, such Contributor by reason
- of your accepting any such warranty or additional liability.
-
- END OF TERMS AND CONDITIONS
-*/
diff --git a/third_party/screen-ai/OWNERS b/third_party/screen-ai/OWNERS
deleted file mode 100644
index 19b1eb5..0000000
--- a/third_party/screen-ai/OWNERS
+++ /dev/null
@@ -1,2 +0,0 @@
-rhalavati@chromium.org
-kyungjunlee@google.com
diff --git a/third_party/screen-ai/README.chromium b/third_party/screen-ai/README.chromium
deleted file mode 100644
index cfc01f6..0000000
--- a/third_party/screen-ai/README.chromium
+++ /dev/null
@@ -1,35 +0,0 @@
-Name: ScreenAI Library
-Short Name: screen-ai
-URL: https://source.corp.google.com/piper///depot/google3/chrome/accessibility/machine_intelligence/chrome_screen_ai/README.md
-Version: 121.2
-Shipped: yes
-Security Critical: no
-
-Description:
-Provides ML models for Reading Mode and PDF OCR. Chrome downloads the binary of
-this library via Component Updater on Windows, Linux, and macOS, but Component
-Updater is not supported in browser tests. Only for browser tests on those
-platforms, the binary will be downloaded from CIPD and used in tests.
-
-Note that this binary is shipped to Chrome via Component Updater from
-(only Googler can access the following) go/chrome_screen_ai_source on Windows,
-Linux, and macOS, but metadata for credits is kept in this directory for license
-attribution.
-
--------------------- DEPENDENCY DIVIDER --------------------
-
-Name: Eigen
-Short Name: eigen
-URL: https://gitlab.com/libeigen/eigen
-Version: 3.4
-License: Apache 2.0
-License File: LICENSE
-Shipped: yes
-Security Critical: no
-
-Description:
-Eigen is a C++ template library for linear algebra, matrices, vectors,
-numerical solvers, and related algorithms.
-
-Local Modifications:
-(none)
diff --git a/ui/accessibility/accessibility_features.cc b/ui/accessibility/accessibility_features.cc
index b84eefd..87be86b 100644
--- a/ui/accessibility/accessibility_features.cc
+++ b/ui/accessibility/accessibility_features.cc
@@ -355,14 +355,6 @@
bool IsScreenAIDebugModeEnabled() {
return base::FeatureList::IsEnabled(::features::kScreenAIDebugMode);
}
-
-// This feature is only used in tests and must not be enabled by default.
-BASE_FEATURE(kScreenAITestMode,
- "ScreenAITestMode",
- base::FEATURE_DISABLED_BY_DEFAULT);
-bool IsScreenAITestModeEnabled() {
- return base::FeatureList::IsEnabled(::features::kScreenAITestMode);
-}
#endif // !BUILDFLAG(IS_ANDROID)
#if BUILDFLAG(IS_MAC)
diff --git a/ui/accessibility/accessibility_features.h b/ui/accessibility/accessibility_features.h
index fb8008d..be6718e 100644
--- a/ui/accessibility/accessibility_features.h
+++ b/ui/accessibility/accessibility_features.h
@@ -264,11 +264,6 @@
// Write some ScreenAI library debug data in /tmp.
AX_BASE_EXPORT BASE_DECLARE_FEATURE(kScreenAIDebugMode);
AX_BASE_EXPORT bool IsScreenAIDebugModeEnabled();
-
-// Enables to use the Screen AI component available for testing.
-// If enabled, ScreenAI library will be loaded from //third_party/screen-ai.
-AX_BASE_EXPORT BASE_DECLARE_FEATURE(kScreenAITestMode);
-AX_BASE_EXPORT bool IsScreenAITestModeEnabled();
#endif // !BUILDFLAG(IS_ANDROID)
#if BUILDFLAG(IS_MAC)
