123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184 |
- <?xml version="1.0" encoding="UTF-8"?>
- <!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.3//EN" "http://www.puppycrawl.com/dtds/configuration_1_3.dtd">
- <!--
- See installation instructions: https://sites.google.com/a/chromium.org/dev/developers/checkstyle
- -->
- <module name="Checker">
- <property name="severity" value="warning"/>
- <property name="charset" value="UTF-8"/>
- <module name="SuppressionFilter">
- <property name="file" value="tools/android/checkstyle/suppressions.xml"/>
- </module>
- <module name="SuppressWarningsFilter"/>
- <module name="TreeWalker">
- <module name="SuppressWarningsHolder"/>
- <module name="AvoidStarImport">
- <property name="severity" value="error"/>
- </module>
- <module name="IllegalCatch">
- <property name="severity" value="info"/>
- </module>
- <module name="ModifierOrder">
- <message key="mod.order" value=""{0}" modifier out of order with the JLS suggestions. The correct order: public protected private abstract default static final transient volatile synchronized native strictfp. See https://crbug.com/1003711 for details."/>
- <property name="severity" value="error"/>
- </module>
- <module name="RedundantImport">
- <message key="import.redundant" value="Redundant import: {0}."/>
- <property name="severity" value="error"/>
- </module>
- <module name="UnusedImports">
- <message key="import.unused" value="Unused import: {0}."/>
- <property name="severity" value="error"/>
- <property name="processJavadoc" value="true"/>
- </module>
- <module name="JavadocType">
- <property name="severity" value="warning"/>
- <property name="tokens" value="INTERFACE_DEF, CLASS_DEF"/>
- <property name="scope" value="public"/>
- <property name="allowedAnnotations" value="NativeMethods"/>
- <message key="javadoc.missing" value="Public classes and interfaces require JavaDoc comments."/>
- </module>
- <module name="JavadocMethod">
- <property name="severity" value="info"/>
- <property name="scope" value="public"/>
- <property name="allowMissingParamTags" value="true"/>
- <property name="allowMissingPropertyJavadoc" value="true"/>
- <property name="allowMissingReturnTag" value="true"/>
- <property name="allowMissingThrowsTags" value="true"/>
- </module>
- <module name="PackageName">
- <property name="severity" value="error"/>
- <property name="format" value="^[a-z]+(\.[a-z][a-z0-9_]{1,})*$"/>
- </module>
- <module name="SimplifyBooleanExpression">
- <property name="severity" value="error"/>
- </module>
- <module name="SimplifyBooleanReturn">
- <property name="severity" value="error"/>
- </module>
- <module name="TypeName">
- <property name="severity" value="error"/>
- </module>
- <module name="ConstantName">
- <property name="severity" value="error"/>
- <property name="format" value="^([A-Z][A-Z0-9]*(_[A-Z0-9]+)*)|(s[A-Z][a-zA-Z0-9]*)$"/>
- <message key="name.invalidPattern" value="Static final field names must either be all caps (e.g. int HEIGHT_PX) for 'true' constants, or start with s (e.g. AtomicInteger sNextId or Runnable sSuspendTask) for fields with mutable state or that don't 'feel' like constants."/>
- </module>
- <!-- Non-public, non-static field names start with m. -->
- <module name="MemberName">
- <property name="severity" value="error"/>
- <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/>
- <property name="applyToPublic" value="false"/>
- <message key="name.invalidPattern" value="Non-public, non-static field names start with m."/>
- </module>
- <!-- Static field names start with s. -->
- <module name="StaticVariableName">
- <property name="severity" value="error"/>
- <property name="format" value="^s[A-Z][a-zA-Z0-9]*$"/>
- <property name="applyToPublic" value="false"/>
- <message key="name.invalidPattern" value="Static field names start with s."/>
- </module>
- <module name="MethodName">
- <property name="id" value="MethodNameCheck"/>
- <property name="severity" value="error"/>
- <property name="format" value="^(([a-z][a-zA-Z0-9_]*)|(__constructor__))$"/>
- <message key="name.invalidPattern" value="Method names should start with a lower case letter (e.g. getWidth())"/>
- </module>
- <module name="ParameterName">
- <property name="severity" value="error"/>
- </module>
- <module name="LocalFinalVariableName">
- <property name="severity" value="error"/>
- <property name="format" value="^m|s|((([ms][a-z0-9])|([a-ln-rt-z]))[a-zA-Z0-9]*)$"/>
- <message key="name.invalidPattern" value="Local variables should be camel-cased (e.g. int minWidth = 4)."/>
- </module>
- <module name="LocalVariableName">
- <property name="severity" value="error"/>
- <property name="format" value="^m|s|((([ms][a-z0-9])|([a-ln-rt-z]))[a-zA-Z0-9]*)$"/>
- <message key="name.invalidPattern" value="Local variables should be camel-cased (e.g. int minWidth = 4)."/>
- </module>
- <module name="FallThrough">
- <property name="severity" value="error"/>
- <property name="reliefPattern" value=".*"/>
- </module>
- <module name="NoFinalizer">
- <property name="severity" value="info"/>
- </module>
- <module name="NeedBraces">
- <property name="severity" value="error"/>
- <property name="allowSingleLineStatement" value="true"/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="StringBufferCheck"/>
- <property name="severity" value="error"/>
- <property name="format" value="StringBuffer"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Avoid StringBuffer; use StringBuilder instead, which is faster (it's not thread-safe, but this is almost never needed)"/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="AlertDialogCheck"/>
- <property name="severity" value="warning"/>
- <property name="format" value="android\.app\.AlertDialog"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Avoid android.app.AlertDialog; if possible, use android.support.v7.app.AlertDialog instead, which has a Material look on all devices. (Some parts of the codebase can’t depend on the support library, in which case android.app.AlertDialog is the only option)"/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="UseSharedPreferencesManagerFromChromeCheck"/>
- <property name="severity" value="warning"/>
- <property name="format" value="(getAppSharedPreferences|getDefaultSharedPreferences)"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Use SharedPreferencesManager instead to access app-wide SharedPreferences from //src/chrome."/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="DefaultSharedPreferencesCheck"/>
- <property name="severity" value="error"/>
- <property name="format" value="getDefaultSharedPreferences"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Use ContextUtils.getAppSharedPreferences() instead to access app-wide SharedPreferences."/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="ConcurrentHashMapCheck"/>
- <property name="severity" value="error"/>
- <property name="format" value="ConcurrentHashMap"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="ConcurrentHashMap has a bug on some Android versions. Use Collections.synchronizedMap() instead (crbug.com/905903)."/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="StrictModeContextIgnoredCheck"/>
- <property name="severity" value="error"/>
- <property name="format" value="try .*StrictModeContext (?!ignored).*="/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Please name the StrictModeContext variable "ignored" (crbug.com/767058#c4)."/>
- </module>
- <module name="RegexpSinglelineJava">
- <!-- Tests are also allowed to use System.exit if need be, please add dirs to tools/android/checkstyle/suppressions.xml -->
- <property name="id" value="SystemExitCheck"/>
- <property name="severity" value="error"/>
- <property name="format" value="System\.exit\("/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Throw an exception instead of calling System#exit (crbug.com/1000651)."/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="AndroidButtonWidgetCheck"/>
- <property name="severity" value="warning"/>
- <property name="format" value="extends\ Button|extends\ AppCompatButton"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Use org.chromium.ui.widget.ButtonCompat instead of Button and AppCompatButton if possible; ButtonCompat is a Material-styled button with a customizable background color. On L devices, this is a true Material button. On earlier devices, the button is similar but lacks ripples and a shadow. (crbug.com/775198)"/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="SetTextColorAndSetTextSizeCheck"/>
- <property name="severity" value="warning"/>
- <property name="format" value="\.setTextColor\(|\.setTextSize\("/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="If possible, please use #setTextAppearance to set one of the pre-defined text appearance styles in src/ui/android/java/res/values-v17/styles.xml, instead of using #setTextColor or #setTextSize. This will ensure that your programmatically set text styles stay in-sync with text styles used in the rest of Chrome. (crbug.com/775198)"/>
- </module>
- <module name="RegexpSinglelineJava">
- <property name="id" value="ActivityScenarioLaunch"/>
- <property name="severity" value="warning"/>
- <property name="format" value="ActivityScenario\.launch"/>
- <property name="ignoreComments" value="true"/>
- <property name="message" value="Launching an ActivityScenario manually is error prone for any test run in a batch. Failing to #close will cause the Activity to leak into the next test. Consider replacing with ActivityScenarioRule. (https://crbug.com/1338271)"/>
- </module>
- </module>
- </module>
|