[gnome-maps/wip/jonasdn/service-gnome-3-18: 2/2] Use GtkClutter for attribution logo

[Jonas Danielsson <jonasdn src gnome org>]

commit f60ff73d8c4d42874f974823701543a57f1ac901
Author: Jonas Danielsson <jonas threetimestwo org>
Date:   Wed Aug 17 12:57:05 2016 +0200

    Use GtkClutter for attribution logo
    
    https://bugzilla.gnome.org/show_bug.cgi?id=769352

 src/mainWindow.js |    5 -----
 src/mapSource.js  |   47 ++++++++++++++++++++++++++++++++++-------------
 src/mapView.js    |    5 +++++
 3 files changed, 39 insertions(+), 18 deletions(-)
---
diff --git a/src/mainWindow.js b/src/mainWindow.js
index d210d92..3ad690b 100644
--- a/src/mainWindow.js
+++ b/src/mainWindow.js
@@ -35,7 +35,6 @@ const Geoclue = imports.geoclue;
 const LayersPopover = imports.layersPopover;
 const LocationServiceNotification = imports.locationServiceNotification;
 const MapView = imports.mapView;
-const MapSource = imports.mapSource;
 const PlaceEntry = imports.placeEntry;
 const PlaceStore = imports.placeStore;
 const Sidebar = imports.sidebar;
@@ -78,10 +77,6 @@ const MainWindow = new Lang.Class({
 
         this._overlay.add(this._mapView);
 
-        if (!this.application.local_tile_path) {
-            this._overlay.add_overlay(new MapSource.AttributionLogo());
-        }
-
         this._mapView.gotoUserLocation(false);
 
         this._sidebar = this._createSidebar();
diff --git a/src/mapSource.js b/src/mapSource.js
index 4b89001..ba23cdf 100644
--- a/src/mapSource.js
+++ b/src/mapSource.js
@@ -22,6 +22,7 @@ const GdkPixbuf = imports.gi.GdkPixbuf;
 const Gio = imports.gi.Gio;
 const GLib = imports.gi.GLib;
 const Gtk = imports.gi.Gtk;
+const GtkClutter = imports.gi.GtkClutter;
 const Lang = imports.lang;
 const Soup = imports.gi.Soup;
 
@@ -36,28 +37,48 @@ const _DEFAULT_SERVICE_FILE = 'maps-service.json';
 const _FILE_CACHE_SIZE_LIMIT = (10 * 1024 * 1024); /* 10Mb */
 const _MEMORY_CACHE_SIZE_LIMIT = 100; /* number of tiles */
 
+const _LOGO_PADDING_X = 10;
+const _LOGO_PADDING_Y = 25;
+
 const AttributionLogo = new Lang.Class({
     Name: 'AttributionLogo',
-    Extends: Gtk.Bin,
-
-    _init: function() {
-        this.parent({ margin_end: 6,
-                      margin_bottom: 25,
-                      halign: Gtk.Align.END,
-                      valign: Gtk.Align.END });
-        this.add(_attributionImage);
+    Extends: GtkClutter.Actor,
+
+    _init: function(view) {
+        this.parent();
+
+        if (_attributionImage)
+            this.contents = _attributionImage;
+        else
+            return;
+
+        view.connect('notify::width', (function() {
+            this._updatePosition(view);
+        }).bind(this));
+
+        view.connect('notify::height', (function() {
+            this._updatePosition(view);
+        }).bind(this));
+
+        this._updatePosition(view);
+    },
+
+    _updatePosition: function(view) {
+        let width = _attributionImage.pixbuf.width;
+        let height = _attributionImage.pixbuf.height;
+
+        this.set_position(view.width  - width  - _LOGO_PADDING_X,
+                          view.height - height - _LOGO_PADDING_Y);
     }
 });
 
 function _updateAttributionImage(source) {
+    if (!source.attribution_logo || source.attribution_logo === "")
+        return;
+
     if (!_attributionImage)
         _attributionImage = new Gtk.Image();
 
-    if (!source.attribution_logo || source.attribution_logo === "") {
-        _attributionImage.visible = false;
-        return;
-    }
-
     let data = GLib.base64_decode(source.attribution_logo);
     let stream = Gio.MemoryInputStream.new_from_bytes(GLib.Bytes.new(data));
     _attributionImage.pixbuf = GdkPixbuf.Pixbuf.new_from_stream(stream, null);
diff --git a/src/mapView.js b/src/mapView.js
index 765bf4f..8433d18 100644
--- a/src/mapView.js
+++ b/src/mapView.js
@@ -158,6 +158,11 @@ const MapView = new Lang.Class({
                 this.view.map_source = MapSource.createAerialSource();
             else
                 this.view.map_source = MapSource.createStreetSource();
+
+            if (!this._attribution) {
+                this._attribution = new MapSource.AttributionLogo(this.view);
+                this.view.add_child(this._attribution);
+            }
         } else {
             let renderer = new Champlain.ImageRenderer();
             let source = new Maps.FileTileSource({