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

Side by Side Diff: chrome/test/data/cld2_component/README.chromium

Issue 289313004: Add test data for dynamic CLD. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
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 unified diff | Download patch
« no previous file with comments | « chrome/test/data/cld2_component/160/manifest.json ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 This directory contains a static copy of the contents of the CLD2 CRX file.
2 The ID of the extension (as well as other relevant information) can be found
3 in the following file:
4 src/chrome/browser/component_updater/cld_component_installer.cc
5
6 In general there should be little need to modify the data here. CLD2 updates
7 are rare and consist of incremental improvements to language detection
8 accuracy. All such updates are expected to be backwards-compatible from CLD2
9 revision 160 onwards, which is what the initial checkin her is in fact based
rmcilroy 2014/05/19 13:16:02 nit - s/which is what the initial checkin her is i
10 upon. Any future format-breaking changes are likely to result in a "CLD3".
11
12 For the sake of maintainability and clarity of purpose, the subdirectory that
13 contains the CRX extract is named for the version that is contained within it.
14 This mimics the behavior of the component updater and makes it obvious at a
15 glance which version of the CRX is checked in here; if you DO update the
16 content here, make sure to rename the directory and update the constant in the
rmcilroy 2014/05/19 13:16:02 s/make sure to rename the directory/create a new d
17 test utilities class:
18 src/chrome/browser/translate/translate_browser_test_utils.cc
19
20 For more information about this directory and how to use it, contact the author:
21 andrewhayden@chromium.org
rmcilroy 2014/05/19 13:16:02 nit - I would drop this and add a "or contact some
22
23 for more information on CLD2, dynamic mode, and generating the data file see:
rmcilroy 2014/05/19 13:16:02 /s/for/For
24 src/third_party/cld_2/README.chromium
25
OLDNEW
« no previous file with comments | « chrome/test/data/cld2_component/160/manifest.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698