Ticket #12964: patch-mapview-fix-zoom-to-bounds.patch

src/org/openstreetmap/josm/Main.java

@@ -569 +569 @@
         getLayerManager().addLayerChangeListener(new LayerChangeListener() {
             @Override
             public void layerAdded(LayerAddEvent e) {
+                Layer layer = e.getAddedLayer();
                 if (map == null) {
-                    Layer layer = e.getAddedLayer();
-                    ProjectionBounds viewProjectionBounds = layer.getViewProjectionBounds();
-                    Main.main.createMapFrame(layer, viewProjectionBounds == null ? null : new ViewportData(viewProjectionBounds));
+                    Main.main.createMapFrame(layer, null);
+                    Main.map.setVisible(true);
+                }
+                ProjectionBounds viewProjectionBounds = layer.getViewProjectionBounds();
+                if (viewProjectionBounds != null) {
+                    Main.map.mapView.scheduleZoomTo(new ViewportData(viewProjectionBounds));
                 }
             }
 
@@ -819 +823 @@
      */
     public final synchronized void addLayer(final Layer layer, ViewportData viewport) {
         getLayerManager().addLayer(layer);
-        if (map != null) {
-            Main.map.setVisible(true);
-        } else if (viewport != null) {
-            Main.map.mapView.zoomTo(viewport);
+        if (viewport != null) {
+            Main.map.mapView.scheduleZoomTo(viewport);
         }
     }
 

src/org/openstreetmap/josm/gui/MapView.java

@@ -1290 +1290 @@
     public final MainLayerManager getLayerManager() {
         return layerManager;
     }
+
+    /**
+     * Schedule a zoom to the given position on the next redraw.
+     * Temporary, may be removed without warning.
+     * @param viewportData
+     * @since xxx
+     */
+    public void scheduleZoomTo(ViewportData viewportData) {
+        initialViewport = viewportData;
+    }
 }