Bug 262392 [DataBinding] CompositeMap constructor should use this.firstMap
Using EclEmma, I noticed that large portions of BidiObservableMap are not run, indicating a problem in CompositeMap. This needs more thought. I am attaching a patch, it includes one test case but we may want to add more tests.
bundles/org.eclipse.core.databinding/src/org/eclipse/core/databinding/observable/map/CompositeMap.java
wrappedMap.remove(element);
					}
				} else {
					Assert.isTrue(false, "unexpected case"); 
				}
			}
			if (adds.size() > 0 || removes.size() > 0 || changes.size() > 0) {
				fireMapChange(new MapDiff() {
					public Set getAddedKeys() {
						return adds;
					}
					public Set getChangedKeys() {
						return changes;
					}
					public Object getNewValue(Object key) {
						return newValues.get(key);
					}
					public Object getOldValue(Object key) {
						return oldValues.get(key);
					}
					public Set getRemovedKeys() {
						return removes;
					}
				});
			}
		}
	};
	private BidiObservableMap firstMap;
	private IObservableMap secondMap;
	private static class WritableSetPlus extends WritableSet {