Add tests for recent popup blocker fixes

Add tests for recent popup blocker fixes

Check that the renderer can't launch a denial of service attack on the browser

Check that the content settings apply to the main frame URL

BUG=289528, 289525
R=bauerb@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223300

[jochen (gone - plz use gerrit), 2013-09-12 13:30:39]

plz review

[Bernhard Bauer, 2013-09-12 13:39:22]

LGTM w/ some nits:

https://codereview.chromium.org/23600041/diff/1/chrome/browser/ui/blocked_con...
File chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc (right):

https://codereview.chromium.org/23600041/diff/1/chrome/browser/ui/blocked_con...
chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc:267: GURL
frame_url(ui_test_utils::GetTestUrl(
Nit: You could move this declaration down to where it's first used.

https://codereview.chromium.org/23600041/diff/1/chrome/browser/ui/blocked_con...
chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc:270:
browser()->profile()->GetHostContentSettingsMap()
Nit: Extract the HCSM into a local variable?

https://codereview.chromium.org/23600041/diff/1/chrome/browser/ui/blocked_con...
chrome/browser/ui/blocked_content/popup_blocker_browsertest.cc:406: // popus.
Nit: "popups"

https://codereview.chromium.org/23600041/diff/1/chrome/test/data/popup_blocke...
File chrome/test/data/popup_blocker/popup-dos.html (right):

https://codereview.chromium.org/23600041/diff/1/chrome/test/data/popup_blocke...
chrome/test/data/popup_blocker/popup-dos.html:7: for (var i = 0; i<1000; ++i)
Nit: space before and after less-than sign.

[commit-bot: I haz the power, 2013-09-15 20:26:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/23600041/10001

[commit-bot: I haz the power, 2013-09-16 02:45:40]

Change committed as 223300