content: convert SSL notifications to observer usage (take 2)

content: convert SSL notifications to observer usage (take 2)

This is a resubmit of https://codereview.chromium.org/12041059

BUG=175308, 170921
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182187

[Paweł Hajdan Jr., 2013-02-12 11:41:17]

I've added a NULL check in ssl_blocking_page.cc, so this should fix the crash,
but I was wondering how to write a test for this.

It's not obvious for me how to trigger the crash in a test. If you want to ask
other people about this, please add them directly to review to decrease latency.

[jam, 2013-02-12 17:49:36]

lgtm

for future reference, it's helpful when reviewing reverted changes to first
upload the original change, and then upload the fix. that way one can do a
patchset-diff to see what new code went in since the last review.

On 2013/02/12 11:41:17, Paweł Hajdan Jr. wrote:
> I've added a NULL check in ssl_blocking_page.cc, so this should fix the crash,
> but I was wondering how to write a test for this.

i haven't looked at the bug in detail to know what the repro steps are.

> 
> It's not obvious for me how to trigger the crash in a test. If you want to ask
> other people about this, please add them directly to review to decrease
latency.