[CRD iOS] Some fixes for rendering

[CRD iOS] Some fixes for rendering

This CL has these fixes:
1. Removed the glViewport call since it doesn't account for the screen DPI in
    iOS client and it isn't required by either the Android or the iOS client. See
    explanation in comments.
2. Do not reset the viewport if the new screen/desktop size have not changed,
   since viewDidLayoutSubviews: may be called while the frame size is not
   changed.
3. Fix the issue that the first rendered frame may not be shown on the screen
   and explain with comments.
4. Releases the GlDisplayHandlerCore when disconnected.

Review-Url: https://codereview.chromium.org/2874143002
Cr-Commit-Position: refs/heads/master@{#471036}
Committed: https://chromium.googlesource.com/chromium/src/+/8e887d183b8176e14c6800b47fc97ab67fc1957a

[Yuwei, 2017-05-11 06:05:32]

Description was changed from

==========
[CRD iOS] Some fixes for rendering

This CL has these fixes:
1. Do not reset the viewport if the new screen/desktop size have not changed,
   since viewDidLayoutSubviews: may be called while the frame size is not
   changed.
2. Fix the issue that the first rendered frame may not be shown on the screen
   and explain with comments.
3. Releases the GlDisplayHandlerCore when disconnected.
==========

to

==========
[CRD iOS] Some fixes for rendering

This CL has these fixes:
1. Removed the glViewport call since it doesn't account for the screen DPI in
    iOS client and it isn't required by either the Android or the iOS client.
See
    explanation in comments.
2. Do not reset the viewport if the new screen/desktop size have not changed,
   since viewDidLayoutSubviews: may be called while the frame size is not
   changed.
3. Fix the issue that the first rendered frame may not be shown on the screen
   and explain with comments.
4. Releases the GlDisplayHandlerCore when disconnected.
==========

[Yuwei, 2017-05-11 06:11:59]

yuweih@chromium.org changed reviewers:
+ nicholss@chromium.org

[Yuwei, 2017-05-11 06:11:59]

PTAL thanks!

[nicholss, 2017-05-11 15:19:50]

LGTM

I am not quite ready to send out my CL yet:
https://codereview.chromium.org/2869303003/ But in it I change the size of the
host view and I am getting some pretty wacky graphics artifacts. 

If you feel like it, perhaps you can pull that down and see how these changes
play with this CL.

[Yuwei, 2017-05-11 18:51:03]

On 2017/05/11 15:19:50, nicholss wrote:
> LGTM
> 
> I am not quite ready to send out my CL yet:
> https://codereview.chromium.org/2869303003/ But in it I change the size of the
> host view and I am getting some pretty wacky graphics artifacts. 
> 
> If you feel like it, perhaps you can pull that down and see how these changes
> play with this CL.

Thanks! I'll leave comments in your CL

[Yuwei, 2017-05-11 18:51:08]

The CQ bit was checked by yuweih@chromium.org

[commit-bot: I haz the power, 2017-05-11 18:52:27]

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, 2017-05-11 19:16:02]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494528668769860,
"parent_rev": "8c1f6622f0279ee6062496672a7b83317b579486", "commit_rev":
"8e887d183b8176e14c6800b47fc97ab67fc1957a"}

[commit-bot: I haz the power, 2017-05-11 19:16:08]

Description was changed from

==========
[CRD iOS] Some fixes for rendering

This CL has these fixes:
1. Removed the glViewport call since it doesn't account for the screen DPI in
    iOS client and it isn't required by either the Android or the iOS client.
See
    explanation in comments.
2. Do not reset the viewport if the new screen/desktop size have not changed,
   since viewDidLayoutSubviews: may be called while the frame size is not
   changed.
3. Fix the issue that the first rendered frame may not be shown on the screen
   and explain with comments.
4. Releases the GlDisplayHandlerCore when disconnected.
==========

to

==========
[CRD iOS] Some fixes for rendering

This CL has these fixes:
1. Removed the glViewport call since it doesn't account for the screen DPI in
    iOS client and it isn't required by either the Android or the iOS client.
See
    explanation in comments.
2. Do not reset the viewport if the new screen/desktop size have not changed,
   since viewDidLayoutSubviews: may be called while the frame size is not
   changed.
3. Fix the issue that the first rendered frame may not be shown on the screen
   and explain with comments.
4. Releases the GlDisplayHandlerCore when disconnected.

Review-Url: https://codereview.chromium.org/2874143002
Cr-Commit-Position: refs/heads/master@{#471036}
Committed:
https://chromium.googlesource.com/chromium/src/+/8e887d183b8176e14c6800b47fc9...
==========

[commit-bot: I haz the power, 2017-05-11 19:16:09]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8e887d183b8176e14c6800b47fc9...