Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(108)

Side by Side Diff: webkit/glue/media/buffered_data_source.h

Issue 5603004: Use stopped_on_render_loop_ to prevent further work from executing in BufferedDataSource. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | webkit/glue/media/buffered_data_source.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef WEBKIT_GLUE_MEDIA_BUFFERED_DATA_SOURCE_H_ 5 #ifndef WEBKIT_GLUE_MEDIA_BUFFERED_DATA_SOURCE_H_
6 #define WEBKIT_GLUE_MEDIA_BUFFERED_DATA_SOURCE_H_ 6 #define WEBKIT_GLUE_MEDIA_BUFFERED_DATA_SOURCE_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 399 matching lines...) Expand 10 before | Expand all | Expand 10 after
410 MessageLoop* render_loop_; 410 MessageLoop* render_loop_;
411 411
412 // Protects |stopped_|. 412 // Protects |stopped_|.
413 Lock lock_; 413 Lock lock_;
414 414
415 // Stop signal to suppressing activities. This variable is set on the pipeline 415 // Stop signal to suppressing activities. This variable is set on the pipeline
416 // thread and read from the render thread. 416 // thread and read from the render thread.
417 bool stop_signal_received_; 417 bool stop_signal_received_;
418 418
419 // This variable is set by CleanupTask() that indicates this object is stopped 419 // This variable is set by CleanupTask() that indicates this object is stopped
420 // on the render thread. 420 // on the render thread and work should no longer progress.
421 bool stopped_on_render_loop_; 421 bool stopped_on_render_loop_;
422 422
423 // This variable is true when we are in a paused state and false when we 423 // This variable is true when we are in a paused state and false when we
424 // are in a playing state. 424 // are in a playing state.
425 bool media_is_paused_; 425 bool media_is_paused_;
426 426
427 // This timer is to run the WatchDogTask repeatedly. We use a timer instead 427 // This timer is to run the WatchDogTask repeatedly. We use a timer instead
428 // of doing PostDelayedTask() reduce the extra reference held by the message 428 // of doing PostDelayedTask() reduce the extra reference held by the message
429 // loop. The RepeatingTimer does PostDelayedTask() internally, by using it 429 // loop. The RepeatingTimer does PostDelayedTask() internally, by using it
430 // the message loop doesn't hold a reference for the watch dog task. 430 // the message loop doesn't hold a reference for the watch dog task.
431 base::RepeatingTimer<BufferedDataSource> watch_dog_timer_; 431 base::RepeatingTimer<BufferedDataSource> watch_dog_timer_;
432 432
433 // Keeps track of whether we used a Range header in the initialization 433 // Keeps track of whether we used a Range header in the initialization
434 // request. 434 // request.
435 bool using_range_request_; 435 bool using_range_request_;
436 436
437 DISALLOW_COPY_AND_ASSIGN(BufferedDataSource); 437 DISALLOW_COPY_AND_ASSIGN(BufferedDataSource);
438 }; 438 };
439 439
440 } // namespace webkit_glue 440 } // namespace webkit_glue
441 441
442 #endif // WEBKIT_GLUE_MEDIA_BUFFERED_DATA_SOURCE_H_ 442 #endif // WEBKIT_GLUE_MEDIA_BUFFERED_DATA_SOURCE_H_
OLDNEW
« no previous file with comments | « no previous file | webkit/glue/media/buffered_data_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698