Fix WMPI::currentTime/seek race causing flaky Blink mediasource-redundant-seek.html on desktop

media/blink/webmediaplayer_impl.h

 // Copyright 2013 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_BLINK_WEBMEDIAPLAYER_IMPL_H_
 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 268 matching lines @@
   double playback_rate_;
   base::TimeDelta paused_time_;
 
   // TODO(scherkus): Replace with an explicit ended signal to HTMLMediaElement,
   // see http://crbug.com/409280
   bool ended_;
 
   // Seek gets pending if another seek is in progress. Only last pending seek
   // will have effect.
   bool pending_seek_;
-  double pending_seek_seconds_;
+  base::TimeDelta pending_seek_time_;
+  base::TimeDelta seek_time_;

[xhwang, 2015/06/09 21:33:57]

Move |seek_time_| to after |seeking_|?

[wolenetz, 2015/06/09 22:09:58]

Done, with comments added, and |playback_rate_| moved closer to its comment.

 
   // Tracks whether to issue time changed notifications during buffering state
   // changes.
   bool should_notify_time_changed_;
 
   blink::WebMediaPlayerClient* client_;
 
   base::WeakPtr<WebMediaPlayerDelegate> delegate_;
 
   WebMediaPlayerParams::DeferLoadCB defer_load_cb_;
@@ skipping 28 matching lines @@
   EncryptedMediaPlayerSupport encrypted_media_support_;
 
   scoped_ptr<RendererFactory> renderer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_