From 849890b62b75f95cf91ebd3fe49f1525ff6595bd Mon Sep 17 00:00:00 2001 From: "brettw@google.com" Date: Fri, 15 Aug 2008 20:10:49 +0000 Subject: Cleans up notifications for the NavigationController. There were several notifications before and some of them were very unclear and misused (STATE_CHANGED). This one, and PRUNED were called unnecessarily in some cases as well. I replaced STATE_CHANGED and INDEX_CHANGED with ENTRY_COMMITTED which is more clear and covers (I think!) all the cases that the callers care about. I added a simple notification testing helper class, and used in the navigation controller unit tests to make sure we get the proper notifications. I had to change NotificationSource/Details to have a = and copy constructor so I can track them easily in my helper. I don't see why this would be bad. As part of this, I got very frustrated recompiling the world whenever navigation_types.h changed. So I removed this dependency from the notification service which everybody includes. Most of the changed files are adding notification_types.h in the .cc file where it's needed. BUG=1325636,1321376,1325779 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@956 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/ssl_policy.cc | 1 - 1 file changed, 1 deletion(-) (limited to 'chrome/browser/ssl_policy.cc') diff --git a/chrome/browser/ssl_policy.cc b/chrome/browser/ssl_policy.cc index 1ae9ae1..4f5f914 100644 --- a/chrome/browser/ssl_policy.cc +++ b/chrome/browser/ssl_policy.cc @@ -346,7 +346,6 @@ class DefaultPolicy : public SSLPolicy { NavigationEntry* entry = navigation_controller->GetActiveEntry(); entry->SetHasMixedContent(); - navigation_controller->EntryUpdated(entry); } void OnDenyCertificate(SSLManager::CertError* error) { -- cgit v1.1