<br><br><div class="gmail_quote">On Fri, May 20, 2011 at 9:32 AM, Sebastian Silva <span dir="ltr"><<a href="mailto:sebastian@somosazucar.org">sebastian@somosazucar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">



  
    
  
  <div bgcolor="#ffffff" text="#000000">
    Here's the patch until now. This avoids a growing instance dir
    except for the case when the machine or browse crash.<br>
    Its not the cache but the partial downloads that fill up the disk
    (generally firefox saves these partial downloads in the filesystem
    along with the download. Browse puts these in this temporary
    directory before transferring them to the journal. <br>
    <br>
    I think its safe to assume if we are the only instance of Browse and
    there are partial downloads in the instance dir, we should remove
    them.<br>
    I'm trying to figure out how to check for this condition.<br>
    <br>
    Regards,<br>
    Sebastian<br>
    <br></div></blockquote><div><br></div><div>I did a change to the patch but it's o.k, other corner case that I encountered is when there </div><div>are already .part files in the /instance directory,</div><div>these should be erased when browse starts. </div>

<div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div bgcolor="#ffffff" text="#000000">
    El 20/05/11 08:56, Gonzalo Odiard escribió:
    <div><div></div><div class="h5"><blockquote type="cite">Part of the problem is the user does not know nothing
      about the existence of this files.<br>
      <br>
      Gonzalo<br>
      <br>
      <div class="gmail_quote">On Fri, May 20, 2011 at 9:51 AM, Walter
        Bender <span dir="ltr"><<a href="mailto:walter.bender@gmail.com" target="_blank">walter.bender@gmail.com</a>></span>
        wrote:<br>
        <blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><br>
          <br>
          <div class="gmail_quote">
            <div>
              <div>On Fri, May 20, 2011 at 1:24 AM, Sebastian
                Silva <span dir="ltr"><<a href="mailto:sebastian@somosazucar.org" target="_blank">sebastian@somosazucar.org</a>></span>
                wrote:<br>
              </div>
            </div>
            <blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">
              <div>
                <div>
                  <div bgcolor="#ffffff" text="#000000"> I've
                    implemented the first part of this patch.<br>
                    It deletes partial downloads when user clicks on
                    cancel button or deletes journal download entry.<br>
                    This should already mitigate thie situation. Alas,
                    the partial download still remains when there is a
                    crash. <br>
                    <br>
                    I could remove partial downloads on start of
                    activity, but i'm worried another instance of Browse
                    might be open and downloading.<br>
                    How should I check for this condition?<br>
                    <br>
                    Thanks<br>
                    Sebastian<br>
                    <br>
                    El 19/05/11 20:20, Sebastian Silva escribió:
                    <div>
                      <div>
                        <blockquote type="cite"> Hi,<br>
                          dirakx and I are working on Browse and are
                          trying to find a way to fix <br>
                          <a href="http://bugs.sugarlabs.org/ticket/2659" target="_blank">http://bugs.sugarlabs.org/ticket/2659</a>
                          - [dx2] Browse should have a cache cleaning
                          policy<span style="border-collapse:separate;color:rgb(0, 0, 0);font-family:'Times New Roman';font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;font-size:medium"><span style="font-family:Verdana,Arial,'Bitstream Vera Sans',Helvetica,sans-serif;font-size:14px"></span></span><br>


                          <br>
                          It seems to us the only way to bloat the
                          .sugar/default/org.laptop.WebActivity/instance
                          dir is by doing a large download and then
                          interrupting it (either because of network
                          problems or by canceling). This leaves partial
                          downloads in this dir and can fill the disk
                          pretty quickly on XO hardware.<br>
                          <br>
                          We have thought of a workaround policy as we
                          can't find an option in xulrunner to handle
                          this by removing these partial files:<br>
                              - on click of "cancel" in a download -
                          erase partial download<br>
                              - on start of new session of browse (if no
                          other instance is running)<br>
                          <br>
                          Is this an acceptable fix?<br>
                          Input welcome as we prepare the patch,<br>
                          <br>
                          Cheers,<br>
                          Sebastian<br>
                        </blockquote>
                        <br>
                      </div>
                    </div>
                  </div>
                  <br>
                </div>
              </div>
              _______________________________________________<br>
              Sugar-devel mailing list<br>
              <a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
              <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
              <br>
            </blockquote>
          </div>
          <br>
          Why not have a button for clearing the cache regardless of the
          circumstances by which it filled? As to where to put the
          button...<br>
          <br>
          -walter<br clear="all">
          <font color="#888888"><br>
            -- <br>
            Walter Bender<br>
            Sugar Labs<br>
            <a href="http://www.sugarlabs.org" target="_blank">http://www.sugarlabs.org</a><br>
            <br>
          </font><br>
          _______________________________________________<br>
          Sugar-devel mailing list<br>
          <a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
          <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
          <br>
        </blockquote>
      </div>
      <br>
    </blockquote>
    <br>
  </div></div></div>

<br>_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br></blockquote></div><br>