From: Alan Webb <alan_webb@us.ibm.com>

  Changes to support client-side image pre-caching in the region. This
  commit adds an additional calling sequence to the DynamicTexture data
  and URL calls. The new interface allows a dynamic image to be loaded
  into a specific object face (rather than the mandatory ALL_SIDES
  supported today.
  This is in part fulfilment of ticket #458.

2 files changed, 105 insertions, 50 deletions

diff --git a/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs b/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs
index 259a333..6509bf3 100644
--- a/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs
+++ b/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs
@@ -42,6 +42,10 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
 {
     public class DynamicTextureModule : IRegionModule, IDynamicTextureManager
     {
+        private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
+
+        private const int ALL_SIDES = -1;
+
         private Dictionary<UUID, Scene> RegisteredScenes = new Dictionary<UUID, Scene>();
 
         private Dictionary<string, IDynamicTextureRender> RenderPlugins =
@@ -106,6 +110,12 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
         public UUID AddDynamicTextureURL(UUID simID, UUID primID, string contentType, string url,
                                          string extraParams, int updateTimer, bool SetBlending, byte AlphaValue)
         {
+            return AddDynamicTextureURL(simID, primID, contentType, url, extraParams, updateTimer, SetBlending, AlphaValue, ALL_SIDES);
+        }
+
+        public UUID AddDynamicTextureURL(UUID simID, UUID primID, string contentType, string url,
+                                         string extraParams, int updateTimer, bool SetBlending, byte AlphaValue, int face)
+        {
             if (RenderPlugins.ContainsKey(contentType))
             {
                 //m_log.Debug("dynamic texture being created: " + url + " of type " + contentType);
@@ -120,6 +130,7 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
                 updater.Params = extraParams;
                 updater.BlendWithOldTexture = SetBlending;
                 updater.FrontAlpha = AlphaValue;
+                updater.Face = face;
 
                 lock (Updaters)
                 {
@@ -144,6 +155,12 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
         public UUID AddDynamicTextureData(UUID simID, UUID primID, string contentType, string data,
                                           string extraParams, int updateTimer, bool SetBlending, byte AlphaValue)
         {
+            return AddDynamicTextureData(simID, primID, contentType, data, extraParams, updateTimer, SetBlending, AlphaValue, ALL_SIDES);
+        }
+
+        public UUID AddDynamicTextureData(UUID simID, UUID primID, string contentType, string data,
+                                          string extraParams, int updateTimer, bool SetBlending, byte AlphaValue, int face)
+        {
             if (RenderPlugins.ContainsKey(contentType))
             {
                 DynamicTextureUpdater updater = new DynamicTextureUpdater();
@@ -156,6 +173,8 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
                 updater.Params = extraParams;
                 updater.BlendWithOldTexture = SetBlending;
                 updater.FrontAlpha = AlphaValue;
+                updater.Face = face;
+                updater.Url = "Local image";
 
                 lock (Updaters)
                 {
@@ -232,6 +251,7 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
             public UUID SimUUID;
             public UUID UpdaterID;
             public int UpdateTimer;
+            public int Face;
             public string Url;
 
             public DynamicTextureUpdater()
@@ -249,7 +269,7 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
 
                 SceneObjectPart part = scene.GetSceneObjectPart(PrimID);
 
-                if (data == null)
+                if (data == null || data.Length <= 1)
                 {
                     string msg = 
                         String.Format("DynamicTextureModule: Error preparing image using URL {0}", Url);
@@ -287,34 +307,51 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture
                 asset.Data = assetData;
                 asset.Name = "DynamicImage" + Util.RandomClass.Next(1, 10000);
                 asset.Type = 0;
-                asset.Description = "dynamic image";
+                asset.Description = String.Format("URL image : {0}", Url);
                 asset.Local = false;
                 asset.Temporary = true;
                 scene.AssetService.Store(asset);
 
-                LastAssetID = asset.FullID;
-
                 IJ2KDecoder cacheLayerDecode = scene.RequestModuleInterface<IJ2KDecoder>();
                 if (cacheLayerDecode != null)
                 {
                     cacheLayerDecode.syncdecode(asset.FullID, asset.Data);
+                    cacheLayerDecode = null;
+                    LastAssetID = asset.FullID;
                 }
-                cacheLayerDecode = null;
 
                 // mostly keep the values from before
                 Primitive.TextureEntry tmptex = part.Shape.Textures;
 
-                // remove the old asset from the cache
-                // UUID oldID = tmptex.DefaultTexture.TextureID;
+                // remove the old asset from the cache later
+                UUID oldID = tmptex.DefaultTexture.TextureID;
                 
-                tmptex.DefaultTexture.TextureID = asset.FullID;
+                if (Face == ALL_SIDES)
+                {
+                    tmptex.DefaultTexture.TextureID = asset.FullID;
+                }
+                else
+                {
+                    try
+                    {
+                        Primitive.TextureEntryFace texface = tmptex.CreateFace((uint)Face);
+                        texface.TextureID = asset.FullID;
+                        tmptex.FaceTextures[Face] = texface;
+                    }
+                    catch(Exception)
+                    {
+                        tmptex.DefaultTexture.TextureID = asset.FullID;
+                    }
+                }
+
                 // I'm pretty sure we always want to force this to true
                 // I'm pretty sure noone whats to set fullbright true if it wasn't true before.
                 // tmptex.DefaultTexture.Fullbright = true;
 
-                part.Shape.Textures = tmptex;
-                part.ScheduleFullUpdate();
-
+                part.UpdateTexture(tmptex);
+                
+                // scene.CommsManager.AssetCache.ExpireAsset(oldID);
+                scene.AssetService.Delete(oldID.ToString());
             }
 
             private byte[] BlendTextures(byte[] frontImage, byte[] backImage, bool setNewAlpha, byte newAlpha)
diff --git a/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs b/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs
index 864d6ac..61c37b9 100644
--- a/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs
+++ b/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs
@@ -162,62 +162,80 @@ namespace OpenSim.Region.CoreModules.Scripting.LoadImageURL
 
         private void HttpRequestReturn(IAsyncResult result)
         {
+
             RequestState state = (RequestState) result.AsyncState;
             WebRequest request = (WebRequest) state.Request;
+            Stream stream = null;
+                        byte[] imageJ2000 = new byte[0];
+
             try
             {
                 HttpWebResponse response = (HttpWebResponse)request.EndGetResponse(result);
-                if (response.StatusCode == HttpStatusCode.OK)
+                if (response != null && response.StatusCode == HttpStatusCode.OK)
                 {
-                    Bitmap image = new Bitmap(response.GetResponseStream());
-                    Size newsize;
-
-                    // TODO: make this a bit less hard coded
-                    if ((image.Height < 64) && (image.Width < 64))
-                    {
-                        newsize = new Size(32, 32);
-                    }
-                    else if ((image.Height < 128) && (image.Width < 128))
-                    {
-                        newsize = new Size(64, 64);
-                    }
-                    else if ((image.Height < 256) && (image.Width < 256))
-                    {
-                        newsize = new Size(128, 128);
-                    }
-                    else if ((image.Height < 512 && image.Width < 512))
-                    {
-                        newsize = new Size(256, 256);
-                    }
-                    else if ((image.Height < 1024 && image.Width < 1024))
+                    stream = response.GetResponseStream();
+                    if(stream != null)
                     {
-                        newsize = new Size(512, 512);
+                                                Bitmap image = new Bitmap(stream);
+                                                Size newsize;
+
+                                                // TODO: make this a bit less hard coded
+                                                if ((image.Height < 64) && (image.Width < 64))
+                                                {
+                                                        newsize = new Size(32, 32);
+                                                }
+                                                else if ((image.Height < 128) && (image.Width < 128))
+                                                {
+                                                        newsize = new Size(64, 64);
+                                                }
+                                                else if ((image.Height < 256) && (image.Width < 256))
+                                                {
+                                                        newsize = new Size(128, 128);
+                                                }
+                                                else if ((image.Height < 512 && image.Width < 512))
+                                                {
+                                                        newsize = new Size(256, 256);
+                                                }
+                                                else if ((image.Height < 1024 && image.Width < 1024))
+                                                {
+                                                        newsize = new Size(512, 512);
+                                                }
+                                                else
+                                                {
+                                                        newsize = new Size(1024, 1024);
+                                                }
+
+                                                Bitmap resize = new Bitmap(image, newsize);
+                                                
+                                                try
+                                                {
+                                                        imageJ2000 = OpenJPEG.EncodeFromImage(resize, true);
+                                                } 
+                                                catch (Exception)
+                                                {
+                                                        m_log.Error("[LOADIMAGEURLMODULE]: OpenJpeg Encode Failed.  Empty byte data returned!");
+                                                }
                     }
                     else
                     {
-                        newsize = new Size(1024, 1024);
-                    }
-
-                    Bitmap resize = new Bitmap(image, newsize);
-                    byte[] imageJ2000 = new byte[0];
-                    
-                    try
-                    {
-                        imageJ2000 = OpenJPEG.EncodeFromImage(resize, true);
-                    } 
-                    catch (Exception)
-                    {
-                        m_log.Error("[LOADIMAGEURLMODULE]: OpenJpeg Encode Failed.  Empty byte data returned!");
+                                                m_log.WarnFormat("[LOADIMAGEURLMODULE] No data returned");
                     }
-
-                    m_textureManager.ReturnData(state.RequestID, imageJ2000);
-                    return;
                 }
             }
             catch (WebException)
             {
                 
             }
+            finally
+            {
+                if(stream != null)
+                {
+                    stream.Close();
+                }
+            }
+            m_log.DebugFormat("[LOADIMAGEURLMODULE] Returning {0} bytes of image data for request {1}",
+                                       imageJ2000.Length, state.RequestID);
+            m_textureManager.ReturnData(state.RequestID, imageJ2000);
         }
 
         #region Nested type: RequestState