Initialize the BrowserContainerView with the right frame.

Initialize the BrowserContainerView with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size, resulting in
the screenshot used in the animation being incorrect.

BUG=671724
Committed: https://crrev.com/50d5ba259ac6c76e63884412fb1a214858b81654
Cr-Commit-Position: refs/heads/master@{#439792}

[jif, 2016-12-20 12:10:27]

Description was changed from

==========
Initialize the BrowserContainerView's with the right frame.

BUG=671724
==========

to

==========
Initialize the BrowserContainerView's with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito BVC is created,
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in the screenshot used in the animation being incorrect.

BUG=671724
==========

[jif, 2016-12-20 12:10:27]

jif@chromium.org changed reviewers:
+ marq@chromium.org

[jif, 2016-12-20 12:21:13]

Description was changed from

==========
Initialize the BrowserContainerView's with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito BVC is created,
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in the screenshot used in the animation being incorrect.

BUG=671724
==========

to

==========
Initialize the BrowserContainerView's with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724
==========

[jif, 2016-12-20 12:22:18]

ptal

[marq (ping after 24h), 2016-12-20 12:36:36]

LGTM with nit.

https://codereview.chromium.org/2591683002/diff/1/ios/chrome/browser/ui/brows...
File ios/chrome/browser/ui/browser_view_controller.mm (right):

https://codereview.chromium.org/2591683002/diff/1/ios/chrome/browser/ui/brows...
ios/chrome/browser/ui/browser_view_controller.mm:1172: // Duplicate .xib layout
for now.
I think this comment can be removed now.

[jif, 2016-12-20 13:07:15]

Description was changed from

==========
Initialize the BrowserContainerView's with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724
==========

to

==========
Initialize the BrowserContainerView with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724
==========

[jif, 2016-12-20 13:13:03]

The CQ bit was checked by jif@chromium.org

[jif, 2016-12-20 13:13:04]

The patchset sent to the CQ was uploaded after l-g-t-m from marq@chromium.org
Link to the patchset: https://codereview.chromium.org/2591683002/#ps20001
(title: "Addressed comment.")

[commit-bot: I haz the power, 2016-12-20 13:13:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-20 14:00:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482239583616510,
"parent_rev": "3232b5e6fb10e881c1c5accd5b21d566b688f53a", "commit_rev":
"02ed55187f01c64d66408b25701a2c0af6b80776"}

[commit-bot: I haz the power, 2016-12-20 14:01:02]

Description was changed from

==========
Initialize the BrowserContainerView with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724
==========

to

==========
Initialize the BrowserContainerView with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724

Review-Url: https://codereview.chromium.org/2591683002
==========

[commit-bot: I haz the power, 2016-12-20 14:01:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-20 14:02:52]

Description was changed from

==========
Initialize the BrowserContainerView with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724

Review-Url: https://codereview.chromium.org/2591683002
==========

to

==========
Initialize the BrowserContainerView with the right frame.

When the iPad Tab Switcher creates an incognito tab for the first time, an
incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its
size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size,
resulting in
the screenshot used in the animation being incorrect.

BUG=671724

Committed: https://crrev.com/50d5ba259ac6c76e63884412fb1a214858b81654
Cr-Commit-Position: refs/heads/master@{#439792}
==========

[commit-bot: I haz the power, 2016-12-20 14:02:53]

Patchset 2 (id:??) landed as
https://crrev.com/50d5ba259ac6c76e63884412fb1a214858b81654
Cr-Commit-Position: refs/heads/master@{#439792}