Remove check for usage stats in the toast experiment...

Remove check for usage stats in the toast experiment

BUG=none
TEST= same as before but you know don't need to opt-in for sending crash reports or usage stats

TBR=rahulk



Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=34130

[Glenn Wilson, 2009-12-09 16:12:10]

On 2009/12/09 01:43:07, cpu wrote:
> 

lgtm

[rahulk, 2009-12-09 17:21:20]

lgtm.

On Tue, Dec 8, 2009 at 5:43 PM, <cpu@chromium.org> wrote:

> Reviewers: rahulk, gwilson,
>
> Description:
> Remove check for usage stats in the toast experiment
>
> BUG=none
> TEST= same as before but you know don't need to opt-in for sending crash
> reports
> or usage stats
>
> TBR=rahulk
>
>
>
> Please review this at http://codereview.chromium.org/464078
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     chrome/installer/util/google_chrome_distribution.cc
>
>
> Index: chrome/installer/util/google_chrome_distribution.cc
> ===================================================================
> --- chrome/installer/util/google_chrome_distribution.cc (revision 34119)
> +++ chrome/installer/util/google_chrome_distribution.cc (working copy)
> @@ -439,10 +439,6 @@
>   if ((installer_util::NEW_VERSION_UPDATED != status) || system_install)
>     return;
>
> -  // If user has not opted-in for usage stats  we don't do the
> experiments.
> -  if (!GoogleUpdateSettings::GetCollectStatsConsent())
> -    return;
> -
>   // currently only two equal experiment groups. 90% get the welcome back
> url.
>   int flavor = (base::RandDouble() > 0.1) ? 0 : 1;
>
>
>
>