| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This test originally tested a CA-specific workaround in chain building.
However, due to more OS X "quirks", the CA-specific workaround was
replaced with a CA-agnostic one. While this unit test did provide some
value for the specific CA test, it was only relevant on pre-10.9 systems
anyways; with OS X <10.9 being deprecated, this test is no longer
relevant for those platforms.
Given that the CertVerifyProcWeakDigestTest covers the superset of the
GTE issue, simply removing this test is sufficient.
BUG=590868
TBR=eroman@chromium.org
Review URL: https://codereview.chromium.org/1751733002
Cr-Commit-Position: refs/heads/master@{#378462}
(cherry picked from commit 9ad3f48a16790967f9fc311d83198cc04404b567)
Review URL: https://codereview.chromium.org/1763853002 .
Cr-Commit-Position: refs/branch-heads/2623@{#571}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
| |
A refactor in m50 made noticing this problem tricky. Now fixed.
TBR=mmenke@chromium.org
BUG=590795
Review URL: https://codereview.chromium.org/1746023003 .
Cr-Commit-Position: refs/branch-heads/2623@{#537}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(Manually merged the fix because of significant refactoring around this
area in the m50 timeframe.)
Writing to a sparse entry could result that entry being evicted while
the write was still in progress. This fix considers sparse entries in
use if any of their children are in use; this can drive up memory use
but avoids tickling the dragon by expanding the hand rolled
refcounting implementation in the in memory cache.
BUG=589186
R=mmenke@chromium.org
Review URL: https://codereview.chromium.org/1735183002 .
Cr-Commit-Position: refs/branch-heads/2623@{#535}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=585591
TBR=rch@chromium.org
Review URL: https://codereview.chromium.org/1684353002
Cr-Commit-Position: refs/heads/master@{#375279}
(cherry picked from commit 98f200dee65e6d75bd75f3884255b1a920db5e47)
Review URL: https://codereview.chromium.org/1696053002 .
Cr-Commit-Position: refs/branch-heads/2623@{#397}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
accessing it.
BUG=546668
TBR=rch@chromium.org, asvitkine@chromium.org
Review URL: https://codereview.chromium.org/1656673002
Cr-Commit-Position: refs/heads/master@{#372752}
(cherry picked from commit 3a731f0d7d50c4ec479f71ca3076a09576b44def)
Review URL: https://codereview.chromium.org/1679083002 .
Cr-Commit-Position: refs/branch-heads/2623@{#307}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CL crrev.com/1423333006 introduced a change that removed
net::CertDatabase observers notification when a user cert is added (see
the CertDatabase::AddUserCert method).
This CL restores the notification of these observers. This should fix an
issue that installation of a network certificate does not allow to
connect to the network until user logs out/in.
BUG=581243
Review URL: https://codereview.chromium.org/1657713002
Cr-Commit-Position: refs/heads/master@{#372695}
(cherry picked from commit 09431c8cb6cc4d73cacdc0742fe83062477b4fa8)
Review URL: https://codereview.chromium.org/1669913002 .
Cr-Commit-Position: refs/branch-heads/2623@{#278}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Service Workers.
Merge to 2623 of crrev.com/1544863002.
Service Workers are not tied to AwContents as regular webviews are and
therefore do not invoke callbacks provided in WebViewClient and
WebChromeClient.
This patch provides initial implementation of AwServiceWorkerClient,
AwServiceWorkerSettings and related infrastructure. In particular it
implements the shouldInterceptRequest callback mechanism for service
worker related requests. The callback is invoked when a service worker
script is registered and also for network fetches from within the
service worker script.
This patch also contains a fix for TestWebServer to make it work with
service workers.
BUG=566027
TBR=mnaganov@chromium.org, torne@chromium.org, phajdan.jr@chromium.org
Review URL: https://codereview.chromium.org/1544863002
Cr-Commit-Position: refs/heads/master@{#370998}
(cherry picked from commit 47612e474a203be05f6c4f8f257902615e8406d2)
Review URL: https://codereview.chromium.org/1638393002 .
Cr-Commit-Position: refs/branch-heads/2623@{#160}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=577377
TBR=rch@chromium.org
NOTRY=true
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/1626703002
Cr-Commit-Position: refs/heads/master@{#371136}
(cherry picked from commit 15656aeb0349b493090c47e6cdbf0a6ad70a3e60)
Review URL: https://codereview.chromium.org/1635683002
Cr-Commit-Position: refs/branch-heads/2623@{#123}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=472009
Review URL: https://codereview.chromium.org/1572983003/
Cr-Commit-Position: refs/heads/master@{#370108}
(cherry picked from commit ec0fd9b4321fb0ae5b27ec5ff582b51710ce9f3a)
ADDITIONAL COMMIT:
Fix division by zero. Test filter with empty output.
BUG=579672
Review URL: https://codereview.chromium.org/1616863002
Cr-Commit-Position: refs/heads/master@{#370675}
(cherry picked from commit b266c8153a66f01644f3361fe9cc7d439df1e981)
NOTRY=true
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/1621663002
Cr-Commit-Position: refs/branch-heads/2623@{#73}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The only supported SPDY versions are SPDY/3.1 and HTTP/2, both of which specify
both stream and session level flow control. Therefore enum FlowControlState,
members and accessors of this type, and associated UMA histograms and event
types are obsolete, and are removed by this CL.
BUG=474216
Review URL: https://codereview.chromium.org/1581103006
Cr-Commit-Position: refs/heads/master@{#370255}
(cherry picked from commit 3ab252814c84ebf3fd02a5d5766bc5f14da5e7e8)
Review URL: https://codereview.chromium.org/1619543003 .
Cr-Commit-Position: refs/branch-heads/2623@{#38}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1597553002
Cr-Commit-Position: refs/heads/master@{#369835}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
persisted in prefs file (local_prefs.json).
Added "max_server_configs_stored_in_properties" cronet
experiment options.
Will delete support for "store_server_configs_in_properties",
after all cronet apps stop using it. Added backward
compatibility code that sets
number_of_server_configs_stored_in_properties to 5
when this option is enabled.
R=rch@chromium.org, mef@chromium.org
Review URL: https://codereview.chromium.org/1572753003
Cr-Commit-Position: refs/heads/master@{#369825}
|
|
|
|
|
|
|
|
|
|
|
|
| |
connection type changes but the max bandwidth doesn't.
Most of the CL is me going nutso with tests.
BUG=576315
Review URL: https://codereview.chromium.org/1580103002
Cr-Commit-Position: refs/heads/master@{#369816}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
because of network change, QUIC closes connection and stream, but
QuicHttpStream's continues to process because WriteHeaders returns
the number of bytes written.
Check if stream_ is closed in DoSendHeadersComplete and if so return
the error code (response_status_).
R=rch@chromium.org
BUG=577377
Review URL: https://codereview.chromium.org/1576353011
Cr-Commit-Position: refs/heads/master@{#369808}
|
|
|
|
|
|
|
|
|
|
|
|
| |
various versions of the root store (e.g. if the user disabled root autoupdates), and switch to SHA-256
This ensures more CAs are recognized as publicly trusted CAs and thus constrained to the Baseline Requirements.
BUG=577980
Review URL: https://codereview.chromium.org/1543133002
Cr-Commit-Position: refs/heads/master@{#369775}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1581283002
Cr-Commit-Position: refs/heads/master@{#369761}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds an 'initiator' field to 'URLRequest' in order to keep track
of the origin of the context which initiated a request. This allows us to
correctly perform the initiator requests specified in [1], which prevent
first-party-only cookies from being sent along with a request if that request
is "unsafe", and is initiated from a third-party origin (e.g. a form
submission to 'bank.com' from 'evil.com').
[1]: https://tools.ietf.org/html/draft-west-first-party-cookies-04#section-4.3
BUG=544114
Review URL: https://codereview.chromium.org/1411813003
Cr-Commit-Position: refs/heads/master@{#369754}
|
|
|
|
|
|
|
|
|
|
|
|
| |
QuicDispatcher::PacketWriterFactory and pass/construct QuicPacketWriter*'s directly. No functional change. Not flag protected.
This is a small step towards removing QuicPerConnectionPacketWriter.
BUG=
Review URL: https://codereview.chromium.org/1564363002
Cr-Commit-Position: refs/heads/master@{#369730}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The PAC performance is constrained mainly by DNS resolution time, so we
optimize for memory usage only.
BUG=544232
R=verwaest@chromium.org,eroman@chromium.org
Review URL: https://codereview.chromium.org/1577213004
Cr-Commit-Position: refs/heads/master@{#369714}
|
|
|
|
|
|
|
|
|
|
|
|
| |
SPDY/3 support (and test coverage) has been disabled long-long time ago, see
https://crbug.com/428086. This CL cleans up some enum values that have never
been used.
BUG=431229
Review URL: https://codereview.chromium.org/1579213006
Cr-Commit-Position: refs/heads/master@{#369655}
|
|
|
|
|
|
|
|
| |
BUG=577346
Review URL: https://codereview.chromium.org/1585263004
Cr-Commit-Position: refs/heads/master@{#369606}
|
|
|
|
|
|
| |
Google pins.
Cr-Commit-Position: refs/heads/master@{#369599}
|
|
|
|
|
|
|
|
|
|
|
| |
This change has no semantic effect.
This change sorts some of the early sections of the HSTS JSON file using
sort(1). This does not alter the generated .h file at all, since the
generation script already effectively sorts the values by building a
tree from them.
Cr-Commit-Position: refs/heads/master@{#369589}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://tools.ietf.org/html/draft-ietf-trans-rfc6962-bis-11#section-9.4.2
describes a much simpler proof consistency algorithm which uses less space.
It needs a few additional checks (which should get reflected in the spec)
to enforce everything, but otherwise it works. See discussion on
https://codereview.chromium.org/1434253003.
BUG=506227
Review URL: https://codereview.chromium.org/1577723005
Cr-Commit-Position: refs/heads/master@{#369576}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new WebSocketBlobSender class needs to call SendFrame() from within
its state-machine loop. For efficiency, this is best done
synchronously. In order that it can detect when it should abort because
the channel has been deleted, return ChannelState from SendFrame().
See design document at
https://docs.google.com/document/d/1CDiXB9pBumhFVVfmIn1CRI6v6byxyqWu2urEE9xp714/edit
Also remove some obsolete comments from the SendFrame() method.
BUG=571656
TEST=net_unittests
Review URL: https://codereview.chromium.org/1581613003
Cr-Commit-Position: refs/heads/master@{#369530}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HSTS preload list.
o7.com: "our domain was incorrectly submited to the HSTS list" (broke www.smp-31.o7.com)
lucameraga.it: tried HSTS on CloudFlare, changed their mind
kotarac.net, hostelinsplit.com: "noticed an issue [with subdomains] in the latest dev build."
BUG=527947, 576249
TBR=agl@chromium.org
Review URL: https://codereview.chromium.org/1588803002
Cr-Commit-Position: refs/heads/master@{#369520}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Employ the same trick used at other places in DnsQuery to calculate the
beginning of the question section.
This allow us to write the code without the reinterpret_cast.
BUG=None
TEST=net_unittests --gtest_filter=*Dns*
R=ttuttle@chromium.org
Review URL: https://codereview.chromium.org/1579413002
Cr-Commit-Position: refs/heads/master@{#369433}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This seems to be a more appropriate place for this function,
than the generic net_util.h header file, which will soon be gone.
BUG=488531
TEST=net_unittests
R=eroman@chromium.org
Review URL: https://codereview.chromium.org/1582083002
Cr-Commit-Position: refs/heads/master@{#369403}
|
|
|
|
|
|
|
|
|
|
| |
connections to migrate to a new network when an old network is disconnected or about to be disconnected. These notifications are available for Android >= L, so QUIC connection migration will only work for Android >= L.
BUG=576998
Review URL: https://codereview.chromium.org/1327923002
Cr-Commit-Position: refs/heads/master@{#369356}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There isn't much that uses bit_cast. bit_cast requires memcpy which
requires <string.h>
On Mac, <string.h> includes AvailabilityInternal.h which is half a
megabyte and defines dozens of macros. Providing <string.h> in
base/macros.h also tempts IWYU violations. Try to avoid it.
Add base/bit_cast.h and include it from the 17 files that need it. Then
remove the <string.h> #include from base/macros.h and IWYU.
BUG=395001
Review URL: https://codereview.chromium.org/399313006
Cr-Commit-Position: refs/heads/master@{#369348}
|
|
|
|
|
|
|
|
|
|
| |
BUG=557422
R=rickyz@chromium.org
TBR=jam@chromium.org
Review URL: https://codereview.chromium.org/1586833002
Cr-Commit-Position: refs/heads/master@{#369328}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1573243011
Cr-Commit-Position: refs/heads/master@{#369311}
|
|
|
|
|
|
|
|
|
| |
BUG=
TBR=agl@chromium.org
Review URL: https://codereview.chromium.org/1588753002
Cr-Commit-Position: refs/heads/master@{#369305}
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=576402
Committed: https://crrev.com/1ad82bbc44d87669926c9cffe2ba77564df420a3
Cr-Commit-Position: refs/heads/master@{#368810}
Review URL: https://codereview.chromium.org/1580583002
Cr-Commit-Position: refs/heads/master@{#369291}
|
|
|
|
|
|
|
|
|
|
| |
Rather than CHECK-ing that test data was parsed correctly, the tests now assert it was parsed correctly so bad data change would be easier to catch.
BUG=
Review URL: https://codereview.chromium.org/1578303002
Cr-Commit-Position: refs/heads/master@{#369199}
|
|
|
|
|
|
|
|
|
|
|
| |
Manual reset events should be reset before reuse. The old code
seems to work as intended, but it's not clear why.
BUG=575722
Review URL: https://codereview.chromium.org/1563323004
Cr-Commit-Position: refs/heads/master@{#369170}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1572253002/ )
Reason for revert:
This issue is known in crbug.com/506663 and we want to fix c++ spec and clang rather than making chromium code silly.
Nico asked to revert it.
Original issue's description:
> net: build fix about CertificateList
>
> clang complains it doesn't initialize const member, so this CL calls the
> default constructor explicitly.
>
> ../../net/cert/multi_threaded_cert_verifier.cc:430:25: error: default initialization of an object of const type 'const CertificateList' (aka 'const vector<scoped_refptr<net::X509Certificate> >') without a user-provided default constructor
> const CertificateList empty_cert_list;
> ^
> 1 error generated.
>
> Committed: https://crrev.com/18937ad93812903be37e75f60510ed258c4117e1
> Cr-Commit-Position: refs/heads/master@{#368585}
TBR=pauljensen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
Review URL: https://codereview.chromium.org/1584783002
Cr-Commit-Position: refs/heads/master@{#369131}
|
|
|
|
|
|
|
|
|
| |
Use more scoped_ptr is url_request_filter which allows to
delete and clean code and make it safer.
Review URL: https://codereview.chromium.org/1557583002
Cr-Commit-Position: refs/heads/master@{#369113}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch moves SockaddrStorage type into its own header file,
as it seems more appropriate than in the generic net_util.h. This
should help breakout net_util.h even further. We are almost there.
BUG=488531
TEST=net_unittests
R=eroman@chromium.org
TBR=vitalybuka@chromium.org
Review URL: https://codereview.chromium.org/1584503002
Cr-Commit-Position: refs/heads/master@{#369081}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1580583002/ )
Reason for revert:
Speculative revert for Android Cronet bots failure (CronetTestInstrumentation)
Original issue's description:
> Add a whitelist for QUIC hosts.
>
> BUG=576402
>
> Committed: https://crrev.com/1ad82bbc44d87669926c9cffe2ba77564df420a3
> Cr-Commit-Position: refs/heads/master@{#368810}
TBR=agl@chromium.org,rsleevi@chromium.org,rch@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576402
Review URL: https://codereview.chromium.org/1574423003
Cr-Commit-Position: refs/heads/master@{#369050}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The workaround is now known to be quite effective, so that confirms the
remaining fallback mass is indeed the Microsoft bug. Numbers were quite low all
around, so I would still like to try to drive those sites to get fixed given
how straightforward the fix is (take Windows updates).
To that end, remove the experimental workaround now so Chrome 49's fallback
metrics will reflect the strategy. If getting those servers fixed fails, we can
restore the workaround as a fallback (hah) strategy.
BUG=536200
Review URL: https://codereview.chromium.org/1580833002
Cr-Commit-Position: refs/heads/master@{#369001}
|
|
|
|
|
|
|
|
|
|
|
| |
This is a cleanup in preparation for implementing some CT changes
including Expect-CT.
BUG=568806
Review URL: https://codereview.chromium.org/1579233002
Cr-Commit-Position: refs/heads/master@{#368993}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a missing return statement so that
SpdySession::TryCreatePushStream returns false when a cross-
origin push by a trusted SPDY proxy advertises an HTTPS URL.
Also, add tests that verify the correctness of trusted SPDY
proxy.
BUG=572766, 572767
Review URL: https://codereview.chromium.org/1545403002
Cr-Commit-Position: refs/heads/master@{#368973}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
relnote: Allowing multipath support negotiation. Protected by --quic_enable_multipath, default value is false.
Merge internal change: 111718558
https://codereview.chromium.org/1572033003
relnote: Implement server push methods in QuicSimpleServerSession. Only modify toy Quic server, not used in production.
Merege internal change: 111716301
https://codereview.chromium.org/1570343005
relnote: Require QUIC handshakes to require either a valid server nonce or a remote strike register.
Merge internal change: 111708360
https://codereview.chromium.org/1569853005
relnote: n/a test-only change.
Change QUIC's General and TCP loss algorithm tests to create
RetransmittableFrames with a QuicStreamFrame instead of no stream
frames.
Merge internal change: 111655037
https://codereview.chromium.org/1570363002
relnote: Re-enable strike register lookups for QUIC replay protection, until at least QUIC_VERSION_31. Not flag protected, as Chrome currently only talks QUIC_VERSION_25.
Merge internal change: 111628540
https://codereview.chromium.org/1574633002
relnote: n/a (test only)
Always create a SerializedPacket with at least one QuicFrame in the
RetransmittableFrames.
Merge internal change: 111570157
https://codereview.chromium.org/1576553003
relnote: quic_supports_push_promise should be disabled by default.
Merge internal change: 111560672
https://codereview.chromium.org/1576623002
relnote: QUIC header streams support to receive PUSH_PROMISE. Protected by --quic_supports_push_promise.
This is part of shared code, but client side functionality only.
Merge internal change: 111542509
https://codereview.chromium.org/1572013002
relnote: always sending kFIXD And extending the lifetime of the flag. When we deprecate the old version we can flip it false again and then nuke the code.
Merge internal change: 111507546
https://codereview.chromium.org/1574623002
relnote: Merge all fields of QueuedPacket into SerializedPacket. No functional change.
Makes it easier to make SerializedPacket a class that owns its members
in the future.
Merge internal change: 111444711
https://codereview.chromium.org/1569823008
relnote: Deprecate reloadable_flag_quic_inplace_encryption.
Merge internal change: 111440524
https://codereview.chromium.org/1570553002
relnote: add new unused functions for client side of QUIC server push. QUIC - Add helper functions for SpdyHeaderBlock values.
Merge internal change: 111439689
https://codereview.chromium.org/1561383002
relnote: n/a (test only) QUIC - make MockConnectionHelper::TestAlarm public.
This is test only, and no-op for now. It will be used by a subsequent
CL in support of client-side server push.
Merge internal change: 111427940
https://codereview.chromium.org/1563083002
relnote: Adds Slow Start with Large Reduction (SSLR) option in QUIC to do a large reduction of the congestion window when exiting slow start.
When exiting slow start due to loss, SSLR causes the congestion window
to be reduced by the number of losses encountered at the end of slow
start. SSLR is expected to reduce retransmission rates when exiting slow
start, and it should also help the congestion window converge rapidly
when the connection's actual BDP is smaller than the IW.
Merge internal change: 111385694
https://codereview.chromium.org/1566853002
relnote: Clear the FEC group in QuicPacketCreator when encryption_level is none and close the connection if SerializeFec is called. Protected by gfe2_reloadable_flag_never_write_unencrypted_data.
Merge internal change: 111385311
https://codereview.chromium.org/1562833003
relnote: Moving many QUIC DFATALS over to QUIC_BUG
Merge internal change: 111359954
https://codereview.chromium.org/1565883003
relnote: Change memcpy to memmove in QuicFramer::EncryptPayload, because the src and dest buffers may be identical. No functional change.
Merge internal change: 111345350
https://codereview.chromium.org/1558423002
rel-note: A small refactoring to make RecordSpuriousRetransmission() of QuicSentPacketManager simpler. (no functional change)
Merge internal change: 111321766
https://codereview.chromium.org/1566633002
BUG=
Review URL: https://codereview.chromium.org/1579033002
Cr-Commit-Position: refs/heads/master@{#368964}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HpackHuffmanDecoder uses less memory and decodes faster (around 30% CPU
reduction in micro benchmark) than previous HpackHuffmanTable decoder. It is
hard coded; i.e., it cannot decode arbitrary codes, but since the
standardization process is complete, that does not seem necessary.
This CL lands server changes 103090098 and 109220294 by jamessynge
and 111872403 by bnc.
BUG=488484
Review URL: https://codereview.chromium.org/1568423002
Cr-Commit-Position: refs/heads/master@{#368954}
|
|
|
|
| |
Cr-Commit-Position: refs/heads/master@{#368891}
|
|
|
|
|
|
|
|
|
|
|
|
| |
According to RFC6962, Signed Tree Heads for trees of size 0 (i.e. the empty tree) must have, as the root hash, the hash of the empty string.
This change adds this check to the STH verification function.
BUG=563471
Review URL: https://codereview.chromium.org/1542753005
Cr-Commit-Position: refs/heads/master@{#368885}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Nothing in net_util_unittest.cc uses it so we remove it. Plus removed a
few unused header includes.
BUG=488531
TEST=net_unittests --gtest_filter=NetUtil*
R=eroman@chromium.org
Review URL: https://codereview.chromium.org/1573123004
Cr-Commit-Position: refs/heads/master@{#368861}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a public type, used outside of net (at the time of this
writting, by cloud_print and chrome), so the use of NET_EXPORT_PRIVATE
is not correct.
BUG=552248
R=ttuttle@chromium.org
Review URL: https://codereview.chromium.org/1573893003
Cr-Commit-Position: refs/heads/master@{#368860}
|
|
|
|
|
|
|
|
| |
BUG=576402
Review URL: https://codereview.chromium.org/1580583002
Cr-Commit-Position: refs/heads/master@{#368810}
|