| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
histograms.xml file.
BUG=416561
Review URL: https://codereview.chromium.org/603023002
Cr-Commit-Position: refs/heads/master@{#296777}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/598053003
Cr-Commit-Position: refs/heads/master@{#296761}
|
|
|
|
|
|
|
|
|
|
| |
Not exact revert of https://codereview.chromium.org/488853002
BUG=374321,416396
Review URL: https://codereview.chromium.org/597933002
Cr-Commit-Position: refs/heads/master@{#296751}
|
|
|
|
|
|
|
|
|
| |
BUG=318404
TBR=zea@chromium.org,mkosiba@chromium.org
Review URL: https://codereview.chromium.org/600983002
Cr-Commit-Position: refs/heads/master@{#296734}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
also:
- ensure that error message is displayed for empty title
- move dom_distiller_viewer.html/js from content to core dir
- added tests.
BUG=403066
Review URL: https://codereview.chromium.org/563923002
Cr-Commit-Position: refs/heads/master@{#296719}
|
|
|
|
|
|
|
|
|
|
| |
Don't validate refresh token before usage to reduce network traffic.
BUG=359700,416612
Review URL: https://codereview.chromium.org/590113004
Cr-Commit-Position: refs/heads/master@{#296715}
|
|
|
|
|
|
|
|
| |
BUG=None
Review URL: https://codereview.chromium.org/599373002
Cr-Commit-Position: refs/heads/master@{#296698}
|
|
|
|
|
|
|
|
|
|
|
| |
Move history_types.{cc,h} and android_history_types.{cc,h} to the
history component and create a new target history_core_android.
BUG=371816
Review URL: https://codereview.chromium.org/584013002
Cr-Commit-Position: refs/heads/master@{#296684}
|
|
|
|
|
|
|
|
|
|
|
| |
So no need to remember the correct header name for one for string resources and one for image resources.
BUG=416395
TBR=jochen@chromium.org for name fix under chrome
Review URL: https://codereview.chromium.org/601733002
Cr-Commit-Position: refs/heads/master@{#296673}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The unittests (WebDataServiceAutofillTest.CreditUpdate) assume that the
newly modified credit card information will be the first one returned.
Since it has the highest date_modified value, this imply a descending
ordering of the cards when iterating.
The unittests were previously flaky as the precision of date_modified
is not high enough and both card ended up with the same value most of
the time. When the test was taking slightly more time to run, then the
modified car would end up last and the test would fail.
BUG=None
Review URL: https://codereview.chromium.org/600873002
Cr-Commit-Position: refs/heads/master@{#296623}
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TBR=mpearson@chromium.org
R=jiayul@chromium.org
Review URL: https://codereview.chromium.org/601033002
Cr-Commit-Position: refs/heads/master@{#296604}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
found something that it can handle.
For chrome/app/chrome_command_ids.h
TBR=cpu@chromium.org
BUG=416582
Review URL: https://codereview.chromium.org/599653002
Cr-Commit-Position: refs/heads/master@{#296594}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #1 id:1 of https://codereview.chromium.org/598253002/)
Reason for revert:
Broken GN build:
http://build.chromium.org/p/chromium.linux/builders/Linux%20GN%20%28dbg%29/builds/13101
Original issue's description:
> Revert of [AiS] Merge answers into highest-scoring result (patchset #6 id:100001 of https://codereview.chromium.org/584653004/)
>
> Reason for revert:
> The new tests are flaky:
> http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%283%29/builds/7944
>
> Original issue's description:
> > [AiS] Merge answers into highest-scoring result
> >
> > Answer contents and type must always be copied to the highest-scoring
> > suggestion, to ensure answers are not lost due to suggestion
> > deduplication.
> >
> > BUG=415836
> > R=mpearson
> >
> > Committed: https://crrev.com/978a4bf4a3be714950ed265660bd9566f7abd922
> > Cr-Commit-Position: refs/heads/master@{#296520}
>
> TBR=mpearson@google.com,mpearson@chromium.org,newt@chromium.org,groby@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=415836
>
> Committed: https://crrev.com/ba5d32190d4f056b11738b050b94895b655c52f7
> Cr-Commit-Position: refs/heads/master@{#296555}
TBR=mpearson@google.com,mpearson@chromium.org,newt@chromium.org,groby@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=415836
Review URL: https://codereview.chromium.org/599273002
Cr-Commit-Position: refs/heads/master@{#296573}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is to make sure the starting list of account
mapping properly reflects the mappings present in the
previous Chrome session.
BUG=374969
Review URL: https://codereview.chromium.org/600053002
Cr-Commit-Position: refs/heads/master@{#296567}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/599243002/)
Reason for revert:
The original revert was a mistake. Relanding this with no changed.
Original issue's description:
> Revert of Fix doodle verification URL. (patchset #3 id:40001 of https://codereview.chromium.org/587943003/)
>
> Reason for revert:
> Caused test failure:
> http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%283%29/builds/7944
>
> Original issue's description:
> > Fix doodle verification URL.
> >
> > When verifying that the cached doodle is still valid, we load the doodle
> > URL and append the query param "async=es_dfp:<fingerprint>". Previously,
> > the ":" was being escape to "%3A", causing the server to respond with a
> > 400 error. This mollifies the server by keeping the colon unescaped.
> >
> > BUG=413845
> >
> > Committed: https://crrev.com/673cc76103079eaada968c537c6605dbaf8d909c
> > Cr-Commit-Position: refs/heads/master@{#296512}
>
> TBR=justincohen@chromium.org,rsleevi@chromium.org,mmenke@chromium.org,newt@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=413845
>
> Committed: https://crrev.com/6101b6321585a402ffc7c45dbb4fc61bc36f0051
> Cr-Commit-Position: refs/heads/master@{#296546}
TBR=justincohen@chromium.org,rsleevi@chromium.org,mmenke@chromium.org,groby@chromium.org,jiayl@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=413845
Review URL: https://codereview.chromium.org/606443004
Cr-Commit-Position: refs/heads/master@{#296556}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:100001 of https://codereview.chromium.org/584653004/)
Reason for revert:
The new tests are flaky:
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%283%29/builds/7944
Original issue's description:
> [AiS] Merge answers into highest-scoring result
>
> Answer contents and type must always be copied to the highest-scoring
> suggestion, to ensure answers are not lost due to suggestion
> deduplication.
>
> BUG=415836
> R=mpearson
>
> Committed: https://crrev.com/978a4bf4a3be714950ed265660bd9566f7abd922
> Cr-Commit-Position: refs/heads/master@{#296520}
TBR=mpearson@google.com,mpearson@chromium.org,newt@chromium.org,groby@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=415836
Review URL: https://codereview.chromium.org/598253002
Cr-Commit-Position: refs/heads/master@{#296555}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:220001 of https://codereview.chromium.org/543753002/)
Reason for revert:
One of the new unit tests failed with uninit memory access: http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20%28valgrind%29%282%29/builds/43137/
Original issue's description:
> [Suggestions] Move ImageManager to the component
>
> Keeps the fetcher functionality as ImageFetcherImpl in chrome/browser/search/suggestions.
>
> BUG=387751
> TBR=blundell
> TEST=ImageManagerTest,ImageFetcherBrowserTest
>
> Committed: https://crrev.com/5a25f30080dd57d200679546c970d70f2247def9
> Cr-Commit-Position: refs/heads/master@{#296503}
TBR=huangs@chromium.org,blundell@chromium.org,mathp@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=387751
Review URL: https://codereview.chromium.org/600203002
Cr-Commit-Position: refs/heads/master@{#296553}
|
|
|
|
|
|
|
|
|
|
|
| |
This is to intended to match the HTTP/2 static table:
http://http2.github.io/http2-spec/compression.html#static.table.definition (entry 16).
BUG=389914
Review URL: https://codereview.chromium.org/437353004
Cr-Commit-Position: refs/heads/master@{#296551}
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=tengs@chromium.org
Review URL: https://codereview.chromium.org/598833002
Cr-Commit-Position: refs/heads/master@{#296549}
|
|
|
|
|
|
|
|
|
|
|
|
| |
EnhancedBookmarkBridge is only used in android platform and should be
moved to chrome/browser folder, bacause most bookmark services going
through this bridge requires profile.
BUG=415774
Review URL: https://codereview.chromium.org/597253003
Cr-Commit-Position: refs/heads/master@{#296547}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/587943003/)
Reason for revert:
Caused test failure:
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%283%29/builds/7944
Original issue's description:
> Fix doodle verification URL.
>
> When verifying that the cached doodle is still valid, we load the doodle
> URL and append the query param "async=es_dfp:<fingerprint>". Previously,
> the ":" was being escape to "%3A", causing the server to respond with a
> 400 error. This mollifies the server by keeping the colon unescaped.
>
> BUG=413845
>
> Committed: https://crrev.com/673cc76103079eaada968c537c6605dbaf8d909c
> Cr-Commit-Position: refs/heads/master@{#296512}
TBR=justincohen@chromium.org,rsleevi@chromium.org,mmenke@chromium.org,newt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=413845
Review URL: https://codereview.chromium.org/599243002
Cr-Commit-Position: refs/heads/master@{#296546}
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=tengs@chromium.org
Review URL: https://codereview.chromium.org/592353003
Cr-Commit-Position: refs/heads/master@{#296544}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previous CL for adding base_search_provider_unittest.cc did not update
BUILD.gn - this CL does.
BUG=none
TBR=mpearson@chromium.org
Review URL: https://codereview.chromium.org/600963002
Cr-Commit-Position: refs/heads/master@{#296538}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Answer contents and type must always be copied to the highest-scoring
suggestion, to ensure answers are not lost due to suggestion
deduplication.
BUG=415836
R=mpearson
Review URL: https://codereview.chromium.org/584653004
Cr-Commit-Position: refs/heads/master@{#296520}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When verifying that the cached doodle is still valid, we load the doodle
URL and append the query param "async=es_dfp:<fingerprint>". Previously,
the ":" was being escape to "%3A", causing the server to respond with a
400 error. This mollifies the server by keeping the colon unescaped.
BUG=413845
Review URL: https://codereview.chromium.org/587943003
Cr-Commit-Position: refs/heads/master@{#296512}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Keeps the fetcher functionality as ImageFetcherImpl in chrome/browser/search/suggestions.
BUG=387751
TBR=blundell
TEST=ImageManagerTest,ImageFetcherBrowserTest
Review URL: https://codereview.chromium.org/543753002
Cr-Commit-Position: refs/heads/master@{#296503}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Keep track of the situations when Chrome expects the data reduction
proxy via header to be present in a response, but the data reduction
proxy via header is missing.
This change is the same as the reverted change
https://codereview.chromium.org/577343002/,
except that it includes a 1-line fix in
data_reduction_proxy_usage_stats_unittests.cc to make the tests pass
when run with Valgrind in release builds.
BUG=412888
Review URL: https://codereview.chromium.org/602503002
Cr-Commit-Position: refs/heads/master@{#296496}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The original usage of the method was added to fix a rendering issue
with the YouTube plugin: see http://crrev.com/2862031 .
Setting an appropriate viewport seems to work as well
and removes usage of a method I'm trying to remove.
BUG=417221
Review URL: https://codereview.chromium.org/596253002
Cr-Commit-Position: refs/heads/master@{#296440}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On CrOS, app windows can already be minimized by clicking their icon in the
taskbar. This just makes them show a minimize button as well.
On Windows, we prevent minimize if the window can't maximize, which
doesn't really make sense. They will now be able to minimize and show a
minimize button.
On Mac and Linux they could always minimize.
BUG=351179
Review URL: https://codereview.chromium.org/603523002
Cr-Commit-Position: refs/heads/master@{#296437}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should reflect the changes in
https://critique.corp.google.com/#review/72343582
BUG=416806
TEST=unit tests
Review URL: https://codereview.chromium.org/601743002
Cr-Commit-Position: refs/heads/master@{#296403}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There were some cases where we use CanMaximize to determine if
minimization should be disabled. They now use CanMinimize:
- custom_frame_view_ash
- hwnd_message_handler
All classes that implement CanMaximize now implement
CanMinimize in the same way, so there should no change in
behavior.
BUG=328241,341010
Review URL: https://codereview.chromium.org/583603004
Cr-Commit-Position: refs/heads/master@{#296381}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Test APK in addition to instrumentation tests contains launching activity and foreground service what let duplicate ChromeShell debug socket to a differently named socket. If a device with running ChromeShell and DevToolsBridgeTest's service connect to a Desktop chrome (with USB debugging enabled) then chrome://inspect#devices will show 2 ChromeChell items. Both are debuggable (https://codereview.chromium.org/521573002/ should be applied to Desktop chrome to make the process less surprising). It's suitable for manual tests.
This CL doesn't care of a few known issues:
1. Data channel buffer overflow. Data channel automatically closed in this case.
2. Uncontrolled number of pending sockets. DevTools throttles opening sockets and and keeps sockets that exceeds a threshold hanging.
3. Uncontrolled number of threads (it's actually not a problem if #2 solved, thanks to DevTools throttling).
To keep this CL reasonably simple this issues will be addressed later.
TEST=org.chromium.components.devtools_bridge.SocketTunnelServerTest, see description fro manual testing.
BUG=383418
Committed: https://crrev.com/15044609893faf9b2c2a533aa9a136f288ea20a1
Cr-Commit-Position: refs/heads/master@{#296360}
Review URL: https://codereview.chromium.org/517233002
Cr-Commit-Position: refs/heads/master@{#296377}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Following up on http://crbug.com/338289#c13, this CL removes the kill switch for PSL matching. Apart from the linked check with our PM, the consideration is that PSL matching has been in stable for a couple of releases now on all platforms (except for KDE, see below), and the bugs seen so far were of low severity, exposing problems in password manager rather than with PSL matching itself.
It also replaces a couple of callsites of IsMatchingEnabled with ShouldPSLDomainMatchingApply -- the former did not take sites with a unified login origin into consideration, and on the affected callsites it seemed like it should.
One caveat: PSL matching has not been added to KDE on Linux yet (it's planned, just not a top priority). Given that PSL is live on other Linux environments, and all other platforms, the concerns about needing the kill switch for KDE are low.
As a follow-up to this CL, the (never submitted) internal CL with the corresponding Finch config will be closed.
BUG=338289
Review URL: https://codereview.chromium.org/508143002
Cr-Commit-Position: refs/heads/master@{#296371}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/517233002/)
Reason for revert:
Fails to compile on Android: https://build.chromium.org/p/chromium.linux/builders/Android%20Arm64%20Builder%20(dbg)/builds/2163
devtools_bridge/android/javatests/src/org/chromium/components/devtools_bridge/SocketTunnelServerTest.java:291: cannot find symbol
symbol: method sleep()
sleep();
^
Original issue's description:
> Implementation of DevTools socket tunneling service for remote debugging.
>
> Test APK in addition to instrumentation tests contains launching activity and foreground service what let duplicate ChromeShell debug socket to a differently named socket. If a device with running ChromeShell and DevToolsBridgeTest's service connect to a Desktop chrome (with USB debugging enabled) then chrome://inspect#devices will show 2 ChromeChell items. Both are debuggable (https://codereview.chromium.org/521573002/ should be applied to Desktop chrome to make the process less surprising). It's suitable for manual tests.
>
> This CL doesn't care of a few known issues:
> 1. Data channel buffer overflow. Data channel automatically closed in this case.
> 2. Uncontrolled number of pending sockets. DevTools throttles opening sockets and and keeps sockets that exceeds a threshold hanging.
> 3. Uncontrolled number of threads (it's actually not a problem if #2 solved, thanks to DevTools throttling).
>
> To keep this CL reasonably simple this issues will be addressed later.
>
> TEST=org.chromium.components.devtools_bridge.SocketTunnelServerTest, see description fro manual testing.
> BUG=383418
>
> Committed: https://crrev.com/15044609893faf9b2c2a533aa9a136f288ea20a1
> Cr-Commit-Position: refs/heads/master@{#296360}
TBR=mnaganov@chromium.org,tedchoc@chromium.org,erikwright@chromium.org,serya@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=383418
Review URL: https://codereview.chromium.org/597063002
Cr-Commit-Position: refs/heads/master@{#296361}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Test APK in addition to instrumentation tests contains launching activity and foreground service what let duplicate ChromeShell debug socket to a differently named socket. If a device with running ChromeShell and DevToolsBridgeTest's service connect to a Desktop chrome (with USB debugging enabled) then chrome://inspect#devices will show 2 ChromeChell items. Both are debuggable (https://codereview.chromium.org/521573002/ should be applied to Desktop chrome to make the process less surprising). It's suitable for manual tests.
This CL doesn't care of a few known issues:
1. Data channel buffer overflow. Data channel automatically closed in this case.
2. Uncontrolled number of pending sockets. DevTools throttles opening sockets and and keeps sockets that exceeds a threshold hanging.
3. Uncontrolled number of threads (it's actually not a problem if #2 solved, thanks to DevTools throttling).
To keep this CL reasonably simple this issues will be addressed later.
TEST=org.chromium.components.devtools_bridge.SocketTunnelServerTest, see description fro manual testing.
BUG=383418
Review URL: https://codereview.chromium.org/517233002
Cr-Commit-Position: refs/heads/master@{#296360}
|
|
|
|
|
|
|
|
|
|
|
|
| |
the browser side.
BUG=407376
TEST=no change
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/589403004
Cr-Commit-Position: refs/heads/master@{#296327}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit ba1e1e3f7ea6edae987e6cfa92086807bf74260b.
<jiayl_> It broken component_unittests: http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20(1)/builds/49443/steps/components_unittests/logs/RecordMissingViaHeaderBytes
BUG=
TBR=jiayl@chromium.org
Review URL: https://codereview.chromium.org/596163002
Cr-Commit-Position: refs/heads/master@{#296317}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the new name. It has the same meaning but additionally with a "you can use the headers" permission.
Rename direct_dependent_configs to public_configs. This is the new name with identical meaning.
TBR=jamesr
Review URL: https://codereview.chromium.org/595073002
Cr-Commit-Position: refs/heads/master@{#296302}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Keep track of the situations when Chrome expects the data reduction
proxy via header to be present in a response, but the data reduction
proxy via header is missing.
BUG=412888
Review URL: https://codereview.chromium.org/577343002
Cr-Commit-Position: refs/heads/master@{#296297}
|
|
|
|
|
|
|
|
|
| |
BUG=none
TBR=pkasting@chromium.org
Review URL: https://codereview.chromium.org/597433002
Cr-Commit-Position: refs/heads/master@{#296266}
|
|
|
|
|
|
|
|
|
|
| |
Files moved and includes adjusted.
BUG=387072
Review URL: https://codereview.chromium.org/591443002
Cr-Commit-Position: refs/heads/master@{#296263}
|
|
|
|
|
|
|
|
| |
BUG=416948
Review URL: https://codereview.chromium.org/600583002
Cr-Commit-Position: refs/heads/master@{#296262}
|
|
|
|
|
|
|
|
|
|
| |
Use the ChromeNetworkDelegate::OnProxyFallback() callback to keep track of number of times the data reduction proxies fall back because of network errors. Display a warning to users if there are network errors when connecting to the proxy and no successful requests through the proxy.
BUG=401244
Review URL: https://codereview.chromium.org/568893002
Cr-Commit-Position: refs/heads/master@{#296244}
|
|
|
|
|
|
|
|
|
| |
BUG=414363
R=jln@chromium.org
Review URL: https://codereview.chromium.org/588143007
Cr-Commit-Position: refs/heads/master@{#296212}
|
|
|
|
|
|
|
|
|
|
|
| |
This CL adds two initial tests that use mock URLRequestJob to
intercept URL requests.
BUG=415781
Review URL: https://codereview.chromium.org/558333007
Cr-Commit-Position: refs/heads/master@{#296201}
|
|
|
|
|
|
|
|
|
| |
BUG=0
NOTRY=true
Review URL: https://codereview.chromium.org/589353002
Cr-Commit-Position: refs/heads/master@{#296196}
|
|
|
|
|
|
|
|
|
| |
R=isherman@chromium.org
BUG=84200
Review URL: https://codereview.chromium.org/442403002
Cr-Commit-Position: refs/heads/master@{#296162}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Propagate the user's supervised state to the ash system
so that it becomes available for views
https://chromereviews.googleplex.com/79527013/
BUG=394417
Committed: https://crrev.com/8616ac4cf7bd77f53dc001eb9d8d61c6c975eaaa
Cr-Commit-Position: refs/heads/master@{#294821}
Review URL: https://codereview.chromium.org/561713002
Cr-Commit-Position: refs/heads/master@{#296160}
|
|
|
|
|
|
|
|
|
|
| |
Fix some style issues regarding order of includes and forward declararions.
BUG=416623
Review URL: https://codereview.chromium.org/590333002
Cr-Commit-Position: refs/heads/master@{#296139}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Remove all #if defined guards from data reduction proxy functionality.
* Hard code proxy urls in params rather than providing them in gyp files.
* Add DataCompressionProxyRollout field trial to all platforms.
* Pass in bypass duration into UpdateRetryInfoOnFallback rather than computing
it based on SPDY_PROXY_AUTH_ORIGIN. Remove unused UMAs.
BUG=384394,384397
Review URL: https://codereview.chromium.org/382313003
Cr-Commit-Position: refs/heads/master@{#296122}
|