Makefile.am (ordinary_java_source_files): Add DefaultContentHandlerFactory.java.
authorBryce McKinlay <mckinlay@redhat.com>
Fri, 23 Jul 2004 01:21:40 +0000 (01:21 +0000)
committerBryce McKinlay <bryce@gcc.gnu.org>
Fri, 23 Jul 2004 01:21:40 +0000 (02:21 +0100)
2004-07-22  Bryce McKinlay  <mckinlay@redhat.com>

* Makefile.am (ordinary_java_source_files): Add
DefaultContentHandlerFactory.java.
* Makefile.in: Rebuilt.
* java/net/URLConnection.java (defaultFactory): New field.
(getContent):
(getContentHandler): Renamed from 'setContentHandler'. Try
defaultFactory after user-set factory, if any. Search for content
handler implementations in gnu.java.net.content, not
gnu.gcj.content.
* gnu/java/net/protocol/file/Connection.java (getHeaderField):
Implemented.
(getLastModified): Implemented.
(getPermission): Create file permission here, instead of in
constructor.
* gnu/java/net/protocol/gcjlib/Connection.java (getHeaderField):
Implemented.
* gnu/java/net/protocol/jar/Connection.java (getHeaderField):
Implemented.
(getLastModified): Implemented.
* gnu/java/awt/ClasspathToolkit.java (createImageProducer): New.
Default implementation.
* gnu/java/awt/peer/gtk/GtkToolkit.java (createImageProducer): New.
Implement using GdkPixbufDecoder.

From-SVN: r85069

libjava/ChangeLog
libjava/Makefile.am
libjava/Makefile.in
libjava/gnu/java/awt/ClasspathToolkit.java
libjava/gnu/java/awt/peer/gtk/GtkToolkit.java
libjava/gnu/java/net/protocol/file/Connection.java
libjava/gnu/java/net/protocol/gcjlib/Connection.java
libjava/gnu/java/net/protocol/jar/Connection.java
libjava/java/net/URLConnection.java

index 8b09956c72208904bac7bdc6fc2625b8fdff3f5b..f315687599df823bb2d3f9511d401a4bcfc50213 100644 (file)
@@ -1,3 +1,28 @@
+2004-07-22  Bryce McKinlay  <mckinlay@redhat.com>
+
+       * Makefile.am (ordinary_java_source_files): Add
+       DefaultContentHandlerFactory.java.
+       * Makefile.in: Rebuilt.
+       * java/net/URLConnection.java (defaultFactory): New field.
+       (getContent): 
+       (getContentHandler): Renamed from 'setContentHandler'. Try 
+       defaultFactory after user-set factory, if any. Search for content 
+       handler implementations in gnu.java.net.content, not gnu.gcj.content.
+       * gnu/java/net/protocol/file/Connection.java (getHeaderField):
+       Implemented.
+       (getLastModified): Implemented.
+       (getPermission): Create file permission here, instead of in
+       constructor.
+       * gnu/java/net/protocol/gcjlib/Connection.java (getHeaderField):
+       Implemented.
+       * gnu/java/net/protocol/jar/Connection.java (getHeaderField):
+       Implemented.
+       (getLastModified): Implemented.
+       * gnu/java/awt/ClasspathToolkit.java (createImageProducer): New.
+       Default implementation.
+       * gnu/java/awt/peer/gtk/GtkToolkit.java (createImageProducer): New.
+       Implement using GdkPixbufDecoder.
+       
 2004-07-21  Michael Koch  <konqueror@gmx.de>
 
        * javax/swing/JTextArea.java
        (BasicPopupMenuUI.PopupMenuHandler): Implemented.
        (BasicPopupMenuUI.TopWindowListener): New ComponentListener.
        Implemented.
-       
-       
 
 2004-07-21  Michael Koch  <konqueror@gmx.de>
 
index 470c8a896caa59ae9429e94b335deaeb319bf2cb..005c8bf9e93a717e9c2b6fb24bea2eb22c0f4b22 100644 (file)
@@ -2436,6 +2436,7 @@ gnu/java/locale/LocaleInformation_zh_HK.java \
 gnu/java/locale/LocaleInformation_zh_SG.java \
 gnu/java/locale/LocaleInformation_zh_TW.java \
 gnu/java/math/MPN.java \
+gnu/java/net/DefaultContentHandlerFactory.java \
 gnu/java/net/HeaderFieldHelper.java \
 gnu/java/net/PlainDatagramSocketImpl.java \
 gnu/java/net/PlainSocketImpl.java \
index 08fe01245c6d68ffba3e2172f0fec68d5486111f..9cf6bacb72b7d7958837a960e575092c9d95885c 100644 (file)
@@ -2119,6 +2119,7 @@ gnu/java/locale/LocaleInformation_zh_HK.java \
 gnu/java/locale/LocaleInformation_zh_SG.java \
 gnu/java/locale/LocaleInformation_zh_TW.java \
 gnu/java/math/MPN.java \
+gnu/java/net/DefaultContentHandlerFactory.java \
 gnu/java/net/HeaderFieldHelper.java \
 gnu/java/net/PlainDatagramSocketImpl.java \
 gnu/java/net/PlainSocketImpl.java \
@@ -3089,7 +3090,7 @@ libgcj-test.spec.in libgcj.pc.in libgcj.spec.in
 
 DISTFILES = $(DIST_COMMON) $(SOURCES) $(HEADERS) $(TEXINFOS) $(EXTRA_DIST)
 
-TAR = tar
+TAR = gtar
 GZIP_ENV = --best
 DIST_SUBDIRS =  @DIRLTDL@ testsuite gcj include @DIRLTDL@ gcj include
 DEP_FILES =  .deps/$(srcdir)/$(CONVERT_DIR)/gen-from-JIS.P \
@@ -3389,7 +3390,9 @@ DEP_FILES =  .deps/$(srcdir)/$(CONVERT_DIR)/gen-from-JIS.P \
 .deps/gnu/java/locale/LocaleInformation_zh_HK.P \
 .deps/gnu/java/locale/LocaleInformation_zh_SG.P \
 .deps/gnu/java/locale/LocaleInformation_zh_TW.P \
-.deps/gnu/java/math/MPN.P .deps/gnu/java/net/HeaderFieldHelper.P \
+.deps/gnu/java/math/MPN.P \
+.deps/gnu/java/net/DefaultContentHandlerFactory.P \
+.deps/gnu/java/net/HeaderFieldHelper.P \
 .deps/gnu/java/net/PlainDatagramSocketImpl.P \
 .deps/gnu/java/net/PlainSocketImpl.P .deps/gnu/java/net/URLParseError.P \
 .deps/gnu/java/net/natPlainDatagramSocketImpl.P \
index 91401f4d5a2bb3b03d74c89a0dc4c66016fa9f3a..92934593d3994d2e98b0795338436b3b0cfb9c3e 100644 (file)
@@ -48,6 +48,7 @@ import java.awt.GraphicsEnvironment;
 import java.awt.HeadlessException;
 import java.awt.Toolkit;
 import java.awt.image.ColorModel;
+import java.awt.image.ImageProducer;
 import java.io.File;
 import java.io.InputStream;
 import java.io.IOException;
@@ -331,4 +332,17 @@ public abstract class ClasspathToolkit
         .initCause(muex);
     }
   }
+  
+  /**
+   * Creates an ImageProducer from the specified URL. The image is assumed
+   * to be in a recognised format. If the toolkit does not implement the
+   * image format or the image format is not recognised, null is returned.
+   * This default implementation is overriden by the Toolkit implementations.
+   *
+   * @param url URL to read image data from.
+   */
+  public ImageProducer createImageProducer(URL url)
+  {
+    return null;
+  }
 }
index 9cc29b4b3a4f7244ed7f513e70df5375bc576397..3f82159c3abf6cf9eed4f97de5d0601f93862d02 100644 (file)
@@ -278,6 +278,17 @@ public class GtkToolkit extends gnu.java.awt.ClasspathToolkit
         return image;        
       }
   }
+  
+  /**
+   * Creates an ImageProducer from the specified URL. The image is assumed
+   * to be in a recognised format. 
+   *
+   * @param url URL to read image data from.
+   */  
+  public ImageProducer createImageProducer(URL url)
+  {
+    return new GdkPixbufDecoder(url);  
+  }
 
   public ColorModel getColorModel () 
   {
index 650d9f21853e95bb6072c724a85a580917f65517..4da6e882c002d4cda7de2df41a479e1d58801824 100644 (file)
@@ -50,6 +50,9 @@ import java.net.ProtocolException;
 import java.net.URL;
 import java.net.URLConnection;
 import java.security.Permission;
+import java.text.SimpleDateFormat;
+import java.util.Date;
+import java.util.Locale;
 
 /**
  * This subclass of java.net.URLConnection models a URLConnection via
@@ -62,9 +65,11 @@ import java.security.Permission;
 public class Connection extends URLConnection
 {
   /**
-   * Default permission for a file
+   * HTTP-style DateFormat, used to format the last-modified header.
    */
-  private static final String DEFAULT_PERMISSION = "read";
+  private static SimpleDateFormat dateFormat
+    = new SimpleDateFormat("EEE, dd MMM yyyy hh:mm:ss 'GMT'",
+                           new Locale ("En", "Us", "Unix"));
 
   /**
    * This is a File object for this connection
@@ -81,19 +86,12 @@ public class Connection extends URLConnection
    */
   private OutputStream outputStream;
   
-  /**
-   * FilePermission to read the file
-   */
-  private FilePermission permission;
-
   /**
    * Calls superclass constructor to initialize.
    */
   public Connection(URL url)
   {
     super (url);
-
-    permission = new FilePermission(getURL().getFile(), DEFAULT_PERMISSION);
   }
   
   /**
@@ -156,45 +154,73 @@ public class Connection extends URLConnection
   }
 
   /**
-   * Get the last modified time of the resource.
-   *
-   * @return the time since epoch that the resource was modified.
+   *  Get an http-style header field. Just handle a few common ones. 
    */
-  public long getLastModified()
+  public String getHeaderField(String field)
   {
     try
       {
        if (!connected)
          connect();
 
-       return file.lastModified();
+       if (field.equals("content-type"))
+          return guessContentTypeFromName(file.getName());
+       else if (field.equals("content-length"))
+          return Long.toString(file.length());
+       else if (field.equals("last-modified"))
+         {
+           synchronized (dateFormat)
+             {
+               return dateFormat.format(new Date(file.lastModified()));
+             }
+         }
       }
     catch (IOException e)
       {
-       return -1;
+        // Fall through.
       }
+    return null;
   }
 
   /**
    * Get the length of content.
-   *
    * @return the length of the content.
    */
   public int getContentLength()
+  {
+    try
+      {
+       if (!connected)
+         connect();
+
+       return (int) file.length();
+      }
+    catch (IOException e)
+      {
+       return -1;
+      }
+  }
+
+  /**
+   * Get the last modified time of the resource.
+   *
+   * @return the time since epoch that the resource was modified.
+   */
+  public long getLastModified()
   {
     try
       {
        if (!connected)
          connect();
-        
-       return (int) file.length();
+
+       return file.lastModified();
       }
     catch (IOException e)
       {
        return -1;
       }
   }
-  
+
   /**
    * This method returns a <code>Permission</code> object representing the
    * permissions required to access this URL.  This method returns a
@@ -205,6 +231,6 @@ public class Connection extends URLConnection
    */
   public Permission getPermission() throws IOException
   {
-    return permission;
+    return new FilePermission(getURL().getFile(), "read");
   }
 }
index b31db56577684d01ae74c36f98aa4696dd164915..4e6e462f26eab5d67e6233257d73d892a1e46b28 100644 (file)
@@ -61,4 +61,23 @@ class Connection extends URLConnection
     connect();
     return new CoreInputStream(core);
   }
+  
+  public String getHeaderField(String field)
+  {
+    try
+      {
+       if (!connected)
+         connect();
+
+       if (field.equals("content-type"))
+          return guessContentTypeFromName(name);
+       else if (field.equals("content-length"))
+          return Long.toString(core.length);
+      }
+    catch (IOException e)
+      {
+        // Fall through.
+      }
+    return null;
+  }  
 }
index 04eb3fe3715f168b1086c292e466088ac544bcb4..9ba2e3fd1ffc8850b76c534819a187f39c33b14b 100644 (file)
@@ -49,8 +49,11 @@ import java.net.MalformedURLException;
 import java.net.ProtocolException;
 import java.net.URL;
 import java.net.URLConnection;
+import java.text.SimpleDateFormat;
+import java.util.Date;
 import java.util.HashMap;
 import java.util.Hashtable;
+import java.util.Locale;
 import java.util.jar.JarEntry;
 import java.util.jar.JarFile;
 import java.util.jar.JarInputStream;
@@ -66,6 +69,14 @@ import java.util.zip.ZipFile;
 public final class Connection extends JarURLConnection
 {
   private static Hashtable file_cache = new Hashtable();
+
+  /**
+   * HTTP-style DateFormat, used to format the last-modified header.
+   */
+  private static SimpleDateFormat dateFormat
+    = new SimpleDateFormat("EEE, dd MMM yyyy hh:mm:ss 'GMT'",
+                           new Locale ("En", "Us", "Unix"));
+
   private JarFile jar_file;
 
   /**
@@ -219,6 +230,32 @@ public final class Connection extends JarURLConnection
     return jar_file;
   }
 
+  public String getHeaderField(String field)
+  {
+    try
+      {
+       if (!connected)
+         connect();
+
+       if (field.equals("content-type"))
+          return guessContentTypeFromName(getJarEntry().getName());
+       else if (field.equals("content-length"))
+          return Long.toString(getJarEntry().getSize());
+       else if (field.equals("last-modified"))
+         {
+           synchronized (dateFormat)
+             {
+               return dateFormat.format(new Date(getJarEntry().getTime()));
+             }
+         }
+      }
+    catch (IOException e)
+      {
+        // Fall through.
+      }
+    return null;
+  }
+
   public int getContentLength()
   {
     if (!connected)
@@ -233,4 +270,19 @@ public final class Connection extends JarURLConnection
        return -1;
       }
   }
+
+  public long getLastModified()
+  {
+    if (!connected)
+      return -1;
+
+    try
+      {
+       return getJarEntry().getTime();
+      }
+    catch (IOException e)
+      {
+       return -1;
+      }
+  }
 }
index 7b5f6a5dd187c38caa38a7a65e2055d029ae7c84..4bf52b860cd1be16e07de615d143cbc64e578526 100644 (file)
@@ -116,6 +116,9 @@ public abstract class URLConnection
    */
   private static boolean defaultUseCaches = true;
 
+  private static ContentHandlerFactory defaultFactory
+    = new gnu.java.net.DefaultContentHandlerFactory();
+
   /**
    * This variable determines whether or not interaction is allowed with
    * the user.  For example, to prompt for a username and password.
@@ -436,7 +439,7 @@ public abstract class URLConnection
     // guessContentTypeFromName() and guessContentTypeFromStream methods
     // as well as FileNameMap class & fileNameMap field & get/set methods.
     String type = getContentType();
-    ContentHandler ch = setContentHandler(type);
+    ContentHandler ch = getContentHandler(type);
 
     if (ch == null)
       return getInputStream();
@@ -963,7 +966,7 @@ public abstract class URLConnection
     fileNameMap = map;
   }
 
-  private ContentHandler setContentHandler(String contentType)
+  private ContentHandler getContentHandler(String contentType)
   {
     ContentHandler handler;
 
@@ -981,12 +984,17 @@ public abstract class URLConnection
       else
        return null;
 
-    // If a non-default factory has been set, use it to find the content type.
+    // If a non-default factory has been set, use it.
     if (factory != null)
       handler = factory.createContentHandler(contentType);
 
-    // Non-default factory may have returned null or a factory wasn't set.
-    // Use the default search algorithm to find a handler for this content type.
+    // Now try default factory. Using this factory to instantiate built-in
+    // content handlers is preferable  
+    if (handler == null)
+      handler = defaultFactory.createContentHandler(contentType);
+
+    // User-set factory has not returned a handler. Use the default search 
+    // algorithm.
     if (handler == null)
       {
        // Get the list of packages to check and append our default handler
@@ -995,7 +1003,7 @@ public abstract class URLConnection
        // ever be needed (or available).
        String propVal = System.getProperty("java.content.handler.pkgs");
        propVal = (propVal == null) ? "" : (propVal + "|");
-       propVal = propVal + "gnu.gcj.content|sun.net.www.content";
+       propVal = propVal + "gnu.java.net.content|sun.net.www.content";
 
        // Replace the '/' character in the content type with '.' and
        // all other non-alphabetic, non-numeric characters with '_'.