Closed Bug 1225279 Opened 9 years ago Closed 9 years ago

Enable safebrowsing on b2g by default

Categories

(Firefox OS Graveyard :: General, defect)

Product:
Firefox OS Graveyard
Component:
General
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox45 fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox45 --- fixed

People

(Reporter: fabrice, Assigned: fabrice)

References

Details

Attachments

(1 file)

default-safebrowsing.patch
2.07 KB, patch
gwagner
: review+
Details | Diff | Splinter Review 
We don't target low end devices anymore so there's no resource issues to worry about.
Tested by opening http://www.itisatrap.org/firefox/its-an-attack.html
We get a nice error page explaining how bad this page is.
Assignee: nobody → fabrice

        Attachment #8688097 -
        Flags: review?(anygregor)
Blocks: 1180503

    
    

    
  
What changed since bug 1180503?

    
    

    
  
iirc François was saying that turning on safebrowsing could cost ~ 3MB, and at the time we still had to support 256MB configuration. This is not the case anymore so I think it's fine to turn on by default. It's easy enough to disable on a per-device basis if we need to.

        Attachment #8688097 -
        Flags: review?(anygregor) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/7fed9cad6d94
Status: NEW → RESOLVED
Closed: 9 years ago

          status-firefox45:
          --- → fixed
Resolution: --- → FIXED

    
    

    
  
This doesn't download stuff while on a mobile data connection?

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: