Skip initial seek to 0 on fling

media/base/android/media_player_bridge.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_ANDROID_MEDIA_PLAYER_BRIDGE_H_
 #define MEDIA_BASE_ANDROID_MEDIA_PLAYER_BRIDGE_H_
 
 #include <jni.h>
 #include <map>
 #include <string>
@@ skipping 124 matching lines @@
   bool InterceptMediaUrl(
       const std::string& url, int* fd, int64* offset, int64* size);
 
   // Whether the player is prepared for playback.
   bool prepared_;
 
   // Pending play event while player is preparing.
   bool pending_play_;
 
   // Pending seek time while player is preparing.
-  base::TimeDelta pending_seek_;
+  base::TimeDelta pending_seek_ = base::TimeDelta::FromMilliseconds(-1);

[aberent, 2014/10/20 14:55:55]

Better in initializer of constructor. The coding standard is unclear on this,
but other members of this class are initialised in the constructor so for
consistency initialize all the variables here.

Also, the magic number is nasty. Better to add a new boolean member to the
class, and use that to decide whether to do a seek.

[dgn, 2014/10/20 15:43:20]

Done.

 
   // Url for playback.
   GURL url_;
 
   // First party url for cookies.
   GURL first_party_for_cookies_;
 
   // User agent string to be used for media player.
   const std::string user_agent_;
 
@@ skipping 26 matching lines @@
 
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<MediaPlayerBridge> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaPlayerBridge);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_ANDROID_MEDIA_PLAYER_BRIDGE_H_