For mozilla-mobile#10555 - Fix InputResultDetail hashcode() collisions

The previous implementation was affected by the fact that the class has 3
integer properties, all with values [0..4] (-1 was added recently) and it
would just add those value to compute the hashcode.
Collisions were a given.

By using decimal places for each property the new implementation should avoid
collisions while allowing for all the other expected guarantees.

components/concept/engine/src/main/java/mozilla/components/concept/engine/InputResultDetail.kt

@@ -134,9 +134,9 @@ class InputResultDetail private constructor(
 
     @Suppress("MagicNumber")
     override fun hashCode(): Int {
-        var hash = inputResult.hashCode() * 31
-        hash += (scrollDirections.hashCode()) * 31
-        hash += (overscrollDirections.hashCode()) * 31
+        var hash = inputResult.hashCode()
+        hash += (scrollDirections.hashCode()) * 10
+        hash += (overscrollDirections.hashCode()) * 100
 
         return hash
     }

components/concept/engine/src/test/java/mozilla/components/concept/engine/InputResultDetailTest.kt

@@ -115,15 +115,15 @@ class InputResultDetailTest {
         assertTrue(inputResultDetail == inputResultDetail)
     }
 
-    // @Test
-    // fun `GIVEN an InputResultDetail WHEN hashCode is called for same values objects THEN it returns the same result`() {
-    //     assertEquals(inputResultDetail.hashCode(), inputResultDetail.hashCode())
-    //
-    //     assertEquals(inputResultDetail.hashCode(), InputResultDetail.newInstance().hashCode())
-    //
-    //     inputResultDetail = inputResultDetail.copy(OVERSCROLL_DIRECTIONS_VERTICAL)
-    //     assertEquals(inputResultDetail.hashCode(), InputResultDetail.newInstance(true).hashCode())
-    // }
+    @Test
+    fun `GIVEN an InputResultDetail WHEN hashCode is called for same values objects THEN it returns the same result`() {
+        assertEquals(inputResultDetail.hashCode(), inputResultDetail.hashCode())
+
+        assertEquals(inputResultDetail.hashCode(), InputResultDetail.newInstance().hashCode())
+
+        inputResultDetail = inputResultDetail.copy(overscrollDirections = OVERSCROLL_DIRECTIONS_VERTICAL)
+        assertEquals(inputResultDetail.hashCode(), InputResultDetail.newInstance(true).hashCode())
+    }
 
     @Test
     fun `GIVEN an InputResultDetail WHEN hashCode is called for different values objects THEN it returns different results`() {