Chromium Code Reviews| Index: chrome/browser/ui/views/frame/browser_view.cc |
| diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc |
| index e34983dae09a4faf5753142b2ef8d19836919241..b8f11f0b88ed47dd6c9083c94ba06becf3b130f0 100644 |
| --- a/chrome/browser/ui/views/frame/browser_view.cc |
| +++ b/chrome/browser/ui/views/frame/browser_view.cc |
| @@ -606,7 +606,15 @@ void BrowserView::Show() { |
| // If the window is already visible, just activate it. |
| if (frame_->IsVisible()) { |
| +#if !defined(OS_MACOSX) |
|
Peter Kasting
2016/12/22 00:38:42
Nit: Remove "!" and reverse arms, so "else" doesn'
|
| frame_->Activate(); |
| +#else |
| + // When opening a URL from other application, macOS will activate browser |
|
Peter Kasting
2016/12/22 00:38:42
Nit: other -> another, browser -> the browser
|
| + // window in case when "Open" action is used, but won't do it in case when |
|
Peter Kasting
2016/12/22 00:38:42
Nit: in case -> in the case (2x)
|
| + // "Open Behind" is requested. Let's assume macOS handles the activation |
| + // for us, just put the window above other browser windows. |
|
Peter Kasting
2016/12/22 00:38:42
Nit: , -> ;
|
| + frame_->ShowInactive(); |
| +#endif |
| return; |
| } |