weston_surface_damage does it wrong

Submitted by Pekka Paalanen @pq

Assigned to Pekka Paalanen @pq

Link to original bug (#18)

Description

/**  
 * XXX: This function does it the wrong way.  
 * surface->damage is the damage from the client, and causes  
 * surface_flush_damage() to copy pixels. No window management action can  
 * cause damage to the client-provided content, warranting re-upload!  
 *  
 * Instead of surface->damage, this function should record the damage  
 * with all the views for this surface to avoid extraneous texture  
 * uploads.  
 */  
WL_EXPORT void  
weston_surface_damage(struct weston_surface *surface);