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

Unified Diff: components/url_formatter/top_domains/README

Issue 2784933002: Mitigate spoofing attempt using Latin letters. (Closed)
Patch Set: add back U+04CF (ÓŹ) -> 'l' map Created 3 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: components/url_formatter/top_domains/README
diff --git a/components/url_formatter/top_domains/README b/components/url_formatter/top_domains/README
new file mode 100644
index 0000000000000000000000000000000000000000..5eafd5eb660b1233b3b8ec17703e6abcff3e5793
--- /dev/null
+++ b/components/url_formatter/top_domains/README
@@ -0,0 +1,20 @@
+* alexa_domains.list
+ It is an input to make_top_domain_list and is made up of list of Alexa
Peter Kasting 2017/05/09 01:37:02 Nit: "The Alexa top 10k domains, one per line, con
jungshik at Google 2017/05/10 18:05:12 Thanks. Done
+ top 10k domains (one per line).
+ It's derived from
+ src/tools/perf/page_sets/alexa1-10000-urls.json with make_alexa_top_list.py
+
+* alexa_names_and_skeletons.gperf
+
+ It is generated by running make_top_domain_list and checked in.
Peter Kasting 2017/05/09 01:37:02 Nit: "The checked-in output of make_top_domain_gpe
jungshik at Google 2017/05/10 18:05:12 Thank you for a much better rewrite.
+ No command line argument needs to be passed.
+
+ When ICU is updated to a new major version, this file needs to be regenerated
+ because skeletons can be different in different ICU versions.
+
+ $ ninja -C $build_outdir make_top_domain_gperf
+ $ $build_outdir/make_top_domain_gperf
+
+ During a build, it is processed by net/tools/dafsa/make_dafsa.py to generate
+ alexa_names_and_skeletons-inc.cc that is included by
+ components/url_formatter/url_formatter.cc

Powered by Google App Engine
This is Rietveld 408576698