Don't include ERR_ABORTED in net error histogram

Don't include ERR_ABORTED in net error histogram

BUG=none
TEST=none

[agl, 2010-12-13 22:28:01]

This is just a thought, if it would be helpful.

[eroman, 2010-12-13 23:00:17]

LGTM

On Mon, Dec 13, 2010 at 2:28 PM,  <agl@chromium.org> wrote:
> Reviewers: eroman,
>
> Message:
> This is just a thought, if it would be helpful.
>
> Description:
> Don't include ERR_ABORTED in net error histogram
>
> BUG=none
> TEST=none
>
> Please review this at http://codereview.chromium.org/5803003/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files:
>  M chrome/browser/renderer_host/resource_dispatcher_host.cc
>
>
> Index: chrome/browser/renderer_host/resource_dispatcher_host.cc
> diff --git a/chrome/browser/renderer_host/resource_dispatcher_host.cc
> b/chrome/browser/renderer_host/resource_dispatcher_host.cc
> index
>
9a4953e74b5dd38d58bee3b000ddf5e39292aaea..19432131e89bce6d07b3ac9edccaf64b00cf675b
> 100644
> --- a/chrome/browser/renderer_host/resource_dispatcher_host.cc
> +++ b/chrome/browser/renderer_host/resource_dispatcher_host.cc
> @@ -1491,7 +1491,8 @@ void
> ResourceDispatcherHost::OnResponseCompleted(net::URLRequest* request) {
>   // If the load for a main frame has failed, track it in a histogram,
>   // since it will probably cause the user to see an error page.
>   if (!request->status().is_success() &&
> -      info->resource_type() == ResourceType::MAIN_FRAME) {
> +      info->resource_type() == ResourceType::MAIN_FRAME &&
> +      request->status().os_error() != net::ERR_ABORTED) {
>     UMA_HISTOGRAM_CUSTOM_ENUMERATION("Net.ErrorCodesForMainFrame",
>                                      -request->status().os_error(),
>                                      GetAllNetErrorCodes());
>
>
>

[eroman, 2010-12-21 00:29:21]

Has this been committed yet? (It still shows up in my code review queue...
please close it).