There are still some conditions that new page will be opened in a new midbrowse window but not a new tab

[Zhao Fan]

Bug detailed description:
--------------------------
We know that midbrowser has been configured as tabbed browsing, but in some conditions, some html code will open a new window anyway to show the webpage. The problem brought by this is that user may never be able to bring out the windows they opened. I think we have two choices to fix the problem:
(1) If we find that the link is trying to open a new midbrowser window, we capture the event and open a new tab instead to show the content. In this way, we will solve the problem radically, but I am not sure how much efforts needed to achieve this.
(2) We need to bring out all the windows opened every time user clicks the midbrowser icon. This is also acceptable because this ensures that user could activate all windows they have opened at least. Carl, I think this is also what you want to do to solve the preference/download/addon dialog problem.

Carl, how do you think about this?

BTW, I tried following the Reproduce steps on both MID and my laptop. on my laptop, a new tab will be openend, but on MID, a new window will pop up.

Reproduce steps:
----------------
(1) Configure the proxy as proxy01.pd.intel.com:911
(2) Visit http://list.mp3.baidu.com/topso/mp3topsong.html?id=1?top2
(3) Click a song randomly such as "日不落“
(4) Click a link from the search list

Current result:
----------------
A new window will pop up

Expected result:
----------------
a new tab instead of a new window should be opened

Basic root-causing log:
----------------------

[cwong1 (carl-wong)]

I have moved this bug to M11 becuase the fix that I had in mind will require changes to the browser's xul core file. Since we are switching over to use the system xul runner, we need to find a different way to fix this issue. I have talked to Mauri and she has agreed.

[Mauri Whalen (mauri-whalen)]

Is this fixed with the change to xul core?

[cwong1 (carl-wong)]

This is not fixed with the change to xul core. It's been there for a while.