commit 70dca51275cda237a1feed1dd35f32b31bc53e5a
author Roberto Lublinerman <rluble@google.com> Fri Jun 17 18:05:12 2016 -0700
committer Roberto Lublinerman <rluble@google.com> Sat Jun 18 01:30:12 2016 +0000
tree 95b4c9a8277f1e4e3183c2c9eb07238628b99cb5
parent ffad42933f2389b4e37ee3f24563b8741d7ccbcd

Fix failing tests due to JsFunctions classes requiring final modifier.

The tests were not updated in patch 
https://gwt-review.googlesource.com/#/c/15190, but should have been.

Change-Id: I81f45a2bc5dad1c2f5ed7b9ccba6f87166edbc7e

dev/core/test/com/google/gwt/dev/CompilerTest.java

diff --git a/dev/core/test/com/google/gwt/dev/CompilerTest.java b/dev/core/test/com/google/gwt/dev/CompilerTest.java
index 137e2ff..90c649f 100644
--- a/dev/core/test/com/google/gwt/dev/CompilerTest.java
+++ b/dev/core/test/com/google/gwt/dev/CompilerTest.java
@@ -993,7 +993,7 @@
         JavaResourceBase.createMockJavaResource(
             "com.foo.Impl",
             "package com.foo;",
-            "public class Impl implements SomeJsFunction {",
+            "public final class Impl implements SomeJsFunction {",
             "  public void m() { SomeInterface.class.getName(); } ",
             "}");
 
@@ -1146,7 +1146,7 @@
         JavaResourceBase.createMockJavaResource(
             "com.foo.Foo",
             "package com.foo;",
-            "public class Foo implements IFoo {",
+            "public final class Foo implements IFoo {",
             "  @Override public int foo(int x) { return 0; }",
             "}");
 

user/test/com/google/gwt/core/interop/JsTypeTest.java

diff --git a/user/test/com/google/gwt/core/interop/JsTypeTest.java b/user/test/com/google/gwt/core/interop/JsTypeTest.java
index bc67cc4..5b7fba7 100644
--- a/user/test/com/google/gwt/core/interop/JsTypeTest.java
+++ b/user/test/com/google/gwt/core/interop/JsTypeTest.java
@@ -458,7 +458,7 @@
     int m();
   }
 
-  static class JavaConcreteJsFunction implements JsFunctionInterface {
+  static final class JavaConcreteJsFunction implements JsFunctionInterface {
     public int m() {
       return 5;
     }

user/test/com/google/gwt/core/interop/JsTypeVarargsTest.java

diff --git a/user/test/com/google/gwt/core/interop/JsTypeVarargsTest.java b/user/test/com/google/gwt/core/interop/JsTypeVarargsTest.java
index 81b95c6..c9a70d0 100644
--- a/user/test/com/google/gwt/core/interop/JsTypeVarargsTest.java
+++ b/user/test/com/google/gwt/core/interop/JsTypeVarargsTest.java
@@ -250,7 +250,7 @@
     Object f(int i, Object... args);
   }
 
-  static class AFunction implements Function {
+  static final class AFunction implements Function {
 
     @Override
     public Object f(int i, Object... args) {

user/test/com/google/gwt/core/interop/MyJsFunctionInterfaceImpl.java

diff --git a/user/test/com/google/gwt/core/interop/MyJsFunctionInterfaceImpl.java b/user/test/com/google/gwt/core/interop/MyJsFunctionInterfaceImpl.java
index 126ede3..7400d1f 100644
--- a/user/test/com/google/gwt/core/interop/MyJsFunctionInterfaceImpl.java
+++ b/user/test/com/google/gwt/core/interop/MyJsFunctionInterfaceImpl.java
@@ -18,7 +18,7 @@
 /**
  * A concrete class that implements a JsFunction interface.
  */
-public class MyJsFunctionInterfaceImpl implements MyJsFunctionInterface {
+public final class MyJsFunctionInterfaceImpl implements MyJsFunctionInterface {
 
   public int publicField = 10;
 

user/test/com/google/gwt/core/interop/NativeJsTypeTest.java

diff --git a/user/test/com/google/gwt/core/interop/NativeJsTypeTest.java b/user/test/com/google/gwt/core/interop/NativeJsTypeTest.java
index ff4d910..367d65b 100644
--- a/user/test/com/google/gwt/core/interop/NativeJsTypeTest.java
+++ b/user/test/com/google/gwt/core/interop/NativeJsTypeTest.java
@@ -255,7 +255,7 @@
     void m();
   }
 
-  static class SomeFunction implements SomeFunctionInterface {
+  static final class SomeFunction implements SomeFunctionInterface {
     public void m() {
     }
   }