- 05 May, 2018 40 commits
-
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/39e3bc6ed30a..3059fd7dba63 $ git log 39e3bc6ed..3059fd7db --date=short --no-merges --format='%ad %ae %s' 2018-05-04 harringtond Improve replay determinism Created with: roll-dep src/third_party/catapult The AutoRoll server is located here: https://catapult-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=sullivan@chromium.org Change-Id: Ib90e53284fce7914887b59a19955de5046f06e5b Reviewed-on: https://chromium-review.googlesource.com/1045616Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#556312}
-
dpapad authored
Bug: 832184 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Idf252a9e10a524e898e2de8fc540d890285ff01e Reviewed-on: https://chromium-review.googlesource.com/1038737Reviewed-by:
Scott Chen <scottchen@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#556311}
-
sangwoo.ko authored
As we can convert plain strings into query url, We shouldn't block non-urls. Bug: 838794 Change-Id: Iaceaf0627a95c11382a4dee72d62a7af2a4e78bd Reviewed-on: https://chromium-review.googlesource.com/1041367 Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#556310}
-
Michael Giuffrida authored
Dumps some info from an AssistRanker ExamplePreprocessorConfig protobuf. Useful when developing and testing integration of an ML model. Change-Id: I49a8865b6a71234e4adfa6ca88dbde2905487e9d Reviewed-on: https://chromium-review.googlesource.com/1027416Reviewed-by:
Roger McFarlane <rogerm@chromium.org> Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Cr-Commit-Position: refs/heads/master@{#556309}
-
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/ea240053f956..5e5f2d60353b $ git log ea240053f..5e5f2d603 --date=short --no-merges --format='%ad %ae %s' 2018-05-04 tandrii bot_update: attempt to disable GC completely on main repo. Created with: roll-dep src/third_party/depot_tools BUG=chromium:822437 The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Change-Id: I9587aced9d864d929fca9566a0a6df34db52fa09 Reviewed-on: https://chromium-review.googlesource.com/1045888 Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#556308}
-
dpapad authored
This is in preparation of updating cr-toggle's style for MD refresh, as well as removing paper-toggle-button from third_party/polymer. Bug: 768073 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ic26a8cd965319e5be453398e68eb49fec556910a Reviewed-on: https://chromium-review.googlesource.com/1043448Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#556307}
-
Gabriel Charette authored
This reverts commit 5b0cb536. Reason for revert: find it identified flakes, I'll investigate on Monday Original change's description: > [TaskScheduler] Label SchedulerWorker threads stacks with an identifying frame > > I intentionally didn't add "Foreground" to non "Background" workers as > they may be used for background tasks in some configurations and I don't > want it to be confusing for developers. > > R=fdoray@chromium.org > > Bug: 839525 > Change-Id: I8c1e928914aa6d2e35fedc1db4e8a639d39116b2 > Reviewed-on: https://chromium-review.googlesource.com/1044501 > Commit-Queue: Gabriel Charette <gab@chromium.org> > Reviewed-by: François Doray <fdoray@chromium.org> > Cr-Commit-Position: refs/heads/master@{#556169} TBR=gab@chromium.org,fdoray@chromium.org Change-Id: I660f43ba88eade2ac3e8fedf4d60495eab708054 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 839525 Reviewed-on: https://chromium-review.googlesource.com/1045747Reviewed-by:
Gabriel Charette <gab@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#556306}
-
Findit authored
This reverts commit c5f1009c. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 556292 as the culprit for failures in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2M1ZjEwMDljZDQyZDI0ZDc3ZmI4MTEwNDI4MTVjZjVkOTRjNzcwZmIM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Win%20Builder%20%28dbg%29/130960 Sample Failed Step: compile Original change's description: > Reland #2 "Move manifest.mojom and content::Manifest to blink" > > This is a reland of c07726e1 > > Original change's description: > > Move manifest.mojom and content::Manifest to blink > > > > ... as a part of Onion Soup project. > > > > This also required moving WebDisplayMode from public/platform > > to public/common since it's referenced both in manifest itself > > and, more generally, browser code. > > > > mojom::blink::Manifest is typemapped to blink::Manifest, which uses > > stl types and should be used by blink's embedder, not blink > > implementation. > > > > Bug: 704441 > > Change-Id: I94d07bb028278f17d215f64c14552793d5f916ae > > Reviewed-on: https://chromium-review.googlesource.com/1006188 > > Commit-Queue: Dmitry Gozman <dgozman@chromium.org> > > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > > Reviewed-by: Reilly Grant <reillyg@chromium.org> > > Reviewed-by: Kentaro Hara <haraken@chromium.org> > > Reviewed-by: Mathieu Perreault <mathp@chromium.org> > > Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> > > Reviewed-by: Mikel Astiz <mastiz@chromium.org> > > Reviewed-by: Scott Violet <sky@chromium.org> > > Reviewed-by: David Trainor <dtrainor@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#554953} > > TBR=dgozman > NOPRESUBMIT=true > > Bug: 704441 > Change-Id: I3f2138335ea98710d2e8f6f06ab9fe9f3d561b96 > Reviewed-on: https://chromium-review.googlesource.com/1040686 > Commit-Queue: Dmitry Gozman <dgozman@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> > Reviewed-by: Dmitry Gozman <dgozman@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Cr-Commit-Position: refs/heads/master@{#556292} Change-Id: I1e43dfd4225b5bf48768c4109ad25e54cf25463b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 704441 Reviewed-on: https://chromium-review.googlesource.com/1045846 Cr-Commit-Position: refs/heads/master@{#556305}
-
Matt Wolenetz authored
This is the result of examining and adjusting the scoping of occurrences of "USE_PROPRIETARY_CODECS" and, in gn files, "proprietary_codecs" to ensure that non-proprietary codecs (like flac and mp3) are included when not building with proprietary codecs. These results appear to only affect test targets and their coverage. Note, further scrubbing outside of //media still remains. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: I020594e6b3790e019f592eab76590104565afae7 Reviewed-on: https://chromium-review.googlesource.com/1045725Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#556304}
-
rbpotter authored
Migrate the AdvancedSettings1Option and AdvancedSettings2Options tests. Bug: 814860 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I9f70fd4bc556bfa7a208822ab0a981a1ac258bca Reviewed-on: https://chromium-review.googlesource.com/1040877Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#556303}
-
sangwoo.ko authored
Otherwise, arrow can stay forever. Bug: 838791 Change-Id: I2fe13be530a66b587511484ea3d093bc39ad4194 Reviewed-on: https://chromium-review.googlesource.com/1041366 Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#556302}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/2839e685..2369c0c1 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: I5dd8083ec89d1bd74e1bb7acef634eea2e9592d1 Reviewed-on: https://chromium-review.googlesource.com/1045675Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#556301}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: Ib61604b0d23d143b58788290519da3645b10d43b Reviewed-on: https://chromium-review.googlesource.com/1045793Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#556300}
-
Peng Huang authored
This is a reland of 66531351 Original change's description: > Turn on presentation feedback for Linux > > Bug: 776877 > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel > Change-Id: I9d9590a5174db94392a447ab280e9218b37c2958 > Reviewed-on: https://chromium-review.googlesource.com/1040725 > Reviewed-by: Antoine Labour <piman@chromium.org> > Commit-Queue: Peng Huang <penghuang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#555782} TBR=piman@chromium.org Bug: 776877 Change-Id: I7b8c4cdfd05b39d0c5bfa39c884775183564b35e Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Reviewed-on: https://chromium-review.googlesource.com/1045845 Commit-Queue: Peng Huang <penghuang@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#556299}
-
Hal Canary authored
BUG=chromium:481429 Change-Id: I601397c2f0c5b48224c68cb968cffa614c19a5bd Reviewed-on: https://chromium-review.googlesource.com/1021915Reviewed-by:
Florin Malita <fmalita@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#556298}
-
iOS Autoroller authored
TBR=smut Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I0a3ae1c5fcdc391c4062e2652a0a9f8f23166aae Reviewed-on: https://chromium-review.googlesource.com/1045788 Commit-Queue: iOS Autoroller <ios-autoroller@chromium.org> Reviewed-by:
iOS Autoroller <ios-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#556297}
-
dpapad authored
Bug: 839270 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I7af505de122b071846617c95dd20454214c49275 Reviewed-on: https://chromium-review.googlesource.com/1043268Reviewed-by:
Scott Chen <scottchen@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#556296}
-
dpapad authored
This is in preparation of updating cr-toggle's style for MD refresh, as well as removing paper-toggle-button from third_party/polymer. Bug: 768073 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ie9433c5789baeefdeed21e6d5f81fad80608e8d3 Reviewed-on: https://chromium-review.googlesource.com/1043049 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#556295}
-
Oystein Eftevaag authored
Also: * Fixed two issues in TraceEventDataSourceTests causing tests to fail in some conditions. * Added more types to the trace_analyzer_test to handle more trace event fields. * Added base/trace_event/OWNERS as owners of base/test/trace_* Bug: 839058 Change-Id: I93aa40ef185aa149cf240e333838ca6c6339a53d Reviewed-on: https://chromium-review.googlesource.com/1041155 Commit-Queue: oysteine <oysteine@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Primiano Tucci <primiano@chromium.org> Cr-Commit-Position: refs/heads/master@{#556294}
-
Scott Chen authored
When initializing, cr-checkbox unnecessarily set tabindex to 0 even if it was set to something else. This CL fixes it Bug: 839277 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I73d6e97ad7b6be8b7cd18759f1677f65acf96947 Reviewed-on: https://chromium-review.googlesource.com/1042864 Commit-Queue: Scott Chen <scottchen@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#556293}
-
Dmitry Gozman authored
This is a reland of c07726e1 Original change's description: > Move manifest.mojom and content::Manifest to blink > > ... as a part of Onion Soup project. > > This also required moving WebDisplayMode from public/platform > to public/common since it's referenced both in manifest itself > and, more generally, browser code. > > mojom::blink::Manifest is typemapped to blink::Manifest, which uses > stl types and should be used by blink's embedder, not blink > implementation. > > Bug: 704441 > Change-Id: I94d07bb028278f17d215f64c14552793d5f916ae > Reviewed-on: https://chromium-review.googlesource.com/1006188 > Commit-Queue: Dmitry Gozman <dgozman@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Reilly Grant <reillyg@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Reviewed-by: Mathieu Perreault <mathp@chromium.org> > Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> > Reviewed-by: Mikel Astiz <mastiz@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: David Trainor <dtrainor@chromium.org> > Cr-Commit-Position: refs/heads/master@{#554953} TBR=dgozman NOPRESUBMIT=true Bug: 704441 Change-Id: I3f2138335ea98710d2e8f6f06ab9fe9f3d561b96 Reviewed-on: https://chromium-review.googlesource.com/1040686 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#556292}
-
Vlad Tsyrklevich authored
After hitting unexpected cfi-icall failures yesterday (e.g. crbug.com/838785) I audited calls to dlsym() and candidates that I missed in uncommon code/build configurations. I chose to disable cfi-icall by attribute for functions that only store dynamically resolved pointers on the stack since they are not favorable candidates to be attacked. base: :GetFunctionPointerFromNativeLibrary() and found a couple Change-Id: I203ea1753b466575c59654d7c3d0fc90c69967a8 Reviewed-on: https://chromium-review.googlesource.com/1044502Reviewed-by:
Peter Collingbourne <pcc@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Peter Collingbourne <pcc@chromium.org> Cr-Commit-Position: refs/heads/master@{#556291}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/694ebb348a44..3d5b83bf9533 $ git log 694ebb348..3d5b83bf9 --date=short --no-merges --format='%ad %ae %s' 2018-05-04 liyuqian Restore old SkBitmapDeviceFilteredSurfaceProps 2018-05-04 swiftshader-skia-autoroll Roll third_party/externals/swiftshader/ e76163e3c..336cdd903 (1 commit) 2018-05-04 senorblanco Remove GR_TESSELLATOR_LEGACY_INVERSION_HANDLING (unused). Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=brianosman@chromium.org Change-Id: I6a7312dafb16b658f99c085892c323fa4210e77e Reviewed-on: https://chromium-review.googlesource.com/1045493Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#556290}
-
Erik Chen authored
This CL is a refactor with no intended behavior change. The logic was pulled out in a refactor [https://chromium-review.googlesource.com/780143] to support the experimental tab strip model. Since then, the experimental tab strip has been deleted. Pulling the logic back into TabStripModel is the first step to improving the performance of TabStripModel::InternalCloseTabs by reducing the number of unnecessary notifications sent to observers. Bug: 826287 Change-Id: I371bd964d8ea6338d50454a0de329a70e5090fae Reviewed-on: https://chromium-review.googlesource.com/1010642 Commit-Queue: Erik Chen <erikchen@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#556289}
-
danakj authored
Removes a piece of logic from WebLayer to help us no longer need the class to exist. We plumb along with paths that set the WebLayer as the contents layer on GraphicsLayer a bool that says if the creator of the layer wants to manage the contents opaque-ness of the WebLayer itself, or let the GraphicsLayer manage it. R=enne@chromium.org Bug: 838693 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ib0cf69441054dab8c031b4ac7f59d8306a0571c5 Reviewed-on: https://chromium-review.googlesource.com/1038565 Commit-Queue: danakj <danakj@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
Lucas Gadani <lfg@chromium.org> Reviewed-by:
Emircan Uysaler <emircan@chromium.org> Reviewed-by:
enne <enne@chromium.org> Cr-Commit-Position: refs/heads/master@{#556288}
-
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/142a92ce8b83..ea240053f956 $ git log 142a92ce8..ea240053f --date=short --no-merges --format='%ad %ae %s' 2018-05-03 ehmaldonado gclient: Update catapult and custom-tab-client origin URLs Created with: roll-dep src/third_party/depot_tools BUG=chromium:643346 The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Change-Id: Id2fd9f19897a69cab6d2cece4526c9f1aa126291 Reviewed-on: https://chromium-review.googlesource.com/1045182Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#556287}
-
Ryan Hamilton authored
Remove unused bytes_acked_since_queue_drained_ from QUIC's BBRSender. Merge internal change: 195185347 https://chromium-review.googlesource.com/c/chromium/src/+/1045098 Deprecate FLAGS_quic_reloadable_flag_quic_register_streams_early2. Merge internal change: 195167465 https://chromium-review.googlesource.com/c/chromium/src/+/1045133 Deprecate FLAGS_FLAGS_quic_reloadable_flag_quic_server_early_version_negotiation Merge internal change: 195167459 https://chromium-review.googlesource.com/c/chromium/src/+/1045132 Refactor validation of received QUIC versions so it can be used in TLS handshake as well Merge internal change: 195166917 https://chromium-review.googlesource.com/c/chromium/src/+/1045095 Refactor more of QuicTransportVersion to ParsedQuicVersion Merge internal change: 195163026 https://chromium-review.googlesource.com/c/chromium/src/+/1045094 Add a flag count for FLAGS_quic_reloadable_flag_quic_bbr_fix_probe_bw. Merge internal change: 195152515 https://chromium-review.googlesource.com/c/chromium/src/+/1045093 Deprecate FLAGS_quic_reloadable_flag_quic_better_crypto_retransmission Merge internal change: 195129678 https://chromium-review.googlesource.com/c/chromium/src/+/1045166 For udp proxying, munge L1=>L0 packets when needed. Protected by --FLAGS_quic_restart_flag_quic_enable_l1_munge. Merge internal change: 195118067 https://chromium-review.googlesource.com/c/chromium/src/+/1045127 Deprecate FLAGS_quic_reloadable_flag_quic_always_discard_packets_after_close. Merge internal change: 195099593 https://chromium-review.googlesource.com/c/chromium/src/+/1045109 Populate peer address and effective client address in stats for HANDSHAKE_FAILED. Protected by exsiting FLAGS_quic_reloadable_flag_quic_log_handshake_failed_in_tcs. Merge internal change: 195089255 https://chromium-review.googlesource.com/c/chromium/src/+/1045088 In QUIC BBR, when draining to the target CWND, use bytes_in_flight, not prior_in_flight. Protected by FLAGS_quic_reloadable_flag_quic_bbr_fix_probe_bw. Merge internal change: 194959773 https://chromium-review.googlesource.com/c/chromium/src/+/1045106 Add a QUIC connection option(NRTT) to cause the receiver of an initial_rtt in the handshake to ignore it. Protected by FLAGS_quic_reloadable_flag_quic_no_irtt. Merge internal change: 194927468 https://chromium-review.googlesource.com/c/chromium/src/+/1045069 Deprecate never verified FLAGS_quic_reloadable_flag_quic_bbr_add_tso_cwnd in the false position. Merge internal change: 194877002 https://chromium-review.googlesource.com/c/chromium/src/+/1044574 Deprecate max_aggregation_bytes_multiplier_ in QUIC BBR and the BBR1 and BBR2 connection options which set it. Merge internal change: 194853070 https://chromium-review.googlesource.com/c/chromium/src/+/1045045 Make all quic tests pass when quic_supports_tls_handshake is true Merge internal change: 194836449 https://chromium-review.googlesource.com/c/chromium/src/+/1044949 Add a connection option(1RTO) to only send one packet upon RTO instead of 2. Protected by FLAGS_quic_reloadable_flag_quic_one_rto. Merge internal change: 194835466 https://chromium-review.googlesource.com/c/chromium/src/+/1044965 Change-Id: I11212aa847e4fc357713e24d6b3e34411be7bd73 Reviewed-on: https://chromium-review.googlesource.com/1045173Reviewed-by:
Nick Harper <nharper@chromium.org> Commit-Queue: Ryan Hamilton <rch@chromium.org> Cr-Commit-Position: refs/heads/master@{#556286}
-
rbpotter authored
This prevents needing to have link-container implement SettingsSectionBehavior and will allow future experimental settings sections to be inside dom-ifs. Bug: 773928 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I7f7ba68318fa671e4303304da17601e60b90f7a8 Reviewed-on: https://chromium-review.googlesource.com/1045585Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#556285}
-
Steven Bennetts authored
This also converts everything to inlined constexpr. Bug: 798569 Change-Id: Ia5f026a2e30b8a590e761860d063d42cfdf54bbc Reviewed-on: https://chromium-review.googlesource.com/1043647Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#556284}
-
Yafei Duan authored
RecentTabsUIAdapter and RecentTabSuggestionProvider are deprecated, so removing them as well as the flag used for it. The furthest the feature got was Canary/Dev M57 and then got cancelled. Previous launch bug: https://crbug.com/659205 Bug: 659205 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I5403d6e5f642531b15a548499aae7b063d9406e1 Reviewed-on: https://chromium-review.googlesource.com/1040907Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Reviewed-by:
vitaliii <vitaliii@chromium.org> Commit-Queue: Yafei Duan <romax@chromium.org> Cr-Commit-Position: refs/heads/master@{#556283}
-
Kyle Horimoto authored
EasyUnlock tries to create Bluetooth connections at the lock screen, and when both Instant Tethering and EasyUnlock try to create a Bluetooth connection to the same device at the same time, they can cause instability. By eliminating the possibility of this conflict, we work around this issue. Note that a full-fledged fix for this problem is in the works; see https://crbug.com/752273. Bug: 839522 Change-Id: I468515579ca46cabf279e11531ca489a71eb3255 Reviewed-on: https://chromium-review.googlesource.com/1045254 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
James Hawkins <jhawkins@chromium.org> Cr-Commit-Position: refs/heads/master@{#556282}
-
Dale Curtis authored
Some of the fuzzing tests invoke errors at weird times, in the case of the linked bug: during seek, preventing the pipeline from reaching have enough again and thus hanging forever. There doesn't seem to be any reason not to include an error handler during all waits done by PipelineIntegrationTests, hanging should not be the behavior used by tests to indicate failure. BUG=834559 TEST=fuzzer test exits instead of hanging Change-Id: Ieccf1b4647ab8aa0f3e5ad22a3174c2f251468ec Reviewed-on: https://chromium-review.googlesource.com/1045562Reviewed-by:
Matthew Wolenetz <wolenetz@chromium.org> Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#556281}
-
Pedro Amaral authored
Bug: 814528 Change-Id: I1474a77352fd493eaf7635d0d76955cad9ad4608 Reviewed-on: https://chromium-review.googlesource.com/1045512Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Pedro Amaral <amaralp@chromium.org> Cr-Commit-Position: refs/heads/master@{#556280}
-
Qiang Xu authored
changes: Refactor LauncherContextMenu: don't make it a subclass of ui::SimpleMenuModel. Instead, exposing a GetMenuModel method to prevent 'calling API and returning self' problem. Bug: 803291 Test: covered by tests Change-Id: I42af308107814bc0fe7ba014b7671316acfbc8aa Reviewed-on: https://chromium-review.googlesource.com/1045097 Commit-Queue: Qiang Xu <warx@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#556279}
-
Alexander Alekseev authored
We no longer need to modify user type in user context, so this Cl makes it read-only. Bug: 837107 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I539b033a60eab82e6295fcfeb44fe16f5bc6b99c Reviewed-on: https://chromium-review.googlesource.com/1038666 Commit-Queue: Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Roger Tawa <rogerta@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Roman Sorokin <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#556278}
-
David Tseng authored
- window.open's second argument is the context; in this case, we don't have one. - Polymer recommends wrapping a paper-button with an anchor when treating the button like a link. - s/options_page/optionsPage Test: load an engine with a options page. Ensure it shows up and opens that page when clicked. Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I4210d6c306e2392841fdd9b9fb90ea1ab5c6ffb3 Reviewed-on: https://chromium-review.googlesource.com/1040679Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Reviewed-by:
Katie Dektar <katie@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#556277}
-
Pavel Feldman authored
This reverts commit 191b1ead. Reason for revert: We can't process protocol synchronously in the headless embedder. Original change's description: > DevTools: change Target.disposeBrowserContext to force-close WebContents > > This patch changes the behavior of Target.disposeBrowserContext method > to force-close all the belonging WebContents. > > R=dgozman > BUG=836272 > > Change-Id: Iec3da859363120a22c2648b31102457aee20e6cb > Reviewed-on: https://chromium-review.googlesource.com/1031774 > Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Dmitry Gozman <dgozman@chromium.org> > Reviewed-by: Alex Clarke <alexclarke@chromium.org> > Reviewed-by: Andrey Kosyakov <caseq@chromium.org> > Cr-Commit-Position: refs/heads/master@{#555640} TBR=dgozman@chromium.org,sky@chromium.org,lushnikov@chromium.org,caseq@chromium.org,alexclarke@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 836272 Change-Id: Ib5d06ad2ce6816a82c0819c693b2eb2819d3a502 Reviewed-on: https://chromium-review.googlesource.com/1045525Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#556276}
-
Erik Staab authored
Bug: 731967 Change-Id: I94e8f9d5342293f8ac9e6e2af4ee0c33f082282c Reviewed-on: https://chromium-review.googlesource.com/1045445Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Erik Staab <estaab@chromium.org> Cr-Commit-Position: refs/heads/master@{#556275}
-
Paul Jensen authored
Maven modules will be generated with a new script as part of http://crrev.com/c/1044349 instead of via GN build targets. Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester Change-Id: I02353acf51951b4d57fba2b846fca1fff8dfcf07 Reviewed-on: https://chromium-review.googlesource.com/1044357Reviewed-by:
Misha Efimov <mef@chromium.org> Commit-Queue: Paul Jensen <pauljensen@chromium.org> Cr-Commit-Position: refs/heads/master@{#556274}
-
Edward Jung authored
Bug: 834899 Change-Id: I9bcbcda4b121d8c732f808e0745b8b33a1da647f Reviewed-on: https://chromium-review.googlesource.com/1043872Reviewed-by:
Adrienne Porter Felt <felt@chromium.org> Commit-Queue: Edward Jung <edwardjung@chromium.org> Cr-Commit-Position: refs/heads/master@{#556273}
-