| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
CID=1634, 11587, 5007, 13234, 13398, 9295, 8294, 8285
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3817007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62945 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The interface to CookieMonster::GarbageCollectDeleteList() was
signalling "delete all cookies in list" by specifying "delete all
cookies in list with last access time before Now()". This was failing
to work properly when the system clock wasn't updated frequently
enough (== flaky test) and is also vulnerable to changes in systems
clocks. That semantic is now explicitly signalled by a null time.
BUG=58197
TEST=net_unittests CookieMonsterTest.* on Win/Linux and CookieMonsterTest.TestHostGarbageCollection (flaky test) repeatedly on Windows.
Review URL: http://codereview.chromium.org/3780004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62943 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=57686
TEST=net_unittests CookieMonsterTest.* on Linux, running original valgrind test and confirming that it completed.
Review URL: http://codereview.chromium.org/3752004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62942 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
clang would complain
/Users/thakis/src/chrome-git/src/net/base/host_resolver_impl.cc:557:5: error: template argument uses local type 'net::HostResolverImpl::Job::Category' [-Wlocal-type-template-args]
DCHECK_LT(category, RESOLVE_MAX); // Be sure it was set.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /Users/thakis/src/chrome-git/src/net/base/host_resolver_impl.cc:5:
In file included from ../net/base/host_resolver_impl.h:18:
In file included from ../net/base/network_change_notifier.h:10:
In file included from ../base/observer_list_threadsafe.h:14:
../base/logging.h:662:31: note: instantiated from:
#define DCHECK_LT(val1, val2) DCHECK_OP(LT, < , val1, val2)
^
../base/logging.h:635:9: note: instantiated from:
logging::Check##name##Impl((val1), (val2), \
^~~~~~~
This seems to be a new warning. Since it only affects this file, we might as well fix the problem instead of disabling the warning.
BUG=None
TEST=clang/mac waterfall stays green after clobber build.
Review URL: http://codereview.chromium.org/3816009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62934 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=45135
TEST=Build works.
Review URL: http://codereview.chromium.org/3747008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62927 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=net_unittests --gtest_filter=*X509*
Review URL: http://codereview.chromium.org/3785001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62925 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Create an SSLHostInfoFactory interface and plumb it from the HttpCache
to the SSLConnectJob. Also, move the SSLHostInfo reference from the
ssl_config to being passed to the SSLClientSocket.
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/3747003
TBR=agl@chromium.org
Review URL: http://codereview.chromium.org/3846005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62922 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Create an SSLHostInfoFactory interface and plumb it from the HttpCache
to the SSLConnectJob. Also, move the SSLHostInfo reference from the
ssl_config to being passed to the SSLClientSocket.
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/3747003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62918 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
removes windows_message_list which isn't used.
This keeps a stub for registry in the old location until we can update that.
TEST=it compiles
BUG=none
Review URL: http://codereview.chromium.org/3836005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62888 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
properly (scoped_cftyperef -> ScopedCFTypeRef).
TEST=it compiles
BUG=none
Review URL: http://codereview.chromium.org/3855001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62887 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
The check-in r62746 clobbered the changes I made in r62689 to host_resolver_impl.cc.
I believe that was an accident since r62746 moved code within the file.
TBR=jar
Review URL: http://codereview.chromium.org/3828008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62879 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Only use Alternate-Protocol for latching onto existing SpdySessions.
This is a temporary change until SSL improvements are made, so we
can begin sending http over spdy when available.
Note that we have to do proxy resolution one time to figure out how to check for the appropriate existing SpdySession. If no SpdySession exists, we fallback again and have to redo proxy resolution (since the alternate protocol of npn-spdy requires rewriting the URI from http:// to https://). So we may end up doing two proxy resolutions in the fallback case.
I had to disable all the tests that test establishing a new
SpdySession for http:// URIs, since we don't do that for now.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3573013
TBR=willchan@chromium.org
Review URL: http://codereview.chromium.org/3751009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62873 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Also converts COOKIE_DLOG (since VLOG can be toggled on a granular basis), removes some unneeded "endl"s and {}s, aligns "<<"s per style guide, and changes KeygenHandler::GenKeyAndSignChallenge() to avoid using "goto".
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3846001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62831 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I'm doing tests to see what the impact of changing the
speculative resolution system parameters is on
resolution time. By default, the speculative system
limits itself to no more than 8 pending resolutions
at any time, and this test looks at 6, 4, and 2
as limits.
It also modulates the speculative congestion avoidance
system, which purges the speculative queue when there is
too much resolution latency, and tries various durations
of time. (the latter was less important... but was already
part of this existing A/B test.)
I also pushed the test rates up to 10% for each case, so
that we'd get data from the canary and/or dev builds.
r=eroman
Review URL: http://codereview.chromium.org/3762006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62746 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
getaddrinfo() has unexpectedly failed.
For example when the system returns WSA_NOT_ENOUGH_MEMORY, we will now show ERR_NAME_RESOLUTION_FAILED rather than ERR_NAME_NOT_RESOLVED.
This allows propagating the error to the user, rather than simply showing them say the linkdoctor page as if the domain they entered was incorrect.
The data shows that a small number of users are getting these sorts of unexpected failures, and I want to get more visibility into it. By having a separate error code we can track how often this actually floats up onto an error page (as opposed to just errors on non-essential background requests).
On Linux we also see EAI_AGAIN with some frequency. My theory is that these occur when you DNS resolve immediately after the network change notification was broadcast (since I can repro this 100% on my laptop). If this is in fact the major cause, then once I land this new error we should NOT see an influx of ERR_NAME_RESOLUTION_FAILED in Net.ErrorCodesForMainFrame.
Review URL: http://codereview.chromium.org/3781005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62705 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
This makes it easier to display the symbolic names of each error code on the server side, since we specify a mapping per histogram name.
Review URL: http://codereview.chromium.org/3771005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62689 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
the Options menu ("Under the Hood" tab) on Windows.
R=mattm
BUG=59125
TEST=net_unittests.exe --gtest_filter=SSLConfigServiceWinTest.SetTest.
Manually verify the Options menu has the two new checkboxes.
Review URL: http://codereview.chromium.org/3779003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62634 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=54745
TEST=manual
Review URL: http://codereview.chromium.org/3582007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62627 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I don't think this needs to be reverted, but git cl dcommit went a bit wacky and I'm being conservative.
The goal is to turn on -Wnon-virtual-dtor warning for gcc. In this case, the virtual Run()'s do not appear necessary,
especially since they are private. If intended to be derived from, there is a "delete this" in the base class which would
point to requiring a virtual destructor.
BUG=45135
TEST=Target builds, validated that there are no derived classes of RRResolverWorker.
Review URL: http://codereview.chromium.org/3824002
TBR=cbentzel@chromium.org
Review URL: http://codereview.chromium.org/3813005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62611 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The goal is to turn on -Wnon-virtual-dtor warning for gcc. In this case, the virtual Run()'s do not appear necessary,
especially since they are private. If intended to be derived from, there is a "delete this" in the base class which would
point to requiring a virtual destructor.
BUG=45135
TEST=Target builds, validated that there are no derived classes of RRResolverWorker.
Review URL: http://codereview.chromium.org/3824002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62605 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
CID=13288
Review URL: http://codereview.chromium.org/3781002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62532 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
put them in the base namespace.
TEST=it compiles
BUG=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62510 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=trybots
Review URL: http://codereview.chromium.org/3750001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62483 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
to support fetching HTTPS URLS over a SPDY Proxy.
BUG=29625
TEST=HttpNetworkTransactionTest.HttpsProxySpdyConnect
Review URL: http://codereview.chromium.org/3417010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62468 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a RequestSockets() API to ClientSocketPool interface.
- no RequestPriority param, all requests default to LOWEST.
- adds a |num_sockets| param to control how many sockets to try to ensure are connected.
Adds an implementation for said function in ClientSocketPoolBaseHelper.
Adds a new ClientSocketPoolBaseHelper::Flag type to modify socket request behavior. In this case, we bypass idle sockets.
Adds a preconnect concept to ConnectJob. This lets normal requests hijack preconnect jobs.
Modifies all ClientSocketPool subclasses to support new RequestSockets API().
Adds new tests.
No client actually uses this API yet. We need to plumb it up to the preconnect system.
BUG=54450
TEST=new tests in ClientSocketPoolBaseTest
Review URL: http://codereview.chromium.org/3689004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62365 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=33185
r=kmixter
Review URL: http://codereview.chromium.org/3702005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62353 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Minor cleanup of CertificateManagerModel by removing some wrappers of CertDatabase readonly functions.
BUG=19991
TEST=manual
Review URL: http://codereview.chromium.org/3516023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62348 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
changing GetCredHandle to return 'int' (the error code).
Add ERR_SSL_CLIENT_AUTH_PRIVATE_KEY_ACCESS_DENIED and
ERR_SSL_CLIENT_AUTH_CERT_NO_PRIVATE_KEY.
R=eroman
BUG=55678
TEST=none
Review URL: http://codereview.chromium.org/3696006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62336 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
net-internals tab, with links to the corresponding sources on
the events tab.
Includes a minor cleanup or two relating to unneeded/unused code/headers for the rendering of SocketPools.
BUG=58034
TEST=manual
Review URL: http://codereview.chromium.org/3565015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62329 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=57456
TEST=valgrind net_unittests
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62323 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
No effective code change.
BUG=none
TEST=net_unittests
Review URL: http://codereview.chromium.org/3728002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62295 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When _res is uninitialized, res_nclose may end up closing fd 0.
Test whether it's initialized before closing.
Patch by Jim Deng.
TBR=evanm
Original review URL: http://codereview.chromium.org/3717002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62292 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When _res is uninitialized, res_nclose may end up closing fd 0.
Test whether it's initialized before closing.
Patch by Jim Deng <jdeng@google.com>.
TEST=chrome without a net connection doesn't crash on startup
Review URL: http://codereview.chromium.org/3717002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62288 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62240 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19991
TEST=net_unittests
Review URL: http://codereview.chromium.org/3576016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62228 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ESET products (anti-virus) and NetNanny can intercept HTTPS connections. However,
they're False Start intolerant. Rather than have connections hang
forever, we detect them by the leaf certificate's issuer common name
and give a helpful error message.
BUG=58152
TEST=Install ESET anti-virus. Configure to intercept HTTPS connections. Goto https://mail.google.com. Confirm that a helpful error page appears.
Review URL: http://codereview.chromium.org/3723001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62209 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, we lumped the intranet host warning in with mixed content warning. This calls it out as a separate warning.
BUG=58162
TEST=None
Review URL: http://codereview.chromium.org/3536019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62178 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a prelim commit for renaming SSLNonSensitiveHostInfo to
SSLHostInfo.
No code change.
BUG=none
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62174 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
the entire certificate
R=wtc
BUG=None
TEST=Build with use_openssl=1, X509Certificate* tests continue to work
Review URL: http://codereview.chromium.org/3677001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62091 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
ERR_ADDRESS_UNREACHABLE, when the user is in offline mode.
This initial changelist includes the implementation just for Windows.
BUG=53473
TEST=unplug network cable, try to connect to www.google.com --> should get the error ERR_INTERNET_DISCONNECTED. Next, try connecting directly to an IP address --> should get the error ERR_INTERNET_DISCONNECTED.
Review URL: http://codereview.chromium.org/3634002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62050 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
if it's really an error
BUG=56681
TEST=none
Review URL: http://codereview.chromium.org/3620015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61965 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also adds SSL_CTX to the OpenSSLInitSingleton to make initialization thread-safe.
Note this depends on http://codereview.chromium.org/3529008
BUG=None
TEST=build with use_openssl=1, open some https pages
Review URL: http://codereview.chromium.org/3591015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61963 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Depends on http://codereview.chromium.org/3565006/show
TEST=Existing *X509* tests should pass.
Review URL: http://codereview.chromium.org/3529008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61955 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change is a pre-requisite for http://codereview.chromium.org/3529008/show
There are no functional changes, it's only refactoring existing code.
BUG=None
TEST=Go to an https:// page and check the certificate info.
Review URL: http://codereview.chromium.org/3565006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61944 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Only use Alternate-Protocol for latching onto existing SpdySessions.
This is a temporary change until SSL improvements are made, so we
can begin sending http over spdy when available.
Note that we have to do proxy resolution one time to figure out how to check for the appropriate existing SpdySession. If no SpdySession exists, we fallback again and have to redo proxy resolution (since the alternate protocol of npn-spdy requires rewriting the URI from http:// to https://). So we may end up doing two proxy resolutions in the fallback case.
I had to disable all the tests that test establishing a new
SpdySession for http:// URIs, since we don't do that for now.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3573013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61897 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19991
TEST=manual
Review URL: http://codereview.chromium.org/3549020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61895 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and adds an extra note emphasizing when IPv6 has been disabled to both
pages.
Also adds a button to enable IPv6 when it's been disabled, and maps
address family values on the DNS tab to their symbolic names.
BUG=55007,55005
TEST=manual
Review URL: http://codereview.chromium.org/3387007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61795 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=None
BUG=0
Review URL: http://codereview.chromium.org/3604010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61748 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Delete certs.
BUG=19991
TEST=manual
Review URL: http://codereview.chromium.org/3556007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61701 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding FLAKY designation to this test.
http://build.chromium.org/buildbot/waterfall/builders/Modules%20XP/builds/35620/steps/net_unittests/logs/stdio
[ OK ] CookieMonsterTest.TestLastAccess (219 ms)
[ RUN ] CookieMonsterTest.TestHostGarbageCollection
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 181 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 183 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 185 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 187 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 189 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 191 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 193 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 195 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 197 vs 180
.\base\cookie_monster_unittest.cc(1080): error: Expected: ((CountInString(cookies_general, '=') + CountInString(cookies_specific, '='))) <= (domain_max_cookies), actual: 199 vs 180
[ FAILED ] CookieMonsterTest.TestHostGarbageCollection (219 ms)
[ RUN ] CookieMonsterTest.NetUtilCookieTest
BUG=58197
TEST=CookieMonsterTest.MAYBE_TestHostGarbageCollection
TBR=rdsmith@chromium.org
Review URL: http://codereview.chromium.org/3610014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61699 0039d316-1c4b-4281-b951-d872f2087c98
|