You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2011/09/29 18:34:24 UTC

svn commit: r1177354 - in /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure: Page.java PageImpl.java

Author: hlship
Date: Thu Sep 29 16:34:24 2011
New Revision: 1177354

URL: http://svn.apache.org/viewvc?rev=1177354&view=rev
Log:
TAP5-1678: Remove Page.getLastAttachTime(), no longer used

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/Page.java
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/PageImpl.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/Page.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/Page.java?rev=1177354&r1=1177353&r2=1177354&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/Page.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/Page.java Thu Sep 29 16:34:24 2011
@@ -182,13 +182,4 @@ public interface Page
      * Invoked to notify {@link PageResetListener} listeners.
      */
     void pageReset();
-
-    /**
-     * Returns time, in milliseconds, that the page was last attached. This is used by the {@link org.apache.tapestry5.internal.services.PageSource} service to
-     * cull unused pages from memory.
-     *
-     * @return milliseconds time of last {@link #attached()}} call
-     * @since 5.3
-     */
-    long getLastAttachTime();
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/PageImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/PageImpl.java?rev=1177354&r1=1177353&r2=1177354&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/PageImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/structure/PageImpl.java Thu Sep 29 16:34:24 2011
@@ -50,8 +50,6 @@ public class PageImpl implements Page
 
     private final Map<String, ComponentPageElement> idToComponent = CollectionFactory.newCaseInsensitiveMap();
 
-    private volatile long lastAttach = System.currentTimeMillis();
-
     /**
      * Obtained from the {@link org.apache.tapestry5.internal.services.PersistentFieldManager} when
      * first needed,
@@ -176,8 +174,6 @@ public class PageImpl implements Page
 
     public void attached()
     {
-        lastAttach = System.currentTimeMillis();
-
         for (PageLifecycleListener listener : lifecycleListeners)
             listener.restoreStateBeforePageAttach();
 
@@ -232,11 +228,6 @@ public class PageImpl implements Page
         }
     }
 
-    public long getLastAttachTime()
-    {
-        return lastAttach;
-    }
-
     public boolean hasResetListeners()
     {
         return !resetListeners.isEmpty();