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

Issue 164199: Merge 21982 - Retry requests that has timed out in BufferedDataSource... (Closed)

Created:
11 years, 4 months ago by laforge
Modified:
9 years, 7 months ago
Reviewers:
hclam
CC:
chromium-reviews_googlegroups.com, fbarchard, Alpha Left Google, kylep, awong, darin (slow to review), scherkus (not reviewing)
Visibility:
Public.

Description

Merge 21982 - Retry requests that has timed out in BufferedDataSource Retries request if the read request is not fulfilled in 5 seconds and at most 3 times. This will handle the case when a user pause a movie and starts it in a later time when the connection to the server has already timed out. TEST=BufferedDataSourceTest.* BUG=17479 done Review URL: http://codereview.chromium.org/159382 TBR=hclam@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22810

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+134 lines, -1 line) Patch
MM webkit/glue/media/buffered_data_source.h View 5 chunks +19 lines, -0 lines 0 comments Download
MM webkit/glue/media/buffered_data_source.cc View 6 chunks +55 lines, -1 line 0 comments Download
MM webkit/glue/media/buffered_data_source_unittest.cc View 5 chunks +60 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
laforge
11 years, 4 months ago (2009-08-07 22:38:46 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698