Changeset 12966 in josm


Ignore:
Timestamp:
2017-10-09T17:14:50+02:00 (7 years ago)
Author:
bastiK
Message:

use RenderingHelper in MapCSSRendererTest; move getBackgroundColor() from PaintColors to ElemStyles (no longer global) (see #15273)

Location:
trunk
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/PaintColors.java

    r12537 r12966  
    55
    66import java.awt.Color;
    7 import java.util.List;
    8 import java.util.Optional;
    97
    108import org.openstreetmap.josm.data.preferences.CachingProperty;
    119import org.openstreetmap.josm.data.preferences.ColorProperty;
    1210import org.openstreetmap.josm.gui.mappaint.MapPaintStyles;
    13 import org.openstreetmap.josm.gui.mappaint.MapPaintStyles.MapPaintSylesUpdateListener;
    14 import org.openstreetmap.josm.gui.mappaint.StyleSource;
    1511
    1612/**
    … …  
    9187    private final CachingProperty<Color> property;
    9288
    93     private static volatile Color backgroundColorCache;
    94 
    95     private static final MapPaintSylesUpdateListener STYLE_OVERRIDE_LISTENER = new MapPaintSylesUpdateListener() {
    96         //TODO: Listen to wireframe map mode changes.
    97         @Override
    98         public void mapPaintStylesUpdated() {
    99             backgroundColorCache = null;
    100         }
    101 
    102         @Override
    103         public void mapPaintStyleEntryUpdated(int idx) {
    104             mapPaintStylesUpdated();
    105         }
    106     };
    107 
    108     static {
    109         MapPaintStyles.addMapPaintSylesUpdateListener(STYLE_OVERRIDE_LISTENER);
    110     }
    111 
    11289    PaintColors(String name, Color defaultColor) {
    11390        baseProperty = new ColorProperty(name, defaultColor);
    … …  
    138115     */
    139116    public static Color getBackgroundColor() {
    140         if (backgroundColorCache != null)
    141             return backgroundColorCache;
    142         List<StyleSource> sources = MapPaintStyles.getStyles().getStyleSources();
    143         for (StyleSource s : sources) {
    144             if (!s.active) {
    145                 continue;
    146             }
    147             Color backgroundColorOverride = s.getBackgroundColorOverride();
    148             if (backgroundColorOverride != null) {
    149                 backgroundColorCache = backgroundColorOverride;
    150             }
    151         }
    152         return Optional.ofNullable(backgroundColorCache).orElseGet(BACKGROUND::get);
     117        return MapPaintStyles.getStyles().getBackgroundColor();
    153118    }
    154119

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

    r12964 r12966  
    13721372        super.getColors();
    13731373        this.highlightColorTransparent = new Color(highlightColor.getRed(), highlightColor.getGreen(), highlightColor.getBlue(), 100);
    1374         this.backgroundColor = PaintColors.getBackgroundColor();
     1374        this.backgroundColor = styles.getBackgroundColor();
    13751375    }
    13761376

  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

    r12881 r12966  
    1010import java.util.Map;
    1111import java.util.Map.Entry;
     12import java.util.Optional;
    1213
    1314import org.openstreetmap.josm.spi.preferences.PreferenceChangeEvent;
    … …  
    1718import org.openstreetmap.josm.data.osm.Relation;
    1819import org.openstreetmap.josm.data.osm.Way;
     20import org.openstreetmap.josm.data.osm.visitor.paint.PaintColors;
    1921import org.openstreetmap.josm.data.osm.visitor.paint.relations.Multipolygon;
    2022import org.openstreetmap.josm.data.osm.visitor.paint.relations.MultipolygonCache;
    … …  
    6971    private final Map<String, String> preferenceCache = new HashMap<>();
    7072
     73    private static volatile Color backgroundColorCache;
     74
    7175    /**
    7276     * Constructs a new {@code ElemStyles}.
    … …  
    7579        styleSources = new ArrayList<>();
    7680        Config.getPref().addPreferenceChangeListener(this);
     81        MapPaintStyles.addMapPaintSylesUpdateListener(new MapPaintStyles.MapPaintSylesUpdateListener() {
     82            //TODO: Listen to wireframe map mode changes.
     83            @Override
     84            public void mapPaintStylesUpdated() {
     85                backgroundColorCache = null;
     86            }
     87
     88            @Override
     89            public void mapPaintStyleEntryUpdated(int idx) {
     90                mapPaintStylesUpdated();
     91            }
     92        });
    7793    }
    7894
    … …  
    94110    public List<StyleSource> getStyleSources() {
    95111        return Collections.<StyleSource>unmodifiableList(styleSources);
     112    }
     113
     114    public Color getBackgroundColor() {
     115        if (backgroundColorCache != null)
     116            return backgroundColorCache;
     117        for (StyleSource s : styleSources) {
     118            if (!s.active) {
     119                continue;
     120            }
     121            Color backgroundColorOverride = s.getBackgroundColorOverride();
     122            if (backgroundColorOverride != null) {
     123                backgroundColorCache = backgroundColorOverride;
     124            }
     125        }
     126        return Optional.ofNullable(backgroundColorCache).orElseGet(PaintColors.BACKGROUND::get);
    96127    }
    97128

  • trunk/src/org/openstreetmap/josm/gui/mappaint/RenderingCLI.java

    r12963 r12966  
    55
    66import java.awt.Dimension;
     7import java.awt.image.BufferedImage;
     8import java.io.File;
    79import java.io.FileInputStream;
    810import java.io.FileNotFoundException;
    … …  
    1416import java.util.function.Supplier;
    1517import java.util.logging.Level;
     18
     19import javax.imageio.ImageIO;
    1620
    1721import org.openstreetmap.gui.jmapviewer.OsmMercator;
    … …  
    155159            RenderingArea area = determineRenderingArea(ds);
    156160            RenderingHelper rh = new RenderingHelper(ds, area.bounds, area.scale, argStyles);
    157             rh.setOutputFile(argOutput);
    158161            checkPreconditions(rh);
    159             rh.render();
     162            BufferedImage image = rh.render();
     163            writeImageToFile(image);
    160164        } catch (FileNotFoundException e) {
    161165            if (Logging.isDebugEnabled()) {
    … …  
    562566        }
    563567    }
     568
     569    private void writeImageToFile(BufferedImage image) throws IOException {
     570        String output = Optional.ofNullable(argOutput).orElse("out.png");
     571        ImageIO.write(image, "png", new File(output));
     572    }
    564573}

  • trunk/src/org/openstreetmap/josm/gui/mappaint/RenderingHelper.java

    r12964 r12966  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
     6import java.awt.Color;
    67import java.awt.Dimension;
    78import java.awt.Graphics2D;
    89import java.awt.Point;
     10import java.awt.RenderingHints;
    911import java.awt.image.BufferedImage;
    10 import java.io.File;
    1112import java.io.IOException;
    1213import java.util.Collection;
    … …  
    1516import java.util.Optional;
    1617
    17 import javax.imageio.ImageIO;
    18 
    1918import org.openstreetmap.josm.Main;
    2019import org.openstreetmap.josm.data.Bounds;
    2120import org.openstreetmap.josm.data.ProjectionBounds;
    2221import org.openstreetmap.josm.data.osm.DataSet;
    23 import org.openstreetmap.josm.data.osm.visitor.paint.PaintColors;
    2422import org.openstreetmap.josm.data.osm.visitor.paint.StyledMapRenderer;
    2523import org.openstreetmap.josm.data.projection.Projection;
    … …  
    4139    private final double scale;
    4240    private final Collection<StyleData> styles;
    43     private String outputFile;
     41    private Color backgroundColor;
     42    private boolean fillBackground = true;
    4443
    4544    /**
    … …  
    7372
    7473    /**
    75      * Set the output file for rendering.
     74     * Set the background color to use for rendering.
    7675     *
    77      * Default is {@code out.png}.
    78      * @param outputFile the output file for rendering
     76     * @param backgroundColor the background color to use, {@code} means
     77     * to determine the background color automatically from the style
     78     * @see #setFillBackground(boolean)
     79     * @since 12966
    7980     */
    80     public void setOutputFile(String outputFile) {
    81         this.outputFile = outputFile;
     81    public void setBackgroundColor(Color backgroundColor) {
     82        this.backgroundColor = backgroundColor;
     83    }
     84
     85    /**
     86     * Decide if background should be filled or left transparent.
     87     * @param fillBackground true, if background should be filled
     88     * @see #setBackgroundColor(java.awt.Color)
     89     * @since 12966
     90     */
     91    public void setFillBackground(boolean fillBackground) {
     92        this.fillBackground = fillBackground;
    8293    }
    8394
    … …  
    93104     * Invoke the renderer.
    94105     *
     106     * @return the rendered image
    95107     * @throws IOException in case of an IOException
    96108     * @throws IllegalDataException when illegal data is encountered (style has errors, etc.)
    97109     */
    98     public void render() throws IOException, IllegalDataException {
     110    public BufferedImage render() throws IOException, IllegalDataException {
    99111        // load the styles
    100112        ElemStyles elemStyles = new ElemStyles();
    … …  
    152164        BufferedImage image = new BufferedImage(imgDimPx.width, imgDimPx.height, BufferedImage.TYPE_INT_ARGB);
    153165        Graphics2D g = image.createGraphics();
    154         g.setColor(PaintColors.getBackgroundColor());
    155         g.fillRect(0, 0, imgDimPx.width, imgDimPx.height);
     166
     167        // Force all render hints to be defaults - do not use platform values
     168        g.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
     169        g.setRenderingHint(RenderingHints.KEY_ALPHA_INTERPOLATION, RenderingHints.VALUE_ALPHA_INTERPOLATION_QUALITY);
     170        g.setRenderingHint(RenderingHints.KEY_COLOR_RENDERING, RenderingHints.VALUE_COLOR_RENDER_QUALITY);
     171        g.setRenderingHint(RenderingHints.KEY_DITHERING, RenderingHints.VALUE_DITHER_DISABLE);
     172        g.setRenderingHint(RenderingHints.KEY_FRACTIONALMETRICS, RenderingHints.VALUE_FRACTIONALMETRICS_ON);
     173        g.setRenderingHint(RenderingHints.KEY_INTERPOLATION, RenderingHints.VALUE_INTERPOLATION_BILINEAR);
     174        g.setRenderingHint(RenderingHints.KEY_RENDERING, RenderingHints.VALUE_RENDER_QUALITY);
     175        g.setRenderingHint(RenderingHints.KEY_STROKE_CONTROL, RenderingHints.VALUE_STROKE_NORMALIZE);
     176        g.setRenderingHint(RenderingHints.KEY_TEXT_ANTIALIASING, RenderingHints.VALUE_TEXT_ANTIALIAS_ON);
     177
     178        if (fillBackground) {
     179            g.setColor(Optional.ofNullable(backgroundColor).orElse(elemStyles.getBackgroundColor()));
     180            g.fillRect(0, 0, imgDimPx.width, imgDimPx.height);
     181        }
    156182        StyledMapRenderer smr = new StyledMapRenderer(g, nc, false);
    157183        smr.setStyles(elemStyles);
    158184        smr.render(ds, false, bounds);
    159 
    160         // write to file
    161         String output = Optional.ofNullable(outputFile).orElse("out.png");
    162         ImageIO.write(image, "png", new File(output));
     185        return image;
    163186    }
    164187

  • trunk/test/functional/org/openstreetmap/josm/gui/mappaint/MapCSSRendererTest.java

    r12965 r12966  
    55import static org.junit.Assert.fail;
    66
    7 import java.awt.Graphics2D;
    87import java.awt.GraphicsEnvironment;
    98import java.awt.Point;
    10 import java.awt.RenderingHints;
    119import java.awt.image.BufferedImage;
    1210import java.io.File;
    … …  
    1816import java.util.Arrays;
    1917import java.util.Collection;
     18import java.util.Collections;
    2019import java.util.List;
    2120import java.util.stream.Collectors;
    … …  
    3837import org.openstreetmap.josm.data.osm.OsmPrimitive;
    3938import org.openstreetmap.josm.data.osm.visitor.paint.StyledMapRenderer;
    40 import org.openstreetmap.josm.data.preferences.sources.SourceEntry;
    41 import org.openstreetmap.josm.data.preferences.sources.SourceType;
    42 import org.openstreetmap.josm.gui.NavigatableComponent;
    43 import org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSStyleSource;
    4439import org.openstreetmap.josm.io.IllegalDataException;
    4540import org.openstreetmap.josm.io.OsmReader;
    … …  
    163158        // load the data
    164159        DataSet dataSet = testConfig.getOsmDataSet();
    165 
    166         // load the style
    167         MapCSSStyleSource.STYLE_SOURCE_LOCK.writeLock().lock();
    168         try {
    169             MapPaintStyles.getStyles().clear();
    170 
    171             MapCSSStyleSource source = new MapCSSStyleSource(testConfig.getStyleSourceEntry());
    172             source.loadStyleSource();
    173             if (!source.getErrors().isEmpty()) {
    174                 fail("Failed to load style file. Errors: " + source.getErrors());
    175             }
    176             MapPaintStyles.getStyles().setStyleSources(Arrays.asList(source));
    177             MapPaintStyles.fireMapPaintSylesUpdated();
    178             MapPaintStyles.getStyles().clearCached();
    179 
    180         } finally {
    181             MapCSSStyleSource.STYLE_SOURCE_LOCK.writeLock().unlock();
    182         }
    183 
    184         // create the renderer
    185         BufferedImage image = new BufferedImage(IMAGE_SIZE, IMAGE_SIZE, BufferedImage.TYPE_INT_ARGB);
    186         NavigatableComponent nc = new NavigatableComponent() {
    187             {
    188                 setBounds(0, 0, IMAGE_SIZE, IMAGE_SIZE);
    189                 updateLocationState();
    190             }
    191 
    192             @Override
    193             protected boolean isVisibleOnScreen() {
    194                 return true;
    195             }
    196 
    197             @Override
    198             public Point getLocationOnScreen() {
    199                 return new Point(0, 0);
    200             }
    201         };
     160        dataSet.allPrimitives().stream().forEach(this::loadPrimitiveStyle);
     161        dataSet.setSelected(dataSet.allPrimitives().stream().filter(n -> n.isKeyTrue("selected")).collect(Collectors.toList()));
     162
    202163        ProjectionBounds pb = new ProjectionBounds();
    203164        pb.extend(Main.getProjection().latlon2eastNorth(testConfig.testArea.getMin()));
    204165        pb.extend(Main.getProjection().latlon2eastNorth(testConfig.testArea.getMax()));
    205166        double scale = (pb.maxEast - pb.minEast) / IMAGE_SIZE;
    206         nc.zoomTo(pb.getCenter(), scale);
    207 
    208         dataSet.allPrimitives().stream().forEach(this::loadPrimitiveStyle);
    209         dataSet.setSelected(dataSet.allPrimitives().stream().filter(n -> n.isKeyTrue("selected")).collect(Collectors.toList()));
    210 
    211         Graphics2D g = image.createGraphics();
    212         // Force all render hints to be defaults - do not use platform values
    213         g.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
    214         g.setRenderingHint(RenderingHints.KEY_ALPHA_INTERPOLATION, RenderingHints.VALUE_ALPHA_INTERPOLATION_QUALITY);
    215         g.setRenderingHint(RenderingHints.KEY_COLOR_RENDERING, RenderingHints.VALUE_COLOR_RENDER_QUALITY);
    216         g.setRenderingHint(RenderingHints.KEY_DITHERING, RenderingHints.VALUE_DITHER_DISABLE);
    217         g.setRenderingHint(RenderingHints.KEY_FRACTIONALMETRICS, RenderingHints.VALUE_FRACTIONALMETRICS_ON);
    218         g.setRenderingHint(RenderingHints.KEY_INTERPOLATION, RenderingHints.VALUE_INTERPOLATION_BILINEAR);
    219         g.setRenderingHint(RenderingHints.KEY_RENDERING, RenderingHints.VALUE_RENDER_QUALITY);
    220         g.setRenderingHint(RenderingHints.KEY_STROKE_CONTROL, RenderingHints.VALUE_STROKE_NORMALIZE);
    221         g.setRenderingHint(RenderingHints.KEY_TEXT_ANTIALIASING, RenderingHints.VALUE_TEXT_ANTIALIAS_ON);
    222         new StyledMapRenderer(g, nc, false).render(dataSet, false, testConfig.testArea);
     167
     168        RenderingHelper.StyleData sd = new RenderingHelper.StyleData();
     169        sd.styleUrl = testConfig.getStyleSourceUrl();
     170        RenderingHelper rh = new RenderingHelper(dataSet, testConfig.testArea, scale, Collections.singleton(sd));
     171        rh.setFillBackground(false);
     172        BufferedImage image = rh.render();
    223173
    224174        if (UPDATE_ALL) {
    … …  
    317267        }
    318268
    319         public SourceEntry getStyleSourceEntry() {
    320             return new SourceEntry(SourceType.MAP_PAINT_STYLE, getTestDirectory() + "/style.mapcss",
    321                     "test style", "a test style", true // active
    322             );
     269        public String getStyleSourceUrl() {
     270            return getTestDirectory() + "/style.mapcss";
    323271        }
    324272
Note: See TracChangeset for help on using the changeset viewer.