Since we know it happens, UMA is not very useful, at least initially, since it only tells you that it happened, which we already know is true, but it does not say how. DumpWithoutCrashing will give us the context on how it happens and should be the first step. Whether we remove it later on is a different question, but I feel SSL state is important enough to have a permanent DWC stay in the code to help us understand when and how it gets in that state.

From my experience as the author of so many of these bugs, the dump is often not helpful. i.e. it shows where this happens, but it doesn't show how which is the hard part.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/969cdd94ea8e0434c090f723abca705485c533e6

commit 969cdd94ea8e0434c090f723abca705485c533e6
Author: John Abd-El-Malek <jam@chromium.org>
Date: Mon Jul 10 22:18:16 2017

Add histogram of how often an https URL commits without a certificate.

BUG= 736375 

Change-Id: I109fab5e41a40ba0d6f5f478c31070d7ef4ea9df
Reviewed-on: https://chromium-review.googlesource.com/565196
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Eric Lawrence <elawrence@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485418}
[modify] https://crrev.com/969cdd94ea8e0434c090f723abca705485c533e6/content/browser/frame_host/navigation_controller_impl.cc
[modify] https://crrev.com/969cdd94ea8e0434c090f723abca705485c533e6/tools/metrics/histograms/histograms.xml

does anyone think this should be merged to M60?

Thanks for fixing this, jam@!

More data is always good (and the CertMissing% we're seeing from the first half day of 61 Canary is higher than we'd hoped). 

That said, I understand that the bar for merges has gone up quite a bit in M60, so I'm not sure if the merge would be accepted?

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9cd697e09f700f73a45680ea445bdd8075794472

commit 9cd697e09f700f73a45680ea445bdd8075794472
Author: John Abd-El-Malek <jam@chromium.org>
Date: Wed Jul 12 21:53:14 2017

Add more logging to figure out which navigation codepaths lose the SSL certificate.

BUG= 736375 

Change-Id: I73c225fed86f536074f6886d22f9a65e43fcde3e
Reviewed-on: https://chromium-review.googlesource.com/567495
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#486111}
[modify] https://crrev.com/9cd697e09f700f73a45680ea445bdd8075794472/content/browser/frame_host/navigation_controller_impl.cc
[modify] https://crrev.com/9cd697e09f700f73a45680ea445bdd8075794472/tools/metrics/histograms/histograms.xml

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7ba40786c7640752a1d2cdb0d1e769565af66f0d

commit 7ba40786c7640752a1d2cdb0d1e769565af66f0d
Author: John Abd-El-Malek <jam@chromium.org>
Date: Tue Jul 18 00:46:11 2017

Add a missing entry to a histogram enum.

I missed this in r486111

BUG= 736375 

Change-Id: I26b4381945063f7216bab29412bfe0df1c3d95a9
Reviewed-on: https://chromium-review.googlesource.com/575393
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487321}
[modify] https://crrev.com/7ba40786c7640752a1d2cdb0d1e769565af66f0d/tools/metrics/histograms/histograms.xml

@elawrence @estark: should i just paste here the data from the last week and then remove the histograms? Or do you want to keep them checked in?

I don't know of any reason to remove the breakout histograms until we drive the overall "Missing SSL Status" count to zero and it stays there for a while.