Update dialog parent handling logic.

Update dialog parent handling logic.

- Use real transient parent for hosted native dialogs since window manager
  supports transient of transient now;
- Use last active normal browser window as parent for option dialogs since
  we don't want to use panel popups as parent;

BUG=chromium-os:3708
TEST=none. This CL gives real transent parent for dialogs but we still need to way to show the dialogs, either exit fullscreen or put them on top of fullscreen.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=48773

[sky, 2010-06-02 17:44:12]

LGTM

http://codereview.chromium.org/2501001/diff/1/2
File chrome/browser/chromeos/options/options_window_view.cc (right):

http://codereview.chromium.org/2501001/diff/1/2#newcode312
chrome/browser/chromeos/options/options_window_view.cc:312: b =
BrowserList::FindBrowserWithType(b->profile(),
FindBrowserWithType searches in order of activation, so you can just use it
instead of first asking for GetLastActive.

[xiyuan, 2010-06-02 17:48:08]

http://codereview.chromium.org/2501001/diff/1/2
File chrome/browser/chromeos/options/options_window_view.cc (right):

http://codereview.chromium.org/2501001/diff/1/2#newcode312
chrome/browser/chromeos/options/options_window_view.cc:312: b =
BrowserList::FindBrowserWithType(b->profile(),
On 2010/06/02 17:44:12, sky wrote:
> FindBrowserWithType searches in order of activation, so you can just use it
> instead of first asking for GetLastActive.
The reason for doing this is FindBrowserWithType expects a profile and I don't
have that. :(

[sky, 2010-06-02 17:52:21]

http://codereview.chromium.org/2501001/diff/1/2
File chrome/browser/chromeos/options/options_window_view.cc (right):

http://codereview.chromium.org/2501001/diff/1/2#newcode312
chrome/browser/chromeos/options/options_window_view.cc:312: b =
BrowserList::FindBrowserWithType(b->profile(),
On 2010/06/02 17:48:08, xiyuan wrote:
> On 2010/06/02 17:44:12, sky wrote:
> > FindBrowserWithType searches in order of activation, so you can just use it
> > instead of first asking for GetLastActive.
> The reason for doing this is FindBrowserWithType expects a profile and I don't
> have that. :(
> 

Ah ok. That makes sense.