diff options
| author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-17 01:19:04 +0000 |
|---|---|---|
| committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-17 01:19:04 +0000 |
| commit | e3a91e7d2cb359692c59b86a29453f1931f87760 (patch) | |
| tree | 83d18e33d0041e888e580dee0e81b15cf3d490fc /chrome_frame/test/data | |
| parent | 584245e578427abc2bc19f224609f820ad71aca0 (diff) | |
| download | chromium_src-e3a91e7d2cb359692c59b86a29453f1931f87760.zip chromium_src-e3a91e7d2cb359692c59b86a29453f1931f87760.tar.gz chromium_src-e3a91e7d2cb359692c59b86a29453f1931f87760.tar.bz2 | |
window.open calls issued by pages within ChromeFrame would not honor the suggested dimensions and would end up
opening a default top level browser window in IE.
ChromeFrame does receive the dimensions from the external tab container when it is notified about a popup being
opened.
Fix is to honor these dimensions by passing them off in the specially crafted url containing other arguments.
Fixes bug http://code.google.com/p/chromium/issues/detail?id=42250
This fix is currently implemented for IE full tab mode only.
Bug=42250
Test=Covered by augmenting the existing window open test to also validate the window size. Added a new unit test
to test the ParseAttachExternalTabUrl helper function.
Review URL: http://codereview.chromium.org/2867007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50064 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/test/data')
| -rw-r--r-- | chrome_frame/test/data/chrome_frame_window_open.html | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome_frame/test/data/chrome_frame_window_open.html b/chrome_frame/test/data/chrome_frame_window_open.html index cf96a9e..22515ed 100644 --- a/chrome_frame/test/data/chrome_frame_window_open.html +++ b/chrome_frame/test/data/chrome_frame_window_open.html @@ -18,8 +18,8 @@ function onLoad() { var new_window; function OpenPopup() { - new_window = window.open("chrome_frame_window_open_popup.html", "mywindow", - "left=10, top=10, height=100, width=100"); + new_window = window.open("chrome_frame_window_open_popup.html", "_blank", + "left=10, top=10, height=250, width=250"); } function OnKeyPress() { |