- 24 Jun, 2020 40 commits
-
-
Fan Yang authored
https://quiche.googlesource.com/quiche.git/+log/6bb377091bbe..7a06f9b64641 $ git log 6bb377091..7a06f9b64 --date=short --no-merges --format='%ad %ae %s' 2020-06-24 fayang In quic, update packet_size_ when first frame gets queued. protected by gfe2_reloadable_flag_quic_update_packet_size. 2020-06-24 fayang Deprecate gfe2_reloadable_flag_quic_extend_idle_time_on_decryptable_packets. 2020-06-24 fayang Deprecate gfe2_reloadable_flag_quic_advance_ack_timeout_update. 2020-06-23 renjietang Add an option to disable 0-RTT in QUIC crypto. 2020-06-23 haoyuewang Internal QUICHE change Created with: roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src R=dschinazi@chromium.org Change-Id: I0be5570c0ca8f04db63cb99a66bfe289772baf1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2265134 Auto-Submit: Fan Yang <fayang@chromium.org> Reviewed-by:
David Schinazi <dschinazi@chromium.org> Commit-Queue: David Schinazi <dschinazi@chromium.org> Cr-Commit-Position: refs/heads/master@{#782091}
-
Dana Fried authored
Behind a flag (nothing happens if flag is not set). This is part 1. It introduces the flag which enables the badging, as well as the ability for menu items to be badged. However, the pipeline to hook up badging is not included. Follow-up CLs will include: 1. hook-up from SimpleMenuModel 2. specific items to be badged for release See: https: //docs.google.com/document/d/1vczPpUCnYOKvSQtSWMYX8K1mK-vhrjwsAyRKFv22eYk Change-Id: If8900433015e71802b5deb0e32bebb9477ea8d62 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208794Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#782090}
-
Shengfa Lin authored
Implemented according to https://w3c.github.io/webdriver/#print. 1. Add endpoint support for print in ChromeDriver 2. Parse parameters according to WebDriver spec 3. Convert the parameters to print parameters for DevTools Protocol printToPDF 4. Python tests for verifying headless print generate pdf files 5. Unit tests for verifying parameters are parsed correctly 6. Update ChromeDriver status document Bug: chromedriver:3481 Change-Id: I4745edca84af02faaa95cc271a32cf7274f7daac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209670 Commit-Queue: Shengfa Lin <shengfa@google.com> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#782089}
-
Oystein Eftevaag authored
R=nyquist@chromium.org,ssid@chromium.org Bug: 1098119 Change-Id: I12279236117ee69bc43110f1b63262afec629a69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264460Reviewed-by:
ssid <ssid@chromium.org> Commit-Queue: oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#782088}
-
Scott Violet authored
I'm only adding the thing I need, which is whether there is at least one browser in a resumed state, but we can add other calls to this as it makes sense. BUG=1025626 Change-Id: I7fd53fb97bc299d85d0171494888b4a6590b96ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261104Reviewed-by:
Evan Stade <estade@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#782087}
-
Amruth Raj authored
During pointer lock, a kRelativeMotionEvent is sent to indicate a recenter when the mouse is nearing the screen boundary. This test verifies that these events are not coalesced with other user generated events. Bug=1090617 Change-Id: Ib62ccf2531f0c3f45e92f5baef7fdc420af4a21e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2260292Reviewed-by:
Mustaq Ahmed <mustaq@chromium.org> Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Reviewed-by:
Vincent Scheib <scheib@chromium.org> Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Cr-Commit-Position: refs/heads/master@{#782086}
-
Xinghui Lu authored
This CL is for refactoring. There is no functional change. Move part of SendRequest and OnURLLoaderComplete into the base class. This can maximize the code that can be reused by the enterprise URL lookup service. Add two virtual functions GetTrafficAnnotationTag and GetRealTimeLookupUrlPrefix that are configurable by the child classes. Bug: 1085261 Change-Id: Ie895ffceacc15b34653644091763744ca970150d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247402 Commit-Queue: Xinghui Lu <xinghuilu@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Cr-Commit-Position: refs/heads/master@{#782085}
-
Joey Arhar authored
When a form submission is scheduled, this patch will make it so any attempt to navigate the target frame which was already sent to the browser process will be canceled in order to make the form submission take precedence over it. This is the same behavior that would occur if we had synchronous form submissions - a synchronous form submission would cancel any navigations already sent to the browser process. Bug: 1091583 Change-Id: I6c8421922960e3209fcc21a7e4582e0fbf7d4bb4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254443Reviewed-by:
Nate Chapin <japhet@chromium.org> Commit-Queue: Joey Arhar <jarhar@chromium.org> Cr-Commit-Position: refs/heads/master@{#782084}
-
Ted Choc authored
Replaces Criteria.equals usage as part of the migration. See https://groups.google.com/a/chromium.org/g/java/c/ZSj5TANUy8Q/m/gFL8BCVmBgAJ for more details. BUG=1071247 Change-Id: Id02d0a737b92533ba8abb462a48a20e2c685e58b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264023Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#782083}
-
Yoichi Osato authored
This feature counts protocol distribution to upload stream. This is recorded per fetch upload attempt regardless of its result. Bug: 688906 Change-Id: I3bba246fe0c41ee987a6d2ae52974fd1ac08b659 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2255925Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Yoichi Osato <yoichio@chromium.org> Cr-Commit-Position: refs/heads/master@{#782082}
-
Christopher Cameron authored
On macOS, most frames should be submitted by CommitOverlayPlanes. Also remove the concept of damage, because on macOS it is not meaningful. Bug: 894929 Change-Id: Idf0d393fd94fbc43f80a1d729bccf8f587431a86 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264935Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#782081}
-
Li Lin authored
There is a issue with text classifier that some capitalized words are not annotated properly. Convert the text to lower case for now. Bug: b/159756246 Test: manual test Change-Id: I66d355f503890a2b1d3c183853210fb1fd4aa8a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261818Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Yue Li <updowndota@chromium.org> Commit-Queue: Li Lin <llin@chromium.org> Cr-Commit-Position: refs/heads/master@{#782080}
-
Patrick Noland authored
Bug: 1098986 Change-Id: I00ae9f0a173b17e2b7bde98982ab4eb1925fcbe5 Tbr: michaelbai@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264603Reviewed-by:
Patrick Noland <pnoland@chromium.org> Commit-Queue: Patrick Noland <pnoland@chromium.org> Cr-Commit-Position: refs/heads/master@{#782079}
-
Chrome Cunningham authored
This reverts commit 30e7e503. Reason for revert: Broke https://ci.chromium.org/p/chrome/builders/ci/win64-chrome/3694 Original change's description: > [Kaleidoscope] Add native parts to OSS repo > > Add the Kaleidoscope native parts to the OSS repo. > > BUG=1044741 > > Change-Id: I8562c8e35db2d1fa0ef8ce140ddbbc57ca7a9991 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2255563 > Commit-Queue: Becca Hughes <beccahughes@chromium.org> > Reviewed-by: Samuel Huang <huangs@chromium.org> > Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Tommy Steimel <steimel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#782073} TBR=ellyjones@chromium.org,dcheng@chromium.org,huangs@chromium.org,beccahughes@chromium.org,steimel@chromium.org Change-Id: I534b8281aabb88bcf9af7f56881e297eaa1efb98 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1044741, 1099007 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264589Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#782078}
-
Elly Fong-Jones authored
This test uses an uninitialized variable to keep track of method calls. Bug: None Change-Id: Icf08c41265986052966420dd4b274e47c83057c1 Tbr: khorimoto@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264463Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#782077}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/43a6e176a60f..2639a144ff41 2020-06-23 kjlubick@google.com [demos] Add directory from which to serve demos.skia.org 2020-06-23 adlai@google.com Ignore calls to freeGpuResources on abandoned contexts If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC lovisolo@google.com,egdaniel@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: lovisolo@google.com,egdaniel@google.com Change-Id: I3618fe822937a3acc912f7b947e4d7b04f8b5fb6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2265157Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#782076}
-
Matt Menard authored
When app installation info and activity is being collected from the user, we need to inform them on the chrome://management page. Bug: 1058453 Change-Id: I6b771870e68b55afe01cce9666a3625472ddcad0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170171 Commit-Queue: Matt Menard <mattme@google.com> Reviewed-by:
Thiemo Nagel <tnagel@chromium.org> Reviewed-by:
Anqing Zhao <anqing@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Roman Aleksandrov <raleksandrov@google.com> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#782075}
-
Charlene Yan authored
Bug: 1018320 Change-Id: I09e27413fc0597d6e4676d31b53dc62920f824b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261407 Commit-Queue: Charlene Yan <cyan@chromium.org> Reviewed-by:
Connie Wan <connily@chromium.org> Cr-Commit-Position: refs/heads/master@{#782074}
-
Becca Hughes authored
Add the Kaleidoscope native parts to the OSS repo. BUG=1044741 Change-Id: I8562c8e35db2d1fa0ef8ce140ddbbc57ca7a9991 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2255563 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Samuel Huang <huangs@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#782073}
-
chrome://net-internalsSteven Bennetts authored
This moves the network debugging level UI from chrome://net-internals to chrome://network. This removes the last of the chromeos specific code from chrome://net-internals. We should keep the links for discovery for at least a version or two, then the chromeos tab can be removed entirely from that UI. Bug: 921726 Change-Id: I589a73a227b24205dde431cb4d329d200a1b6787 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261409 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#782072}
-
David Dorwin authored
These additional services are technically required always or when presenting using Scenic. Change-Id: If526d285e8d86d95878736562f1b5fd418dd361c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261167Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Wez <wez@chromium.org> Auto-Submit: David Dorwin <ddorwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#782071}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/bdfbdd275b29..87c8b9163983 2020-06-24 iannucci@chromium.org [cipd] Update to version with aix-ppc64 binary. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: chromium:1098127 Tbr: agable@chromium.org Change-Id: I2f941cb9c58cf9e0f7f77dd3c2baf572151859d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264843 Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#782070}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-linux-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I1fbc26765c3b2b74abc98bc1bf81e3515998ccf8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264888Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#782069}
-
Sunny Sachanandani authored
This reverts commit fc04607a. Reason for revert: Likely caused several regressions Original change's description: > cc: Subtract estimated display draw time from begin frame deadline > > Display draw time was being subtraced from display scheduler's deadline, > but a similar adjustment was missing from cc scheduler's deadline. To > fix this inconsistency, this CL applies the deadline adjustment at a > single place, the begin frame source so that there's no mismatch. > > This makes cc scheduler have a later deadline and can cause it to wait > longer for the main thread. Recent experimentation has shown improved > latency in the absence of a scheduler deadline. It's hoped that some of > the improvement can be achieved by just fixing the deadline. > > Bug: 1042584 > Change-Id: Ia948b64c1dbba5239d1b97062224e73487e35fba > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2045210 > Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> > Auto-Submit: Sunny Sachanandani <sunnyps@chromium.org> > Reviewed-by: Khushal <khushalsagar@chromium.org> > Cr-Commit-Position: refs/heads/master@{#781159} TBR=sadrul@chromium.org,sunnyps@chromium.org,khushalsagar@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1042584, 1098444, 1098229 Change-Id: I3dcad61af5f1ef6d8a38a3290c6c2e04c503dbb3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264629Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Cr-Commit-Position: refs/heads/master@{#782068}
-
Elly Fong-Jones authored
Bug: None Change-Id: Iade75c41046ef96acb32c9f074a0e632fbccd488 Tbr: chcunningham@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264863Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#782067}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/29d59a140248..75fd127640bd 2020-06-24 sprang@webrtc.org Allows FEC generation after pacer step. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: Ia480a8d36ee44b65a2adaccf3d542f8f3729b6b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264916Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#782066}
-
Dominic Farolino authored
R=fs@opera.com Bug: 993273 Change-Id: If692195679f4b10f32449c04d34196260c22c55c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2256853 Commit-Queue: Dominic Farolino <dom@chromium.org> Reviewed-by:
Fredrik Söderquist <fs@opera.com> Reviewed-by:
Scott Little <sclittle@chromium.org> Cr-Commit-Position: refs/heads/master@{#782065}
-
evliu authored
This CL updates the DefaultRendererFactory to use the "OnReady" speech recognition callback to determine if and when speech recognition is available rather than continuously making mojo calls to the render thread. Bug: 1098552 Change-Id: Iabf19155f350bb6bd5aac8a6b157decb457d110e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261740Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Evan Liu <evliu@google.com> Cr-Commit-Position: refs/heads/master@{#782064}
-
Leonard Grey authored
Bug: 1062493 Change-Id: I9b157a9c7e1f03563c090e4b629babc1d682e83c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250347Reviewed-by:
Taylor Bergquist <tbergquist@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#782063}
-
Robert Ogden authored
If the origin probe fails, then no subresources should be loaded from the isolated cache. Bug: 1082505 Change-Id: I8f3411ad9e0fcbc2376cf43317fe8a6cfb125b32 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261411Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#782062}
-
Victor Fei authored
CL:2242046 introduced ui::UiaRegistrarWin for registering custom UIA property and event. This change moves the custom UIA event used for signaling test end to ui::UiaRegistrarWin. There is no functionality change. AX-Relnotes:n/a Bug: 847971 Change-Id: Id1b8852d788d5533e14bc63588fd26669ae78eb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261876Reviewed-by:
Ian Prest <iapres@microsoft.com> Commit-Queue: Victor Fei <vicfei@microsoft.com> Cr-Commit-Position: refs/heads/master@{#782061}
-
Meilin Wang authored
This is a follow-up CL of the original: https://chromium-review.googlesource.com/c/chromium/src/+/2253363/ Upon shown, wake lock will only be requested when the battery is charging, and also be release when the charger is disconnected later. Bug: b/159382866 Test: covered by unittests. Change-Id: I62d94d645a67c07c2310b6d1a7ee8220a24c77f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2259773 Commit-Queue: Meilin Wang <meilinw@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#782060}
-
Ian Kilpatrick authored
Previously if we hit an anonymous flex-item we wouldn't update the hit-test result with the flexbox itself. This updates the hit-test with the flexbox if no node is set yet. Bug: 1097654 Change-Id: I4b9d1754061aae14dd8fa6a9f33d8d065c838db6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264017Reviewed-by:
David Grogan <dgrogan@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#782059}
-
Jason Chase authored
Minor refactor to have all AddToken* methods have |token(s)| as the first parameter. Split off from http://crrev.com/c/2173473. Bug: 1073920 Change-Id: Icbb2699849e74743e7584f981aa831e407134986 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261634Reviewed-by:
Ian Clelland <iclelland@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Jason Chase <chasej@chromium.org> Cr-Commit-Position: refs/heads/master@{#782058}
-
Darwin Huang authored
Change-Id: Ie2125b4bc4cf46db8f0e0f73f80dd30224ae049a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254352Reviewed-by:
Darwin Huang <huangdarwin@chromium.org> Reviewed-by:
Jarryd Goodman <jarrydg@chromium.org> Auto-Submit: Darwin Huang <huangdarwin@chromium.org> Commit-Queue: Darwin Huang <huangdarwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#782057}
-
Patrick Noland authored
This reverts commit ea45d4f3. Reason for revert: breaks compile (https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8876580958035597312/+/steps/compile/0/stdout) Original change's description: > Add supports to Lens Shopping experiments for google.com > - enable lens shopping experiments variants for google.com > - add unit tests to LensUtilsTest for shopping intent > > Bug: 158099013 > Change-Id: Ib605afcfe069320a0b2a52d4ea045759a39e3592 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261093 > Commit-Queue: Yu Su <yusuyoutube@google.com> > Reviewed-by: Theresa <twellington@chromium.org> > Cr-Commit-Position: refs/heads/master@{#782024} TBR=twellington@chromium.org,benwgold@google.com,yusuyoutube@google.com Change-Id: I8c607c310d540b6b491ff07c20147f9ba1ecdb7a No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 158099013 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264505Reviewed-by:
Patrick Noland <pnoland@chromium.org> Commit-Queue: Patrick Noland <pnoland@chromium.org> Cr-Commit-Position: refs/heads/master@{#782056}
-
Tommy C. Li authored
Before this CL, Entity suggestions from ZeroSuggest would have their text color messed when switching back and forth between dark mode. This CL fixes it, and adds a TODO explaining some technical debt we have. This CL also makes a similar fix to Answers, which doesn't seem to happen in practice, since Answers don't show up in ZeroSuggest, but this is the more correct thing to do. Bug: 1095205 Change-Id: I2364bb578bbfe47705c594bfe423da694df6aad2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2264853 Commit-Queue: Tommy Li <tommycli@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#782055}
-
Evan Stade authored
This adds the glue code for MediaSession to work in WebLayer. A new Service class is required in the client library to display the notification. This service proxies lifetime events to WebLayer implementation, which then glues together componentized code. Of note: 1. notifications displayed with Service#startForeground cannot have a tag, so the ID must be unique across all application notifications. There's a default ID for the MediaSession notification, but an API is also added to WebLayer to allow setting a different ID, in case this one conflicts. 2. favicons are not supported (normally they'd be used as fallbacks for iconography/images not provided by the system) 3. Extra handling is required for the action button icons to work, see WebLayerNotificationBuilder. On L, we fall back to system icons, which look bad because of their low resolution, but are at least functional. https://googlechrome.github.io/samples/media-session/ Test: automated tests TODO; manual testing on Bug: 1066263 Change-Id: I5e2ae13a07315e73aae474251cb5c0424804a427 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2220551Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#782054}
-
Ng Zhi An authored
Tested with: python build.py --without-android --without-fuchsia \ --run-tests --skip-checkout With python == python3.7 (inside a pipenv). - get maxint from sys.maxsize from Py3 (sys.maxint is removed) - use StringIO from io - use .values() instead of .itervalues() - tweak lambda used in sort (became syntax error in Py3) - check_output with universal newlines, will open stream as text Bug: chromium:941669 Change-Id: I80ca9ae6d2f6d712c6e3cac5343627512ce84b86 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261715Reviewed-by:
Anton Bikineev <bikineev@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#782053}
-
Alison Maher authored
http://crrev.com/c/2118693 caused the invert bubble to start showing up on Mac in increased contrast mode when a new incognito window was opened. http://crrev.com/c/2247407 addressed this temporarily by disabling the bubble for incognito windows. This change is a follow-up to http://crrev.com/c/2247407 as a more permanent solution to the original problem. The invert bubble should show up in the High Contrast dark theme on Windows. However, GetHighContrastColorScheme() was calling into UsesHighContrastColors() to detect this. The problem with this is that on Mac, UsesHighContrastColors() has a different meaning than on Windows. To fix this, make GetHighContrastColorScheme() a virtual function and only return the High Contrast theme on Windows. Bug: 1095855 Change-Id: I24a7ed2dd5b9194a795d1560ba1e5f4be85afd98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250224Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Alison Maher <almaher@microsoft.com> Cr-Commit-Position: refs/heads/master@{#782052}
-