Also read initially visible marker fields from extensions #2193

Author: travkin79

bundles/org.eclipse.ui.ide/src/org/eclipse/ui/views/markers/internal/ContentGeneratorDescriptor.java

@@ -174,7 +174,12 @@ public String getId() {
 	 * @return {@link MarkerField}[]
 	 */
 	public MarkerField[] getInitialVisible() {
-		return initialVisibleWithoutExtensions;
+		List<MarkerField> allVisibleFieldsList = new ArrayList<>(Arrays.asList(initialVisibleWithoutExtensions));
+
+		getExtensionsDescriptorsStream().flatMap(descriptor -> Arrays.stream(descriptor.getInitialVisible()))
+				.forEach(field -> allVisibleFieldsList.add(field));
+
+		return allVisibleFieldsList.toArray(new MarkerField[allVisibleFieldsList.size()]);
 	}
 
 	/**

tests/org.eclipse.ui.tests/Eclipse UI Tests/org/eclipse/ui/tests/markers/MarkerSupportViewTest.java

@@ -141,7 +141,6 @@ public void markerContentGeneratorExtensionLoaded() throws Exception {
 		assertNotNull(descriptor);
 
 		MarkerField[] allFields = descriptor.getAllFields();
-
 		boolean foundAdditionalField1 = false;
 		boolean foundRecursivelyAddedField2 = false;
 		for (MarkerField field : allFields) {
@@ -160,6 +159,16 @@ public void markerContentGeneratorExtensionLoaded() throws Exception {
 		assertTrue("MarkerField not loaded from recursive marker content generator extension.",
 				foundRecursivelyAddedField2);
 
+		MarkerField[] initiallyVisibleFields = descriptor.getInitialVisible();
+		boolean foundAdditionalVisbileField = false;
+		for (MarkerField field : initiallyVisibleFields) {
+			if ("Problem Key".equals(field.getName())) {
+				foundAdditionalVisbileField = true;
+			}
+		}
+		assertTrue("Visible attribute from marker fields from marker content generator extensions not loaded.",
+				foundAdditionalVisbileField);
+
 		MarkerType markerTypeFromExtension = descriptor.getType("org.eclipse.ui.tests.markers.artificial.problem");
 		assertNotNull("Marker type from marker content generator extensions not loaded.", markerTypeFromExtension);
 		assertTrue(descriptor.getMarkerTypes().contains(markerTypeFromExtension));