Reduce generic warnings shown on dmg downloads

Reduce generic warnings shown on dmg downloads.
Old behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome shows the generic warning;
(2) if dmg analysis fails, and the user is not a safe browsing
   extended reporting user (or in incognito mode), chrome shows generic
   warning without asking SB backend.

New behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome regards it as SAFE.
(2) if dmg analysis fails, send ping to SB backend and let it decide.

BUG=621936
Committed: https://crrev.com/3c9ad45a9517df20e14c1a902922615f838d35cd
Cr-Commit-Position: refs/heads/master@{#402551}

[Jialiu Lin, 2016-06-27 23:25:13]

Patchset #1 (id:1) has been deleted

[Jialiu Lin, 2016-06-27 23:25:20]

Patchset #2 (id:40001) has been deleted

[Jialiu Lin, 2016-06-27 23:28:40]

Description was changed from

==========
allow sending pings when dmg analyzer fails

BUG=
==========

to

==========
Reduce generic warnings shown on dmg downloads.
Old behavior:
(1) if dmg is successfully analyzed and has no executable inside, chrome shows
the generic warning; 
(2) if dmg analysis fails, and the user is not a safe browsing extended
reporting user (or in incognito mode), chrome shows generic warning without
asking SB backend. 

New behavior:
(1) if dmg is successfully analyzed and has no executable inside, chrome regards
it as SAFE. 
(2) if dmg analysis fails, send ping to SB backend and let it decide. 


BUG=621936
==========

[Jialiu Lin, 2016-06-27 23:29:11]

Description was changed from

==========
Reduce generic warnings shown on dmg downloads.
Old behavior:
(1) if dmg is successfully analyzed and has no executable inside, chrome shows
the generic warning; 
(2) if dmg analysis fails, and the user is not a safe browsing extended
reporting user (or in incognito mode), chrome shows generic warning without
asking SB backend. 

New behavior:
(1) if dmg is successfully analyzed and has no executable inside, chrome regards
it as SAFE. 
(2) if dmg analysis fails, send ping to SB backend and let it decide. 


BUG=621936
==========

to

==========
Reduce generic warnings shown on dmg downloads.
Old behavior:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome shows the generic warning; 
(2) if dmg analysis fails, and the user is not a safe browsing
   extended reporting user (or in incognito mode), chrome shows generic
   warning without asking SB backend. 

New behavior:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome regards it as SAFE. 
(2) if dmg analysis fails, send ping to SB backend and let it decide. 


BUG=621936
==========

[Jialiu Lin, 2016-06-27 23:29:25]

Description was changed from

==========
Reduce generic warnings shown on dmg downloads.
Old behavior:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome shows the generic warning; 
(2) if dmg analysis fails, and the user is not a safe browsing
   extended reporting user (or in incognito mode), chrome shows generic
   warning without asking SB backend. 

New behavior:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome regards it as SAFE. 
(2) if dmg analysis fails, send ping to SB backend and let it decide. 


BUG=621936
==========

to

==========
Reduce generic warnings shown on dmg downloads.
Old behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome shows the generic warning; 
(2) if dmg analysis fails, and the user is not a safe browsing
   extended reporting user (or in incognito mode), chrome shows generic
   warning without asking SB backend. 

New behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome regards it as SAFE. 
(2) if dmg analysis fails, send ping to SB backend and let it decide. 


BUG=621936
==========

[Jialiu Lin, 2016-06-28 01:36:25]

jialiul@chromium.org changed reviewers:
+ nparker@chromium.org

[Nathan Parker, 2016-06-28 19:09:21]

lgtm

[Jialiu Lin, 2016-06-28 20:02:34]

The CQ bit was checked by jialiul@chromium.org

[commit-bot: I haz the power, 2016-06-28 20:04:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-28 21:38:14]

Committed patchset #2 (id:2)

[commit-bot: I haz the power, 2016-06-28 21:39:41]

Description was changed from

==========
Reduce generic warnings shown on dmg downloads.
Old behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome shows the generic warning; 
(2) if dmg analysis fails, and the user is not a safe browsing
   extended reporting user (or in incognito mode), chrome shows generic
   warning without asking SB backend. 

New behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome regards it as SAFE. 
(2) if dmg analysis fails, send ping to SB backend and let it decide. 


BUG=621936
==========

to

==========
Reduce generic warnings shown on dmg downloads.
Old behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome shows the generic warning;
(2) if dmg analysis fails, and the user is not a safe browsing
   extended reporting user (or in incognito mode), chrome shows generic
   warning without asking SB backend.

New behaviors:
(1) if dmg is successfully analyzed and has no executable inside,
    chrome regards it as SAFE.
(2) if dmg analysis fails, send ping to SB backend and let it decide.

BUG=621936

Committed: https://crrev.com/3c9ad45a9517df20e14c1a902922615f838d35cd
Cr-Commit-Position: refs/heads/master@{#402551}
==========

[commit-bot: I haz the power, 2016-06-28 21:39:42]

Patchset 2 (id:??) landed as
https://crrev.com/3c9ad45a9517df20e14c1a902922615f838d35cd
Cr-Commit-Position: refs/heads/master@{#402551}