Determine the element location and click synchronously on the renderer.

chrome/browser/automation/automation_provider_observers.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
 #define CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
 #pragma once
 
 #include <deque>
 #include <map>
@@ skipping 33 matching lines @@
 #include "chrome/browser/password_manager/password_store_consumer.h"
 #include "chrome/browser/search_engines/template_url_service_observer.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 #include "chrome/common/automation_constants.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/download_manager.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/notification_types.h"
+#include "ui/gfx/point.h"
 #include "ui/gfx/size.h"
 
 class AutomationProvider;
 class BalloonCollection;
 class Browser;
 class Extension;
 class ExtensionProcessManager;
 class ExtensionService;
 class InfoBarTabHelper;
 class Notification;
 class Profile;
 class SavePackage;
 class TranslateInfoBarDelegate;
 
+namespace automation {
+class Error;
+}
+
 #if defined(OS_CHROMEOS)
 namespace chromeos {
   class ExistingUserController;
 }
 #endif  // defined(OS_CHROMEOS)
 
 namespace IPC {
 class Message;
 }
 
@@ skipping 1230 matching lines @@
 
   base::WeakPtr<AutomationProvider> automation_;
   scoped_ptr<IPC::Message> reply_message_;
   TabContentsWrapper* tab_contents_;
   FilePath image_path_;
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(PageSnapshotTaker);
 };
 
+// Observes when an automation requested mouse event is processed
+// by the renderer and sends an automation reply.
+class MouseEventProcessedObserver : public TabEventObserver,
+                                    public content::NotificationObserver {
+ public:
+  MouseEventProcessedObserver(AutomationProvider* automation,
+                              IPC::Message* reply_message,
+                              TabContentsWrapper* tab_contents);
+  virtual ~MouseEventProcessedObserver();
+
+ private:
+  // TabEventObserver overrides.
+  virtual void OnWillProcessMouseEventAt(const gfx::Point& point) OVERRIDE;
+  virtual void OnProcessMouseEventACK(
+      bool success,
+      const std::string& error_msg) OVERRIDE;
+  // NotificationObserver overrides.
+  virtual void Observe(int type,

[craigdh, 2012/05/08 21:18:40]

Why are the three preceding functions private when they are public in the parent
classes?

[kkania, 2012/05/09 18:19:59]

The idea is to communicate to the client of this class that these methods should
not be called directly. The method is only called through the appropriate
interface. I don't think there's a hard rule about whether to do this or not,
and browser.h
(http://code.google.com/codesearch#OAMlx_jo-ck/src/chrome/browser/ui/browser.h...)
seems to have some of its overrides public and private. I'd be fine moving it to
public.

+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details);

[craigdh, 2012/05/08 21:18:40]

Add OVERRIDE.

[kkania, 2012/05/09 18:19:59]

Done.

+
+  // Helper method to send a response back to the client. Deletes this.
+  void SendMessage(bool success, const automation::Error& error);

[craigdh, 2012/05/08 21:18:40]

Start private function names with an _.

[craigdh, 2012/05/08 21:21:45]

nvm.

+
+  base::WeakPtr<AutomationProvider> automation_;
+  scoped_ptr<IPC::Message> reply_message_;
+  content::NotificationRegistrar registrar_;
+  bool has_point_;
+  gfx::Point point_;
+
+  DISALLOW_COPY_AND_ASSIGN(MouseEventProcessedObserver);
+};
+
 class NTPInfoObserver : public content::NotificationObserver {
  public:
   NTPInfoObserver(AutomationProvider* automation,
                   IPC::Message* reply_message,
                   CancelableRequestConsumer* consumer);
   virtual ~NTPInfoObserver();
 
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details);
@@ skipping 482 matching lines @@
  private:
   base::WeakPtr<AutomationProvider> automation_;
   scoped_ptr<IPC::Message> reply_message_;
   std::string extension_id_;
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionPopupObserver);
 };
 
 #endif  // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_