Fix an issue that video is not seekable

Fix an issue that video is not seekable

When updating the ReadyState, HTMLMediaElement will immediately check the duration.
If at that time has_valid_metadata_ is false, we will return NaN.
So the has_valid_metadata_ has to be set before ReadyState is updated.

BUG=414499
Committed: https://crrev.com/21f1f757864d7ec360ee1c7a00a6cb31d7af1231
Cr-Commit-Position: refs/heads/master@{#294979}

[qinmin, 2014-09-15 22:54:51]

qinmin@chromium.org changed reviewers:
+ xhwang@chromium.org

[qinmin, 2014-09-15 22:54:59]

PTAL

[xhwang, 2014-09-15 23:59:43]

lgtm % nit

https://codereview.chromium.org/572803002/diff/1/content/renderer/media/andro...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/572803002/diff/1/content/renderer/media/andro...
content/renderer/media/android/webmediaplayer_android.cc:738:
has_valid_metadata_ = success;
Add a comment that setting it here instead of later is critical?

[qinmin, 2014-09-16 00:30:52]

https://codereview.chromium.org/572803002/diff/1/content/renderer/media/andro...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/572803002/diff/1/content/renderer/media/andro...
content/renderer/media/android/webmediaplayer_android.cc:738:
has_valid_metadata_ = success;
On 2014/09/15 23:59:43, xhwang wrote:
> Add a comment that setting it here instead of later is critical?

Done.

[qinmin, 2014-09-16 00:30:55]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-09-16 00:37:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/572803002/20001

[commit-bot: I haz the power, 2014-09-16 02:17:13]

Committed patchset #2 (id:20001) as 15a99e5add86d458fb93b47f2a2e71a98052a439

[commit-bot: I haz the power, 2014-09-16 02:21:04]

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