mac: Opening a URL should replace the NTP.

mac: Opening a URL should replace the NTP.

When a browser window is instructed to open a URL, if the only existing tab is
the NTP, the NTP should be replaced with a tab with the given URL.

This was partially fixed by https://codereview.chromium.org/240273002, but that
only tackled the case where Chrome was launched as a result of an open URL
Apple Event. This CL replicates the behavior for Chrome receiving a URL from
Handoff, and for Chrome receiving an open URL Apple Event while Chrome is
already running.

BUG=428081, 43520
Committed: https://crrev.com/19ee3922f7438d37833ae5ec58af10518b4a6c47
Cr-Commit-Position: refs/heads/master@{#302304}

[erikchen, 2014-10-28 22:36:44]

Patchset #1 (id:1) has been deleted

[erikchen, 2014-10-28 22:38:35]

erikchen@chromium.org changed reviewers:
+ avi@chromium.org

[erikchen, 2014-10-28 22:38:35]

avi: Please review.

[Avi (use Gerrit), 2014-10-29 14:38:59]

lgtm

https://codereview.chromium.org/678423003/diff/20001/chrome/browser/app_contr...
File chrome/browser/app_controller_mac.mm (right):

https://codereview.chromium.org/678423003/diff/20001/chrome/browser/app_contr...
chrome/browser/app_controller_mac.mm:235: // Opens a tab for each GURL in
|urls|. If there exactly 1 tab open before this
"If there is exactly one tab"...

[erikchen, 2014-10-29 17:18:02]

https://codereview.chromium.org/678423003/diff/20001/chrome/browser/app_contr...
File chrome/browser/app_controller_mac.mm (right):

https://codereview.chromium.org/678423003/diff/20001/chrome/browser/app_contr...
chrome/browser/app_controller_mac.mm:235: // Opens a tab for each GURL in
|urls|. If there exactly 1 tab open before this
On 2014/10/29 14:38:59, Avi wrote:
> "If there is exactly one tab"...

Done.

[erikchen, 2014-10-29 17:18:33]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2014-10-29 17:19:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/678423003/40001

[commit-bot: I haz the power, 2014-10-29 18:14:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-29 18:14:55]

Try jobs failed on following builders:
  mac_chromium_rel on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel/...)

[erikchen, 2014-10-29 18:19:20]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2014-10-29 18:21:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/678423003/40001

[commit-bot: I haz the power, 2014-10-29 19:36:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-29 19:36:19]

Try jobs failed on following builders:
  mac_chromium_rel on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel/...)

[erikchen, 2014-10-29 19:55:32]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2014-10-29 19:56:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/678423003/60001

[commit-bot: I haz the power, 2014-10-29 22:02:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-29 22:02:56]

Try jobs failed on following builders:
  mac_chromium_rel on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel/...)

[erikchen, 2014-10-31 18:28:24]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2014-10-31 18:29:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/678423003/80001

[commit-bot: I haz the power, 2014-10-31 19:14:29]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2014-10-31 19:15:05]

Patchset 4 (id:??) landed as
https://crrev.com/19ee3922f7438d37833ae5ec58af10518b4a6c47
Cr-Commit-Position: refs/heads/master@{#302304}