Merge "Enable quirk detector for 📷camera-viewfinder" into androidx-main

Author: Treehugger Robot

camera/camera-viewfinder/build.gradle

@@ -66,6 +66,10 @@ android {
     }
 
     testOptions.unitTests.includeAndroidResources = true
+
+    lintOptions {
+        enable 'CameraXQuirksClassDetector'
+    }
 }
 
 androidx {

camera/camera-viewfinder/src/main/java/androidx/camera/viewfinder/internal/quirk/QuirkSummary.java

@@ -23,9 +23,13 @@
 /**
  * A quirk where a scaled up SurfaceView is not cropped by the parent View.
  *
- * <p> On certain Xiaomi devices, when the scale type is FILL_* and the preview is scaled up
- * to be larger than its parent, the SurfaceView is not cropped by its parent. As the result, the
- * preview incorrectly covers the neighboring UI elements. b/211370840
+ * <p>QuirkSummary
+ *     Bug Id: 211370840
+ *     Description: On certain Xiaomi devices, when the scale type is FILL_* and the preview is
+ *                  scaled up to be larger than its parent, the SurfaceView is not cropped by its
+ *                  parent. As the result, the preview incorrectly covers the neighboring UI
+ *                  elements.
+ *     Device(s): XIAOMI M2101K7AG
  */
 @RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
 public class SurfaceViewNotCroppedByParentQuirk implements Quirk {

camera/camera-viewfinder/src/main/java/androidx/camera/viewfinder/internal/quirk/SurfaceViewNotCroppedByParentQuirk.java

@@ -23,8 +23,11 @@
 /**
  * A quirk where SurfaceView is stretched.
  *
- * <p> On certain Samsung devices, transform APIs (e.g. View#setScaleX) do not work as intended.
- * b/129403806
+ * <p>QuirkSummary
+ *     Bug Id: 129403806
+ *     Description: On certain Samsung devices, transform APIs (e.g. View#setScaleX) do not work
+ *                  as intended.
+ *     Device(s): SAMSUNG F2Q Q2Q
  */
 @RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
 public class SurfaceViewStretchedQuirk implements Quirk {

camera/camera-viewfinder/src/main/java/androidx/camera/viewfinder/internal/quirk/SurfaceViewStretchedQuirk.java

@@ -54,9 +54,12 @@ class CameraXQuirksClassDetector : Detector(), Detector.UastScanner {
                 comments.forEach { sb.append(it.text) }
                 val comment = sb.append("\n").toString()
 
-                if (!comment.contains("@QuirkSummary")) {
+                if (!comment.contains("<p>QuirkSummary") ||
+                    !comment.contains("Bug Id:") ||
+                    !comment.contains("Description:") ||
+                    !comment.contains("Device(s):")) {
                     val implForInsertion = """
-                         * @QuirkSummary
+                         * <p>QuirkSummary
                          *     Bug Id:
                          *     Description:
                          *     Device(s):

lint-checks/src/main/java/androidx/build/lint/CameraXQuirksClassDetector.kt

@@ -38,7 +38,7 @@ class CameraXQuirksClassDetectorTest : AbstractLintDetectorTest(
         val expected = """
             src/androidx/CameraXMissingQuirkSummaryJava.java:22: Error: CameraX quirks should include this template in the javadoc:
 
-            * @QuirkSummary
+            * <p>QuirkSummary
             *     Bug Id:
             *     Description:
             *     Device(s):