Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5543)

Unified Diff: chrome/test/data/cld2_component/README.chromium

Issue 285293004: Allow browser tests to run with dynamic CLD data. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cld_uma
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/cld2_component/README.chromium
diff --git a/chrome/test/data/cld2_component/README.chromium b/chrome/test/data/cld2_component/README.chromium
new file mode 100644
index 0000000000000000000000000000000000000000..51a79de286fe639fcdc75d9142f6e774823023fb
--- /dev/null
+++ b/chrome/test/data/cld2_component/README.chromium
@@ -0,0 +1,21 @@
+This directory contains a static copy of the contents of the CLD2 CRX file.
+The ID of the extension (as well as other relevant information) can be found
+in the following file:
+ src/chrome/browser/component_updater/cld_component_installer.cc
+
+In general there should be little need to modify the data here. CLD2 updates
+are rare and consist of incremental improvements to language detection
+accuracy. All such updates are expected to be backwards-compatible from CLD2
+revision 160 onwards, which is what the initial checkin her is in fact based
+upon. Any future format-breaking changes are likely to result in a "CLD3".
+
+For the sake of maintainability and clarity of purpose, the subdirectory that
+contains the CRX extract is named for the version that is contained within it.
+This mimics the behavior of the component updater and makes it obvious at a
+glance which version of the CRX is checked in here; if you DO update the
+content here, make sure to rename the directory and update the constant in the
+test utilities class:
+ src/chrome/browser/translate/translate_browser_test_utils.cc
+
+For more information about this directory and how to use it, contact the author:
+ andrewhayden@chromium.org

Powered by Google App Engine
This is Rietveld 408576698