Recommend the JavaTimeConversion static conversion methods instead of decomposition-based conversions.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=313601612

Author: kluever

core/src/main/java/com/google/errorprone/bugpatterns/time/PreferJavaTimeOverload.java

@@ -256,8 +256,6 @@ public Description matchMethodInvocation(MethodInvocationTree tree, VisitorState
           return describeMatch(tree, fix.build());
         }
 
-        // We could suggest using JavaTimeConversions.toJavaDuration(jodaDuration), but that
-        // requires an additional dependency and isn't open-sourced.
         fix.replace(
             arg0,
             String.format(
@@ -293,8 +291,6 @@ public Description matchMethodInvocation(MethodInvocationTree tree, VisitorState
           return describeMatch(tree, fix.build());
         }
 
-        // We could suggest using JavaTimeConversions.toJavaInstant(jodaInstant), but that
-        // requires an additional dependency and isn't open-sourced.
         fix.replace(
             arg0,
             String.format(

core/src/test/java/com/google/errorprone/bugpatterns/time/PreferJavaTimeOverloadTest.java

@@ -186,6 +186,25 @@ public void callingJodaDurationMethodWithDurationOverload_privateMethod() {
         .doTest();
   }
 
+  @Test
+  public void callingJodaInstantMethodWithInstantOverload_privateMethod() {
+    helper
+        .addSourceLines(
+            "TestClass.java",
+            "import java.time.Instant;",
+            "public class TestClass {",
+            "  private void bar(org.joda.time.Instant i) {",
+            "  }",
+            "  private void bar(Instant i) {",
+            "  }",
+            "  public void foo(org.joda.time.Instant jodaInstant) {",
+            "    // BUG: Diagnostic contains: bar(Instant.ofEpochMilli(jodaInstant.getMillis()));",
+            "    bar(jodaInstant);",
+            "  }",
+            "}")
+        .doTest();
+  }
+
   @Test
   public void callingJodaDurationMethodWithDurationOverload_privateMethod_jodaDurationMillis() {
     helper
@@ -296,19 +315,19 @@ public void callingJodaDurationMethodWithSupertypeJavaDurationOverload() {
   }
 
   @Test
-  public void callingJodaReadableDurationMethodWithDurationOverload_privateMethod() {
+  public void callingJodaReadableInstantMethodWithInstantOverload_privateMethod() {
     helper
         .addSourceLines(
             "TestClass.java",
-            "import java.time.Duration;",
+            "import java.time.Instant;",
             "public class TestClass {",
-            "  private void bar(org.joda.time.ReadableDuration d) {",
+            "  private void bar(org.joda.time.ReadableInstant i) {",
             "  }",
-            "  private void bar(Duration d) {",
+            "  private void bar(Instant i) {",
             "  }",
-            "  public void foo(org.joda.time.Duration jodaDuration) {",
-            "    // BUG: Diagnostic contains: bar(Duration.ofMillis(jodaDuration.getMillis()));",
-            "    bar(jodaDuration);",
+            "  public void foo(org.joda.time.Instant jodaInstant) {",
+            "    // BUG: Diagnostic contains: bar(Instant.ofEpochMilli(jodaInstant.getMillis()));",
+            "    bar(jodaInstant);",
             "  }",
             "}")
         .doTest();