Revert of Delete VideoFrame default constructor, and IsZeroSize method.

Revert of Delete VideoFrame default constructor, and IsZeroSize method. (patchset #5 id:80001 of https://codereview.webrtc.org/2541863002/ )

Reason for revert:
Crashes perf tests, e.g.,

./out/Debug/webrtc_perf_tests --gtest_filter='FullStackTest.ScreenshareSlidesVP8_2TL_VeryLossyNet'

dies with an assert related to rtc::Optional.

Original issue's description:
> Delete VideoFrame default constructor, and IsZeroSize method.
>
> This ensures that the video_frame_buffer method never can return a
> null pointer.
>
> BUG=webrtc:6591
>
> Committed: https://crrev.com/bfcf561923a42005e4c7d66d8e72e5932155f997
> Cr-Commit-Position: refs/heads/master@{#15574}

TBR=magjed@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6591
Committed: https://crrev.com/0989fbcad2ca4eb5805a77e8ebfefd3af06ade23
Cr-Commit-Position: refs/heads/master@{#15597}

[nisse-webrtc, 2016-12-14 10:06:35]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2016-12-14 10:06:35]

Created Revert of Delete VideoFrame default constructor, and IsZeroSize method.

[commit-bot: I haz the power, 2016-12-14 10:06:38]

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

[commit-bot: I haz the power, 2016-12-14 10:06:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481709995000570, "parent_rev":
"89a517032a06df6be85be1fd1ef51453b8391555", "commit_rev":
"14a3e8053805123397aebf9a5948186a41f28013"}

[commit-bot: I haz the power, 2016-12-14 10:06:47]

Description was changed from

==========
Revert of Delete VideoFrame default constructor, and IsZeroSize method.
(patchset #5 id:80001 of https://codereview.webrtc.org/2541863002/ )

Reason for revert:
Crashes perf tests, e.g., 

./out/Debug/webrtc_perf_tests
--gtest_filter='FullStackTest.ScreenshareSlidesVP8_2TL_VeryLossyNet'

dies with an assert related to rtc::Optional.

Original issue's description:
> Delete VideoFrame default constructor, and IsZeroSize method.
> 
> This ensures that the video_frame_buffer method never can return a
> null pointer.
> 
> BUG=webrtc:6591
> 
> Committed: https://crrev.com/bfcf561923a42005e4c7d66d8e72e5932155f997
> Cr-Commit-Position: refs/heads/master@{#15574}

TBR=magjed@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6591
==========

to

==========
Revert of Delete VideoFrame default constructor, and IsZeroSize method.
(patchset #5 id:80001 of https://codereview.webrtc.org/2541863002/ )

Reason for revert:
Crashes perf tests, e.g., 

./out/Debug/webrtc_perf_tests
--gtest_filter='FullStackTest.ScreenshareSlidesVP8_2TL_VeryLossyNet'

dies with an assert related to rtc::Optional.

Original issue's description:
> Delete VideoFrame default constructor, and IsZeroSize method.
> 
> This ensures that the video_frame_buffer method never can return a
> null pointer.
> 
> BUG=webrtc:6591
> 
> Committed: https://crrev.com/bfcf561923a42005e4c7d66d8e72e5932155f997
> Cr-Commit-Position: refs/heads/master@{#15574}

TBR=magjed@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6591

Review-Url: https://codereview.webrtc.org/2574123002
==========

[commit-bot: I haz the power, 2016-12-14 10:06:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-14 10:06:54]

Description was changed from

==========
Revert of Delete VideoFrame default constructor, and IsZeroSize method.
(patchset #5 id:80001 of https://codereview.webrtc.org/2541863002/ )

Reason for revert:
Crashes perf tests, e.g., 

./out/Debug/webrtc_perf_tests
--gtest_filter='FullStackTest.ScreenshareSlidesVP8_2TL_VeryLossyNet'

dies with an assert related to rtc::Optional.

Original issue's description:
> Delete VideoFrame default constructor, and IsZeroSize method.
> 
> This ensures that the video_frame_buffer method never can return a
> null pointer.
> 
> BUG=webrtc:6591
> 
> Committed: https://crrev.com/bfcf561923a42005e4c7d66d8e72e5932155f997
> Cr-Commit-Position: refs/heads/master@{#15574}

TBR=magjed@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6591

Review-Url: https://codereview.webrtc.org/2574123002
==========

to

==========
Revert of Delete VideoFrame default constructor, and IsZeroSize method.
(patchset #5 id:80001 of https://codereview.webrtc.org/2541863002/ )

Reason for revert:
Crashes perf tests, e.g.,

./out/Debug/webrtc_perf_tests
--gtest_filter='FullStackTest.ScreenshareSlidesVP8_2TL_VeryLossyNet'

dies with an assert related to rtc::Optional.

Original issue's description:
> Delete VideoFrame default constructor, and IsZeroSize method.
>
> This ensures that the video_frame_buffer method never can return a
> null pointer.
>
> BUG=webrtc:6591
>
> Committed: https://crrev.com/bfcf561923a42005e4c7d66d8e72e5932155f997
> Cr-Commit-Position: refs/heads/master@{#15574}

TBR=magjed@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6591

Committed: https://crrev.com/0989fbcad2ca4eb5805a77e8ebfefd3af06ade23
Cr-Commit-Position: refs/heads/master@{#15597}
==========

[commit-bot: I haz the power, 2016-12-14 10:06:55]

Patchset 1 (id:??) landed as
https://crrev.com/0989fbcad2ca4eb5805a77e8ebfefd3af06ade23
Cr-Commit-Position: refs/heads/master@{#15597}

[nisse-webrtc, 2016-12-14 13:16:45]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2574183002/ by nisse@webrtc.org.

The reason for reverting is: Fixing perf tests..