- 10 Sep, 2019 40 commits
-
-
Saman Sami authored
vulkan_content_browsertests does not need an exception to not run on Linux FYI GPU TSAN Release. It's already excluded. Bug: 940723 Change-Id: I58ff26b57f5d5fa60ca54627b320149fe4de3d89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795983Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Saman Sami <samans@chromium.org> Cr-Commit-Position: refs/heads/master@{#695199}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/a47836e01af9..b2d5d40a3393 git log a47836e01af9..b2d5d40a3393 --date=short --no-merges --format='%ad %ae %s' 2019-09-10 bsalomon@google.com Adapt GrRectBlur for cases with rect W/H less than six sigma. 2019-09-10 jvanverth@google.com Hook up pinch-zoom and swipe gestures. 2019-09-10 bsalomon@google.com Remove per-format glCheckframebufferStatus checks. Created with: gclient setdep -r src/third_party/skia@b2d5d40a3393 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 mtklein@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/+/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 TBR=mtklein@google.com Bug: None Change-Id: I7a6344c8dc37b12e5fc9b962cacba850c9ddbb48 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796105Reviewed-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@{#695198}
-
Shivani Sharma authored
top-frame-origin was added to RedirectInfo for splitting the http cache using the top-frame-origin and that had to be updated when navigation requests were redirected. Now split cache uses network isolation key (computed in the browser process) and it is updated on redirect in the URLLoader->FollowRedirect. After these changes the top-frame-origin in RedirectInfo is no longer needed. Navigation resources with split cache are already being tested in existing browser tests and this removal does not break any of those tests. Bug: 978968 Change-Id: I58a69b4799fe3ba567cfe09989890e9af86924be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1732794Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Shivani Sharma <shivanisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#695197}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /content/browser/devtools. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=caseq@chromium.org Bug: 891670 Change-Id: I42f549fef6bf7c3e1da3c6b08fdd64353f741a69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786901 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#695196}
-
John Chen authored
Perf test system_health.common_mobile/background:news:nytimes is failing on all Android devices. Disabling it. TBR=crouleau@chromium.org NOTRY=true Bug: 1002546 Change-Id: If54c8c34edace7711c12fbe809044158ea6708b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795753Reviewed-by:
John Chen <johnchen@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#695195}
-
Koji Ishii authored
This patch fixes `vertical-align: top` and `bottom` when they are applied to a box that has children with `text-top` or `text-bottom`. In LayoutNG, `vertical-align` is handled in 3 different timings. 1. `top` and `bottom` are added to the pending list of the line box, or to the nearest ancestor box that has `top` or `bottom`. 2. `text-bop` and `text-bottom` are added to the pending list of the parent box. 3. Other values are computed when the box is closed. When a box has both 1 and 2, this patch changes to compute 1 (`top` and `bottom`) after 2. Before this patch, both 1 and 2 are computed at the same time for each box. 3 test cases out of 20 in this test fails without this patch. Bug: 1001664 Change-Id: I0e7746447f5e401837c2c28e308171f0987eba35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792396Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#695194}
-
Abigail Klein authored
This flag will be used for improvements to the chrome:accessibility page. The page queries for the accessibility tree, which is sensitive code that powers accessibility. Making changes to that code needs to be done carefully and will require its own QA pass, which is what motivates adding this flag. Bug: 785493 Change-Id: I37d57761447ca1ab17a91e410b09547bf7ac4a89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790067Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Abigail Klein <abigailbklein@google.com> Cr-Commit-Position: refs/heads/master@{#695193}
-
Matt Menke authored
This CL deletes FileProtocolHandler from net/, makes URLRequestFileJob a test-only class (HTTP request mocks subclass it), and deletes URLRequestFileDirJob and NetworkDelegate::CanAccessFile. File URLs are now handled outside of the network service, so this code is no longer being used. This CL leaves in place net/directory_* files, which are used by the new code (And should probably be moved somewhere more appropriate for their new use, in another CL), and the network build option disable_file_support, which should also be cleaned up when moving those files. TBR=seantopping@chromium.org Bug: 934009 Change-Id: I2ef493849847adbb340a364de12ea41e91870b6d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787187 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#695192}
-
Matt Menke authored
Currently, they're always empty, because HttpServerPropertiesManager neither saves nor loads them, nor does HttpServerProperties take them as input. Bug: 969890 Change-Id: Ifd59c763be96ef2ded39501f5af2cbcc3dea3906 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769527 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#695191}
-
Kamila authored
This CL implements two experimental UI flavors for surfacing notification permission requests to users on Desktop that are quieter than the currently implemented permission bubbles hanging off the left-hand side of the Omnibox. The new UX consists of showing a "Notifications blocked" icon in the Omnibox, clicking on which will display a bubble where users have the ability to add a per-site allow exception by clicking "Allow for this site", as well as a "Manage" button for managing notification preferences. The first UX flavor shows a static icon, while the second flavor shows a sliding icon that appears with animation and is initially accompanied by an explanatory text label before it is minimized. Detailed changes: - Show the notification prompt as an indicator icon in the location bar. - Change the logic to show an indicator when a permission is requested (instead of when a permission is accepted or denied). - Let permission request manager know when the user allows notifications for a site. - Implement the new notifications permission request image and bubble model. - Implement the icon variations in UI in content_setting_image_model. - Add experimental variations, complementary to Android. Screenshots: Static icon http://screen/XVLQ1sAWHQD Sliding icon http://screen/mDdFC3utKaP Bug: 986737 Change-Id: Id40db28465d36f502d09f09c0ad0cb4c2cf904e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726092 Commit-Queue: Kamila Hasanbega <hkamila@google.com> Reviewed-by:Evan Stade <estade@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#695190}
-
Mandy Chen authored
The city names in settings are not localized because the localization parser doesn't know about them. Modify the parser to recognize these strings. Bug: 941561 Change-Id: If1974dd471b3d646c900dea6932f427a273c19f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769300Reviewed-by:
Erik Luo <luoe@chromium.org> Commit-Queue: Mandy Chen <mandy.chen@microsoft.com> Cr-Commit-Position: refs/heads/master@{#695189}
-
Peter Kasting authored
Bug: 1000055 Change-Id: I18e9dda2079ac4634abcb4cadd35fbcb11b82c08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793257 Auto-Submit: Peter Kasting <pkasting@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#695188}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /services/video_capture. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=chfremer@chromium.org Bug: 891670 Change-Id: Icccc112e5c7be94fd38e2901c013657910579288 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787110 Auto-Submit: Gabriel Charette <gab@chromium.org> Reviewed-by:
Christian Fremerey <chfremer@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#695187}
-
Gyuyoung Kim authored
This CL applies pending_remote to AssistantService::Init and Assistant::AddAssistantInteractionSubscriber. - Convert mojo::Binding to mojo::Receiver - Rename AddBinding to AddReceiver - Apply pending_remote to AssistantInteractionSubscriberPtr Bug: 955171 Change-Id: I6f6653c10c56d60dfc259eb25b5e27936a2cfd0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1778896Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#695186}
-
John Chen authored
rendering.mobile/balls_svg_animations has already been disabled on android-webview, but consistent failures are also observed on non-webview devices. Disabling on all Android devices. TBR=crouleau@chromium.org NOTRY=true Bug: 1000473 Change-Id: I8fa8008e97bd768ffd70758376dc73ade025d662 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795750Reviewed-by:
-
Anastasiia Nikolaienko authored
To allow reauthentication of secondary accounts from ARC, remove check for account value. Secondary accounts which have been added directly from ARC may not be in the list of Chrome accounts. If invalidated account is in ARC, but not in Chrome OS Account Manager, return an error. Bug: b/137921583 Change-Id: I4a0a904481e1afc39a17e2a755987b7aaad72da1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746173 Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Reviewed-by:
Mattias Nissler <mnissler@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Cr-Commit-Position: refs/heads/master@{#695184}
-
Aga Wronska authored
Change-Id: Ia92d28d7287bd14c5da2d706b266556b4b3c0de1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793768Reviewed-by:
-
Nick Burris authored
Add a 'selector' member to window.location which is currently just an empty object but can be used for feature detection for scroll to text, and can be extended later to include properties for the fragment directive. Also add a test for the member to the web platform test. Tested updated WPT locally with run_web_tests.py --additional-driver-flag= '--enable-blink-features=TextFragmentIdentifiers' Bug: 1000308 Change-Id: I99b5ea14008e0b84f1e67129b7af27107b1fdec0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791793Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/master@{#695182}
-
Donn Denman authored
Fixes a bug with the Contextual Search panel on tablets: when not full width it is no longer centered. Now it is. Details in the bug. BUG=1000952 Change-Id: I39d7f4c11f42b7e78a6c8373b1ca8e09600ac012 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790327 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Auto-Submit: Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#695181}
-
Anton Bikineev authored
After this commit the GarbageCollectedFinalized class becomes a dumb class. It will be removed as soon as the blink-gc-plugin is updated and clang is rolled. Change-Id: I6c6454bf434f1daac0ee3b741995113c3e68c15e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738378 Commit-Queue: Anton Bikineev <bikineev@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#695180}
-
Sinan Sahin authored
testDismissContextMenuOnClick instrumentation test fails because we click on a notification instead of a view in the app which causes a java.lang.SecurityException: "Injecting to another application requires INJECT_EVENTS permission." To work around this issue, this CL changes the test to click roughly the bottom right corner of the screen. Bug: 1001966 Change-Id: I3f0d3c06ad72450217b936ea17d896db4162deb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792685Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Sinan Sahin <sinansahin@google.com> Cr-Commit-Position: refs/heads/master@{#695179}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/9d28be075d5f..c1ec0cbb921b Created with: gclient setdep -r src-internal@c1ec0cbb921b If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@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/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I5cbf845b61917a517a7454defa1b5b21ee83e9fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795884Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#695178}
-
Adithya Srinivasan authored
Bug: 1002527 Change-Id: If65a5f849941a7fe1d1ed88d59a7bd8dc7198b37 No-Try: true Tbr: eae@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795746Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#695177}
-
Tsuyoshi Horo authored
I removed "import android.util.Log;" from ContentUriUtils.java in crrev.com/c/1788428. So org.chromium.base.Log is used for logging in the file. org.chromium.base.Log uses the second argument to call String.format(). So this may cause UnknownFormatConversionException if the second argument contains format specifiers. Bug: 995177 Change-Id: I032a83828dac43d99097727788cedae5d80e172f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795529 Commit-Queue: Tsuyoshi Horo <horo@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#695176}
-
Ulan Degenbaev authored
Bug: 999004 Change-Id: I9a95be621b29e398509bdcd3cfadf87902a399a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789306Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org> Cr-Commit-Position: refs/heads/master@{#695175}
-
Mohamed Amir Yosef authored
This is a pure refactoring CL to rename PasswordManagerClient.GetPasswordStore() to PasswordManagerClient.GetProfilePasswordStore() Follow up patches will introduce PasswordManagerClient.GetAccountPasswordStore() TBR=vasilii@chromium.org Change-Id: Ic9e4d44bce0aa019a9ee46eb1ea0f2d868d2a307 Bug: 1002000 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795004Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Cr-Commit-Position: refs/heads/master@{#695174}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/f7b1aa440dc0..45b01c7962da git log f7b1aa440dc0..45b01c7962da --date=short --no-merges --format='%ad %ae %s' 2019-09-10 nisse@webrtc.org Delete some dead code in vcm::VideoReceiver and VCMReceiver 2019-09-10 nisse@webrtc.org Move code related to VideoCodingModule to its own build target 2019-09-10 danilchap@webrtc.org Mark test::DriftingClock constants as constexpr 2019-09-10 titovartem@webrtc.org Add ability to disable PSNR and SSIM computation in DVQA Created with: gclient setdep -r src/third_party/webrtc@45b01c7962da 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/+/master/autoroll/README.md TBR=webrtc-chromium-sheriffs-robots@google.com Bug: chromium:None,chromium:None Change-Id: Iee16eb483973717531465458ae4a9f399d31108a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795883Reviewed-by:
-
Adam Langley authored
This is a prelude to adding v2 elements in a subsequent change. Bug: 1002262 Change-Id: I4313a83d3dd9670ed0a92cd67794e47f0974bf88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785660Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#695172}
-
Jérôme Lebel authored
Adding comment to explain when the SigninInteractionCoordinator needs to be dismissed -[MainController closeSettingsAnimated:completion:]. Related to: crrev.com/c/1788026/4/ios/chrome/app/main_controller.mm#2275 Change-Id: Ie7685cadf00f9d7893ce55fb346667f014272d15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795904Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#695171}
-
Ryan Sturm authored
This code was added to remove the page load capping DB from M74-M78, if any clients (that were exposed to the experiment) skipped from M73 to M79+ they will not delete the DB and it will just hang around, but this should be rare as only 1% of certain countries had page capping enabled in Stable. Bug: 937489 Change-Id: Id67670b2d32fbee45a3b0b0097aacfb6784ae42f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793542Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#695170}
-
Anqing Zhao authored
- Timestamp of last reboot. - Timestamp of last update check. - Add a new policy 'ReportDeviceOsUpdateStatus': Report OS update information such as update status, platform version, update check and last reboot. Bug: 983180 Change-Id: Ib296a64863e819e40329305cf85440e8c0037a66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787648Reviewed-by:
Sergey Poromov <poromov@chromium.org> Commit-Queue: Anqing Zhao <anqing@google.com> Cr-Commit-Position: refs/heads/master@{#695169}
-
Primiano Tucci authored
https://android.googlesource.com/platform/external/perfetto.git/+log/23d211c8..c85e95e7f Catch up with https://android-review.googlesource.com/c/platform/external/perfetto/+/1116517 that removed a method and changed a header location. This is a pure refactoring with no behavioral change. git log 23d211c8..c85e95e7f --date=short --no-merges --format='%ad %ae %s' 2019-09-10 primiano@google.com Fix chrome roll: add PERFETTO_EXPORT to SlicedProtobufInputStream 2019-09-09 primiano@google.com PERFETTO_EXPORT PacketStreamValidator 2019-09-09 primiano@google.com Disable watchdog on OSX using compile-time checks 2019-09-09 primiano@google.com Remove TracePacket::Decode and protobuf dependencies Change-Id: Iac741162bb179d4497407346dd4d9abdbddfc9e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792865 Commit-Queue: Primiano Tucci <primiano@chromium.org> Auto-Submit: Primiano Tucci <primiano@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/heads/master@{#695168}
-
Miyoung Shin authored
This CL converts BluetoothSystemClient{Ptr, Request} in ash and services to the new Mojo type. Bug: 955171 Change-Id: Ie2628589f4bed2828405ee9f445b8bad680d7505 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787667 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#695167}
-
Adithya Srinivasan authored
Reverted the wrong test in https://crrev.com/c/1795627. Undo that revert, and revert the correct test. Bug: 1002488 Change-Id: Id1674227e0db390b5f7bf75755a5bec3cae97042 Tbr: tbarzic@chromium.org No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796044Reviewed-by:
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /components/image_fetcher. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=skym@chromium.org Bug: 891670 Change-Id: I8736433bb72b9f06daa837f867063d5e0a52b938 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787212 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Reviewed-by:
Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#695165}
-
Robbie Gibson authored
The changes are replacing darker, primary text with kTextPrimaryColor and lighter, secondary text with kTextSecondaryColor. Bug: 1002409 Change-Id: I2a1cfa0eb5907cc1c40b722c2ddbd9285187a56a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1794865 Commit-Queue: Robbie Gibson <rkgibson@google.com> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/3255f493db0d..79ad0411911b git log 3255f493db0d..79ad0411911b --date=short --no-merges --format='%ad %ae %s' 2019-09-10 tobine@google.com Vulkan:Pass through min/max program texel offset Created with: gclient setdep -r src/third_party/angle@79ad0411911b If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC syoussefi@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/+/master/autoroll/README.md 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 TBR=syoussefi@chromium.org Bug: None Change-Id: I9951758efd804dc271129cc54e794d66784b0857 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795281Reviewed-by:
-
Robert Ma authored
TBR=lpz No-Try: True Bug: 1002514 Change-Id: Ia082b22d887ad6561cfa90e95be2c963547d467a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795483Reviewed-by:
Robert Ma <robertma@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#695162}
-
Vladimir Levin authored
This patch ensures that we update layout if the value of content-size changes. R=chrishtr@chromium.org, futhark@chromium.org Bug: 1002231 Change-Id: I68e64b162d936cc45da45d6c0e70c5a5a9de6af8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790417 Commit-Queue: vmpstr <vmpstr@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#695161}
-
Clemens Arbesser authored
This CL will ensure that by default, the most complete contact profile with the same email address as the one provided for the access token is selected. Bug: b/140781117 Change-Id: I6a54ddac7190b6aed2f15b848a1a1ca81c684009 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795325 Commit-Queue: Clemens Arbesser <arbesser@google.com> Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Cr-Commit-Position: refs/heads/master@{#695160}
-
