913387 - [Buri][Monitor][Video]The video will back to start when invoking recent app

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Video, defect, P1)

Description

[sync-1]

Firefox os  v1.1
  Mozilla build ID:20130902041201
 
 Created an attachment (id=510965)
 pic
 
 DEFECT DESCRIPTION:
 
 
  REPRODUCING PROCEDURES:
 precondition:launch music,camera and any other application
 1、launch video player->tap on any video in the tile view and play it
 2、long press home key,invoking recent app->launch other app such as music,camera->return to video app
 3、The video will back to start  ->KO
  
  EXPECTED BEHAVIOUR:
 The video should keeps in the position where you've played
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 moderate
 
  REPRODUCING RATE:
 2/5
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: pic

Comment 3

[sync-1]

Clone from brother

Comment 4

[sync-1]

Attachment: logcat

Comment 5

[John Hu [:johnhu][:johu][:醬糊小弟]]

The STR can be simplified as the following:

 1、launch video player->tap on any video in the tile view and play it
 2、short press home key
 3、launch video player, and short press home key immediately
 4、launch video player => KO

After some investigation, there is a case that we press home key before the metadata is loaded. That means we haven't restored the playing time[1] and the video app is trying to store the currentTime of player[2]. 

This bug is also happened to master. 

[1] https://github.com/mozilla-b2g/gaia/blob/ab52c3d9c7e57b9c3fe8c82e79786be85f320277/apps/video/js/video.js#L994
[2] https://github.com/mozilla-b2g/gaia/blob/ab52c3d9c7e57b9c3fe8c82e79786be85f320277/apps/video/js/video.js#L985

Comment 6

[John Hu [:johnhu][:johu][:醬糊小弟]]

It is caused by that video app is trying to store the information when video element doesn't load metadata. It is harder to reproduce this bug with high-end device.

Comment 7

[John Hu [:johnhu][:johu][:醬糊小弟]]

Attachment: patch of this bug.

add small piece of code to check if video element has metadata which means the currentTime is significant.

Comment 8

[David Flanagan [:djf]]

Comment on attachment 801400 [details]
patch of this bug.

A minor nit on github, but r+ to land this.

Comment 9

[John Hu [:johnhu][:johu][:醬糊小弟]]

merged to master:
https://github.com/mozilla-b2g/gaia/commit/73d7f74142f204241505bd6dfbb84d0ed83c323a