### Eclipse Workspace Patch 1.0
#P org.eclipse.jdt.core.tests.compiler
Index: src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java
===================================================================
RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java,v
retrieving revision 1.36
diff -u -r1.36 GenericsCompletionParserTest.java
--- src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java	27 Jun 2008 16:04:46 -0000	1.36
+++ src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java	26 Feb 2009 12:43:16 -0000
@@ -39,7 +39,7 @@
 	String completeBehind = "Z<Y>.";
 	int cursorLocation = str.indexOf("Z<Y>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Z<Y>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Z<Y>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Z<Y>.";
 	String expectedUnitDisplayString =
@@ -67,7 +67,7 @@
 	String completeBehind = "Z<Y>.W";
 	int cursorLocation = str.indexOf("Z<Y>.W") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Z<Y>.W>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Z<Y>.W>";
 	String completionIdentifier = "W";
 	String expectedReplacedSource = "Z<Y>.W";
 	String expectedUnitDisplayString =
@@ -95,7 +95,7 @@
 	String completeBehind = "X<Y>.Z";
 	int cursorLocation = str.indexOf("X<Y>.Z") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:test0001.X<Y>.Z>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:test0001.X<Y>.Z>";
 	String completionIdentifier = "Z";
 	String expectedReplacedSource = "test0001.X<Y>.Z";
 	String expectedUnitDisplayString =
@@ -428,7 +428,11 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnClass:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = 
+		"public class X extends <CompleteOnClass:Y<Z>.> {\n" + 
+		"  public X() {\n" + 
+		"  }\n" + 
+		"}";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -623,7 +627,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -654,7 +658,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -685,7 +689,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -715,7 +719,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -745,7 +749,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -777,7 +781,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -809,7 +813,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -841,7 +845,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -872,7 +876,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -902,7 +906,7 @@
 	String completeBehind = "Y<Z>.";
 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Y<Z>.";
 	String expectedUnitDisplayString =
@@ -7792,7 +7796,7 @@
 	String completeBehind = "Z";
 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Z>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:Z>";
 	String completionIdentifier = "Z";
 	String expectedReplacedSource = "Z";
 	String expectedUnitDisplayString =
@@ -7820,7 +7824,7 @@
 	String completeBehind = "Z";
 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:X.Z>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:X.Z>";
 	String completionIdentifier = "Z";
 	String expectedReplacedSource = "X.Z";
 	String expectedUnitDisplayString =
@@ -7848,7 +7852,7 @@
 	String completeBehind = "Z";
 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:X<W>.Z>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:X<W>.Z>";
 	String completionIdentifier = "Z";
 	String expectedReplacedSource = "X<W>.Z";
 	String expectedUnitDisplayString =
@@ -7877,7 +7881,7 @@
 	String completeBehind = "Z";
 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:X<W>.Z>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "T extends <CompleteOnType:X<W>.Z>";
 	String completionIdentifier = "Z";
 	String expectedReplacedSource = "X<W>.Z";
 	String expectedUnitDisplayString =
@@ -9057,7 +9061,7 @@
 	String completeBehind = "Strin";
 	int cursorLocation = str.indexOf("Strin") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Strin>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "? extends <CompleteOnType:Strin>";
 	String completionIdentifier = "Strin";
 	String expectedReplacedSource = "String";
 	String expectedUnitDisplayString =
@@ -10682,7 +10686,7 @@
 	String completeBehind = "Obj";
 	int cursorLocation = str.indexOf("Obj") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnType:Obj>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = "? extends <CompleteOnType:Obj>";
 	String completionIdentifier = "Obj";
 	String expectedReplacedSource = "Obj";
 	String expectedUnitDisplayString =
Index: src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java
===================================================================
RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java,v
retrieving revision 1.49
diff -u -r1.49 CompletionParserTest2.java
--- src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java	27 Jun 2008 16:04:46 -0000	1.49
+++ src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java	26 Feb 2009 12:43:11 -0000
@@ -9327,7 +9327,13 @@
 	String completeBehind = "Z.";
 	int cursorLocation = str.indexOf("Z.") + completeBehind.length() - 1;
 	String expectedCompletionNodeToString = "<CompleteOnClass:Z.>";
-	String expectedParentNodeToString = "<NONE>";
+	String expectedParentNodeToString = 
+		"public class X extends <CompleteOnClass:Z.> {\n" + 
+		"  {\n" + 
+		"  }\n" + 
+		"  public X() {\n" + 
+		"  }\n" + 
+		"}";
 	String completionIdentifier = "";
 	String expectedReplacedSource = "Z.";
 	String expectedUnitDisplayString =
#P org.eclipse.jdt.core
Index: codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java
===================================================================
RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java,v
retrieving revision 1.201
diff -u -r1.201 CompletionParser.java
--- codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java	16 Jan 2009 14:29:29 -0000	1.201
+++ codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java	26 Feb 2009 12:43:29 -0000
@@ -2076,6 +2076,11 @@
 protected void consumeClassHeaderExtends() {
 	pushOnElementStack(K_NEXT_TYPEREF_IS_CLASS);
 	super.consumeClassHeaderExtends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		TypeDeclaration typeDecl = (TypeDeclaration) this.astStack[this.astPtr];
+		if (typeDecl != null && typeDecl.superclass == this.assistNode)
+			this.assistNodeParent = typeDecl;
+	}
 	popElement(K_NEXT_TYPEREF_IS_CLASS);
 	popElement(K_EXTENDS_KEYWORD);
 
@@ -3752,18 +3757,38 @@
 }
 protected void consumeTypeParameterWithExtends() {
 	super.consumeTypeParameterWithExtends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
+		if (typeParameter != null && typeParameter.type == this.assistNode)
+			this.assistNodeParent = typeParameter;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeTypeParameterWithExtendsAndBounds() {
 	super.consumeTypeParameterWithExtendsAndBounds();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
+		if (typeParameter != null && typeParameter.type == this.assistNode)
+			this.assistNodeParent = typeParameter;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeTypeParameter1WithExtends() {
 	super.consumeTypeParameter1WithExtends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
+		if (typeParameter != null && typeParameter.type == this.assistNode)
+			this.assistNodeParent = typeParameter;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeTypeParameter1WithExtendsAndBounds() {
 	super.consumeTypeParameter1WithExtendsAndBounds();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
+		if (typeParameter != null && typeParameter.type == this.assistNode)
+			this.assistNodeParent = typeParameter;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeWildcard() {
@@ -3808,18 +3833,38 @@
 }
 protected void consumeWildcardBoundsExtends() {
 	super.consumeWildcardBoundsExtends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
+		if (wildcard != null && wildcard.bound == this.assistNode)
+			this.assistNodeParent = wildcard;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeWildcardBounds1Extends() {
 	super.consumeWildcardBounds1Extends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
+		if (wildcard != null && wildcard.bound == this.assistNode)
+			this.assistNodeParent = wildcard;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeWildcardBounds2Extends() {
 	super.consumeWildcardBounds2Extends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
+		if (wildcard != null && wildcard.bound == this.assistNode)
+			this.assistNodeParent = wildcard;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeWildcardBounds3Extends() {
 	super.consumeWildcardBounds3Extends();
+	if (this.assistNode != null && this.assistNodeParent == null) {
+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
+		if (wildcard != null && wildcard.bound == this.assistNode)
+			this.assistNodeParent = wildcard;
+	}
 	popElement(K_EXTENDS_KEYWORD);
 }
 protected void consumeUnaryExpression(int op) {
Index: codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java
===================================================================
RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java,v
retrieving revision 1.388
diff -u -r1.388 CompletionEngine.java
--- codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java	5 Feb 2009 09:38:12 -0000	1.388
+++ codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java	26 Feb 2009 12:43:26 -0000
@@ -482,6 +482,7 @@
 	private final static int NONE = 0;
 	private final static int SUPERTYPE = 1;
 	private final static int SUBTYPE = 2;
+	private final static int FINALTYPE = 4; // https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399
 	
 	int expectedTypesPtr = -1;
 	TypeBinding[] expectedTypes = new TypeBinding[1];
@@ -1113,6 +1114,7 @@
 		this.foundTypesCount++;
 		
 		if (this.options.checkDeprecation && (modifiers & ClassFileConstants.AccDeprecated) != 0) return;
+		if (((this.forbbidenBindingsFilter & FINALTYPE) != 0) && ((modifiers & ClassFileConstants.AccFinal) != 0)) return;
 
 		if (this.options.checkVisibility) {
 			if((modifiers & ClassFileConstants.AccPublic) == 0) {
@@ -3725,6 +3727,22 @@
 
 	private Scope computeForbiddenBindings(ASTNode astNode, ASTNode astNodeParent, Scope scope) {
 		this.forbbidenBindingsFilter = NONE;
+		// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399, don't propose final types for extension.
+		if (astNodeParent != null) {
+			if (astNodeParent instanceof TypeDeclaration) {
+				TypeDeclaration typeDeclaration = (TypeDeclaration) astNodeParent;
+				if (typeDeclaration.superclass == astNode)
+					this.forbbidenBindingsFilter = FINALTYPE;	
+			} else if (astNodeParent instanceof TypeParameter) {
+				TypeParameter typeParameter = (TypeParameter) astNodeParent;
+				if (typeParameter.type == astNode)
+					this.forbbidenBindingsFilter = FINALTYPE;
+			} else if (astNodeParent instanceof Wildcard) {
+				Wildcard wildcard = (Wildcard) astNodeParent;
+				if (wildcard.bound == astNode && wildcard.kind == Wildcard.EXTENDS)
+					this.forbbidenBindingsFilter = FINALTYPE;
+			}
+		}
 		if(scope instanceof ClassScope) {
 			TypeDeclaration typeDeclaration = ((ClassScope)scope).referenceContext;
 			if(typeDeclaration.superclass == astNode) {
@@ -7319,6 +7337,8 @@
 
 			if (this.options.checkDeprecation && memberType.isViewedAsDeprecated()) continue next;
 
+			if (isForbidden(memberType)) continue next;
+
 			if (this.options.checkVisibility
 				&& !memberType.canBeSeenBy(this.unitScope.fPackage))
 				continue next;
@@ -10271,6 +10291,9 @@
 						!scope.isDefinedInSameUnit(sourceType))
 					continue;
 
+				if (isForbidden(sourceType))
+					continue;
+
 				int accessibility = IAccessRule.K_ACCESSIBLE;
 				if(sourceType.hasRestrictedAccess()) {
 					AccessRestriction accessRestriction = this.lookupEnvironment.getAccessRestriction(sourceType);
@@ -10414,6 +10437,9 @@
 							!scope.isDefinedInSameUnit(refBinding))
 						continue next;
 
+					if (isForbidden(refBinding))
+						continue next;
+
 					int accessibility = IAccessRule.K_ACCESSIBLE;
 					if(refBinding.hasRestrictedAccess()) {
 						AccessRestriction accessRestriction = this.lookupEnvironment.getAccessRestriction(refBinding);
@@ -11596,6 +11622,11 @@
 				}
 			}
 		}
+		if((this.forbbidenBindingsFilter & FINALTYPE) != 0) {
+			if (binding instanceof ReferenceBinding && ((ReferenceBinding)binding).isFinal()) {
+				return true;
+			}
+		}
 		return false;
 	}
 
#P org.eclipse.jdt.core.tests.model
Index: src/org/eclipse/jdt/core/tests/model/CompletionTests.java
===================================================================
RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/CompletionTests.java,v
retrieving revision 1.194
diff -u -r1.194 CompletionTests.java
--- src/org/eclipse/jdt/core/tests/model/CompletionTests.java	4 Feb 2009 16:17:14 -0000	1.194
+++ src/org/eclipse/jdt/core/tests/model/CompletionTests.java	26 Feb 2009 12:43:47 -0000
@@ -19353,4 +19353,71 @@
 			"",
 			requestor.getResults());
 }
+
+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 don't propose final types for extends type completion.
+public void testCompletionOnExtendFinalClass () throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+		"/Completion/src/test/Test.java",
+		"package test;"+
+		"class ThisClassIsNotFinal {}" +
+		"final class ThisClassIsFinal {}" +
+		"public class Event extends test.ThisClassI {\n" +
+		"}");
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "test.ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_NON_RESTRICTED + R_CLASS) + "}",	
+			requestor.getResults());
+}
+
+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 don't propose final types for extends type completion.
+public void testCompletionOnExtendFinalClass2() throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+		"/Completion/src/test/Test.java",
+		"package test;"+
+		"class ThisClassIsNotFinal {}" +
+		"final class ThisClassIsFinal {}" +
+		"public class Event extends ThisClassI {\n" +
+		"}");
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED + R_CLASS) + "}",
+			requestor.getResults());
+}
+
+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we continue to propose final types
+// in other (non extends context).
+public void testCompletionOnExtendFinalClass3() throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+		"/Completion/src/test/Test.java",
+		"package test;"+
+		"class ThisClassIsNotFinal {}" +
+		"final class ThisClassIsFinal {}" +
+		"public class Event extends ThisClassIsFinal {\n" +
+		"	void Boo (ThisClassI x) {}\n" +
+		"}");
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsFinal[TYPE_REF]{ThisClassIsFinal, test, Ltest.ThisClassIsFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}\n" +
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
+			requestor.getResults());
+}
+
 }
Index: src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java
===================================================================
RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java,v
retrieving revision 1.111
diff -u -r1.111 CompletionTests_1_5.java
--- src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java	30 Jan 2009 15:47:20 -0000	1.111
+++ src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java	26 Feb 2009 12:43:55 -0000
@@ -323,8 +323,8 @@
             ".Y001");
 
     assertResults(
-            "expectedTypesSignatures=null\n" +
-            "expectedTypesKeys=null",
+            "expectedTypesSignatures={Ltest0011.Z0011<Ljava.lang.Object;>;}\n" +
+            "expectedTypesKeys={Ltest0011/Test~Z0011<Ljava/lang/Object;>;}",
             result.context);
 
     assertResults(
@@ -490,8 +490,8 @@
             ".Y001");
 
     assertResults(
-            "expectedTypesSignatures=null\n" +
-            "expectedTypesKeys=null",
+            "expectedTypesSignatures={Ltest0018.Z0018<Ljava.lang.Object;>;}\n" +
+            "expectedTypesKeys={Ltest0018/Test~Z0018<Ljava/lang/Object;>;}",
             result.context);
 
     assertResults(
@@ -513,8 +513,8 @@
             ".Y001");
 
     assertResults(
-            "expectedTypesSignatures=null\n" +
-            "expectedTypesKeys=null",
+            "expectedTypesSignatures={Ltest0019.Z0019<Ljava.lang.Object;>;}\n" +
+            "expectedTypesKeys={Ltest0019/Test~Z0019<Ljava/lang/Object;>;}",
             result.context);
 
     assertResults(
@@ -13476,4 +13476,105 @@
 			requestor.getResults());
 }
 
+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we continue to propose final
+// types in extends contexts but where they are not directly extended.
+public void testCompletionOnExtends() throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+			"/Completion/src3/test/Test.java",
+			"package test;\n" +
+			"class X<T> {};\n" +
+			"final class ThisClassIsFinal {}\n" +
+			"class ThisClassIsNotFinal {}\n" +
+			"public class Test extends X<ThisClassI> {}");
+
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsFinal[TYPE_REF]{ThisClassIsFinal, test, Ltest.ThisClassIsFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED + R_CLASS) + "}\n" +
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED + R_CLASS) + "}",
+			requestor.getResults());
+}
+
+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we don't propose final
+//types in extends contexts where we should not.
+public void testCompletionOnExtends2() throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+			"/Completion/src3/test/Test.java",
+			"package test;\n" +
+			"final class ThisClassIsFinal {}\n" +
+			"class ThisClassIsNotFinal {}\n" +
+			"public class Test <T extends ThisClassI> {}\n");
+
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
+			requestor.getResults());
+}
+
+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we don't propose final
+//types in extends contexts where we should not.
+public void testCompletionOnExtends3() throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+			"/Completion/src3/test/Test.java",
+			"package test;\n" +
+			"final class ThisClassIsFinal {}\n" +
+			"class ThisClassIsNotFinal {}\n" +
+			"public class Test {\n" +
+			"    Test(Bag<? extends ThisClassI> p) {}" +
+			"}\n");
+
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
+			requestor.getResults());
+}
+
+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we do propose final
+//types in super contexts where we should.
+public void testCompletionOnExtends4() throws JavaModelException {
+	this.workingCopies = new ICompilationUnit[1];
+	this.workingCopies[0] = getWorkingCopy(
+			"/Completion/src3/test/Test.java",
+			"package test;\n" +
+			"final class ThisClassIsFinal {}\n" +
+			"class ThisClassIsNotFinal {}\n" +
+			"public class Test {\n" +
+			"    void boo() {\n" +
+			"        Bag<? super ThisClassI> local;\n" +
+			"    }\n" +
+			"}\n");
+
+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
+
+	String str = this.workingCopies[0].getSource();
+	String completeBehind = "ThisClassI";
+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
+
+	assertResults(
+			"ThisClassIsFinal[TYPE_REF]{ThisClassIsFinal, test, Ltest.ThisClassIsFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}\n" +
+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
+			requestor.getResults());
+}
+
 }