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

Issue 6926002: Merge 83942 - Partial revert of 82061 so we keep the initial unbounded range request. (Closed)

Created:
9 years, 7 months ago by scherkus (not reviewing)
Modified:
9 years, 7 months ago
CC:
chromium-reviews, hclam+watch_chromium.org, sjl, ddorwin+watch_chromium.org, fischman+watch_chromium.org, Paweł Hajdan Jr., acolwell GONE FROM CHROMIUM, annacc, darin-cc_chromium.org, ajwong+watch_chromium.org, vrk (LEFT CHROMIUM), scherkus (not reviewing)
Visibility:
Public.

Description

Merge 83942 - Partial revert of 82061 so we keep the initial unbounded range request. Also loosened restrictions for detecting servers that support range requests. Sending "Accept-Ranges: bytes" is optional so if we attempt a range request and a server replies correctly we should assume that it does indeed support range requests. BUG=80187 TEST=test_shell_tests --gtest_filter=Buffered* Review URL: http://codereview.chromium.org/6889019 TBR=scherkus@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=84001

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -13 lines) Patch
M webkit/glue/media/buffered_resource_loader.h View 1 chunk +2 lines, -3 lines 0 comments Download
M webkit/glue/media/buffered_resource_loader.cc View 3 chunks +8 lines, -7 lines 0 comments Download
M webkit/glue/media/buffered_resource_loader_unittest.cc View 2 chunks +50 lines, -3 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
scherkus (not reviewing)
9 years, 7 months ago (2011-05-03 23:22:13 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698