Remove DCHECK() in FindInPageNotificationObserver::Observe that gets triggered when FindInPage automation messages come in too quickly.

BUG=Multiple AutomationMsg_Find messages sent rapidly to Chrome (without waiting for the synchronous responses) will trigger this DCHECK().

TEST=Send many AutomationMsg_Find messages very quickly without waiting for responses. Ensure no DCHECKing.
Review URL: http://codereview.chromium.org/155644

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20885 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 7 deletions

diff --git a/chrome/browser/automation/automation_provider.cc b/chrome/browser/automation/automation_provider.cc
index 36e348d..ed4fd46 100644
--- a/chrome/browser/automation/automation_provider.cc
+++ b/chrome/browser/automation/automation_provider.cc
@@ -634,13 +634,14 @@ class FindInPageNotificationObserver : public NotificationObserver {
         if (find_details->active_match_ordinal() > -1)
           active_match_ordinal_ = find_details->active_match_ordinal();
         if (find_details->final_update()) {
-          DCHECK(reply_message_ != NULL);
-
-          AutomationMsg_FindInPage::WriteReplyParams(reply_message_,
-              active_match_ordinal_, find_details->number_of_matches());
-
-          automation_->Send(reply_message_);
-          reply_message_ = NULL;
+          if (reply_message_ != NULL) {
+            AutomationMsg_FindInPage::WriteReplyParams(reply_message_,
+                active_match_ordinal_, find_details->number_of_matches());
+            automation_->Send(reply_message_);
+            reply_message_ = NULL;
+          } else {
+            DLOG(WARNING) << "Multiple final Find messages observed.";
+          }
         } else {
           DLOG(INFO) << "Ignoring, since we only care about the final message";
         }