Closed Bug 317627 Opened 19 years ago Closed 19 years ago

Display hex dump of cache data in about:cache-entry

Categories

(Core :: Networking: Cache, enhancement)

Product:
Core
Component:
Networking: Cache
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9alpha1

People

(Reporter: darin.moz, Assigned: darin.moz)

Details

Attachments

(1 file, 2 obsolete files)

v1.2 patch
4.51 KB, patch
Biesinger
: review+
Details | Diff | Splinter Review 
Display hex dump of cache data in about:cache-entry

There are times when you really want to see the data for a cache entry.  It's really easy to add a hexdump of the data to about:cache-entry, so why not? ;-)

Patch coming up...
Attached patch v1 patch (obsolete) — — Splinter Review 

        Attachment #204085 -
        Flags: review?(cbiesinger)

    
    

    
  
hm... what does isprint return for \n or \r? (\v ?) it's also locale-sensitive... although that shouldn't cause problems for chars < 0x80, I think.

    
    

    
  

      
      
      
      

        Attached patch
          
          
        v1.1 patch (obsolete)
        — 
          — Splinter Review
      

    


  
better is printable handling.

        Attachment #204085 -
        Attachment is obsolete: true

        Attachment #204090 -
        Flags: review?(cbiesinger)

        Attachment #204085 -
        Flags: review?(cbiesinger)

    
    

    
  

      
      
      
      

        Attached patch
          
          
        v1.2 patch
        — 
          — Splinter Review
      

    


  
Slightly improved patch.

        Attachment #204090 -
        Attachment is obsolete: true

        Attachment #205342 -
        Flags: review?(cbiesinger)

        Attachment #204090 -
        Flags: review?(cbiesinger)

    
    

    
  
Comment on attachment 205342 [details] [diff] [review]
v1.2 patch

hm... this always dumps the entire cache contents? might lead to a long page. either way, r=biesi

        Attachment #205342 -
        Flags: review?(cbiesinger) → review+

    
    

    
  
Yeah, I decided that it was more useful to have a way to actually see the cache contents than to have a responsive UI.  Afterall, about:cache is a hidden feature of the developer oriented browser.

    
    

    
  
> Afterall, about:cache is a hidden feature of the developer oriented browser.

Uhm... no coffee yet.  I meant:

 "Afterall, about:cache is a hidden developer-oriented feature of the browser."

    
    

    
  
fixed-on-trunk
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: