Use the top-level URL for popup blocking.

chrome/browser/chrome_content_browser_client.cc

Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index bcc19670fbbb302b04432cd99c00cdc717a7d867..14d9731e907741efe5259c250446ddee7bfe9fdc 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -1955,6 +1955,7 @@ void ChromeContentBrowserClient::CancelDesktopNotification(
 
 bool ChromeContentBrowserClient::CanCreateWindow(
     const GURL& opener_url,
+    const GURL& opener_top_level_frame_url,
     const GURL& source_origin,
     WindowContainerType container_type,
     const GURL& target_url,
@@ -2021,8 +2022,8 @@ bool ChromeContentBrowserClient::CanCreateWindow(
 
   if (!user_gesture && !CommandLine::ForCurrentProcess()->HasSwitch(
         switches::kDisablePopupBlocking)) {
-    if (content_settings->GetContentSetting(opener_url,
-                                            opener_url,
+    if (content_settings->GetContentSetting(opener_top_level_frame_url,
+                                            opener_top_level_frame_url,

[Bernhard Bauer, 2013/09/11 16:37:46]

Should we pass the |opener_url| as one of the parameters here?

[jochen (gone - plz use gerrit), 2013/09/11 16:45:25]

We can consider that in a future cl. For now, I want to restore the original
behavior.

                                             CONTENT_SETTINGS_TYPE_POPUPS,
                                             std::string()) ==
         CONTENT_SETTING_ALLOW) {