Added histograms to https://codereview.chromium.org/207553008/.

chrome/renderer/net/net_error_helper_core.h

Index: chrome/renderer/net/net_error_helper_core.h
diff --git a/chrome/renderer/net/net_error_helper_core.h b/chrome/renderer/net/net_error_helper_core.h
index ce053a6f3718ae8c8793f75bfc3fb0f59b7ae7d3..a74fe67f82ad2a0d7257afde1c004e2bfeddee1c 100644
--- a/chrome/renderer/net/net_error_helper_core.h
+++ b/chrome/renderer/net/net_error_helper_core.h
@@ -213,8 +213,9 @@ class NetErrorHelperCore {
   int auto_reload_count_;
   bool can_auto_reload_page_;
 
-  // Non-NO_BUTTON only when a navigation has been initiated from the error
-  // page.  Used to detect when such navigations result in errors.
+  // This value is set (|navigation_from_button_ != NO_BUTTON|) only

[jar (doing other things), 2014/04/23 01:58:51]

nit: I actually like the new prose much better... but would prefer to omit the
pseudo code (as I waste too much time reading the double negative... and don't
get more out of it that the prose).

This value is set only when a navigation has been initiated from the error page.

YMMV... not worth arguing.  The prose helped me a lot. thx!

[Randy Smith (Not in Mondays), 2014/04/23 18:36:59]

Yeah, I wasn't sure that everyone would realize that "NO_BUTTON" is the same as
not set.  But It's clear enough, and I agree with you the text is better without
the pseudo-code.  Done.

+  // when a navigation has been initiated from the error page.
+  // It is used to detect when such navigations result in errors.
   Button navigation_from_button_;
 };