Also fire onBeforeRetarget events if now tabs/windows are opened due to user actions.

BUG=none
TEST=api tests

Review URL: http://codereview.chromium.org/6627040

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

3 files changed, 17 insertions, 1 deletions

diff --git a/content/browser/tab_contents/tab_contents.cc b/content/browser/tab_contents/tab_contents.cc
index 71a7644..5ca1e0f 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -646,8 +646,12 @@ bool TabContents::NeedToFireBeforeUnload() {
 void TabContents::OpenURL(const GURL& url, const GURL& referrer,
                           WindowOpenDisposition disposition,
                           PageTransition::Type transition) {
-  if (delegate_)
+  if (delegate_) {
     delegate_->OpenURLFromTab(this, url, referrer, disposition, transition);
+    // Notify observers.
+    FOR_EACH_OBSERVER(TabContentsObserver, observers_,
+                      DidOpenURL(url, referrer, disposition, transition));
+  }
 }
 
 bool TabContents::NavigateToPendingEntry(
diff --git a/content/browser/tab_contents/tab_contents_observer.cc b/content/browser/tab_contents/tab_contents_observer.cc
index d045792..50bf910 100644
--- a/content/browser/tab_contents/tab_contents_observer.cc
+++ b/content/browser/tab_contents/tab_contents_observer.cc
@@ -79,6 +79,12 @@ void TabContentsObserver::RenderViewGone() {
 void TabContentsObserver::StopNavigation() {
 }
 
+void TabContentsObserver::DidOpenURL(const GURL& url,
+                                     const GURL& referrer,
+                                     WindowOpenDisposition disposition,
+                                     PageTransition::Type transition) {
+}
+
 TabContentsObserver::TabContentsObserver(TabContents* tab_contents) {
   SetTabContents(tab_contents);
   tab_contents_->AddObserver(this);
diff --git a/content/browser/tab_contents/tab_contents_observer.h b/content/browser/tab_contents/tab_contents_observer.h
index 87694d8..bfe7760 100644
--- a/content/browser/tab_contents/tab_contents_observer.h
+++ b/content/browser/tab_contents/tab_contents_observer.h
@@ -8,6 +8,7 @@
 #include "content/browser/tab_contents/navigation_controller.h"
 #include "content/common/page_transition_types.h"
 #include "ipc/ipc_channel.h"
+#include "webkit/glue/window_open_disposition.h"
 
 struct ViewHostMsg_FrameNavigate_Params;
 
@@ -66,6 +67,11 @@ class TabContentsObserver : public IPC::Channel::Listener,
   virtual void RenderViewGone();
   virtual void StopNavigation();
 
+  virtual void DidOpenURL(const GURL& url,
+                          const GURL& referrer,
+                          WindowOpenDisposition disposition,
+                          PageTransition::Type transition);
+
 #if 0
   // For unifying with delegate...