Turn on better popup blocker by default.

chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc

Index: chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc
diff --git a/chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc b/chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc
index 2c1218937215d4dde508d32a845d6050e80430ba..3bf144d56ffbae292327d912db43af9e1f8d287f 100644
--- a/chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc
+++ b/chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc
@@ -74,6 +74,11 @@ class PopupBlockerBrowserTest : public InProcessBrowserTest {
  public:
   PopupBlockerBrowserTest() {}
 
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    InProcessBrowserTest::SetUpCommandLine(command_line);
+    command_line->AppendSwitch(switches::kDisableBetterPopupBlocking);
+  }
+
   // Returns a url that shows one popup.
   GURL GetTestURL() {
     return ui_test_utils::GetTestUrl(
@@ -219,8 +224,7 @@ class BetterPopupBlockerBrowserTest : public PopupBlockerBrowserTest {
   virtual ~BetterPopupBlockerBrowserTest() {}
 
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {

[marja, 2013/07/25 13:42:51]

This overridden func is now unnecessary.

[marja, 2013/07/25 13:47:11]

I was fooled by the code! This is for bypassing the immediate parent function!
How sneaky!

-    PopupBlockerBrowserTest::SetUpCommandLine(command_line);
-    command_line->AppendSwitch(switches::kEnableBetterPopupBlocking);
+    InProcessBrowserTest::SetUpCommandLine(command_line);
   }
 
  private: