- 31 Oct, 2018 40 commits
-
-
Chromium WPT Sync authored
Using wpt-import in Chromium 2fd20d12. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/26811 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: kojii@chromium.org: external/wpt/css/css-scoping NOAUTOREVERT=true TBR=lukebjerring No-Export: true Change-Id: I0c25127f99cfc65d10251011351b6c8b27b5f945 Reviewed-on: https://chromium-review.googlesource.com/c/1309681 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#604296}
-
Tarun Bansal authored
This reverts commit a3bc216e. Reason for revert: Reverting the CL did not fix the flaky test. See crbug.com/900606. Original change's description: > Speculatively Revert "[url_pattern_index] Print matching rule for tracing" > > This reverts commit 1c7e7665. > > Reason for revert: > Fails test org.chromium.chrome.browser.SubresourceFilterTest#resourceFilteredClose > See https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20CFI/3471 > > Original change's description: > > [url_pattern_index] Print matching rule for tracing > > > > This is useful for debugging. > > > > Bug: None > > Change-Id: I8e6260f09b3c17f0a9952739387f70a431571011 > > Reviewed-on: https://chromium-review.googlesource.com/c/1308013 > > Commit-Queue: Charlie Harrison <csharrison@chromium.org> > > Reviewed-by: Josh Karlin <jkarlin@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#603990} > > TBR=jkarlin@chromium.org,csharrison@chromium.org > > Change-Id: I44e487142ab5c3477ff77945a3f2854bb9762056 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: None > Reviewed-on: https://chromium-review.googlesource.com/c/1308904 > Reviewed-by: Tarun Bansal <tbansal@chromium.org> > Commit-Queue: Tarun Bansal <tbansal@chromium.org> > Cr-Commit-Position: refs/heads/master@{#604073} TBR=jkarlin@chromium.org,tbansal@chromium.org,csharrison@chromium.org Bug: 900606 Change-Id: Ibf69e14dfdc650776d542ce6334102b67317fa89 Reviewed-on: https://chromium-review.googlesource.com/c/1310115Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#604295}
-
Collin Baker authored
ReopenTabInProductHelp watches for certain user interactions and possibly triggers in-product help in response to them. These hooks call into this when such user events happen. Bug: 887991 Change-Id: I8f07626f07375a902f87f62dd662d51e8dbfc2ca Reviewed-on: https://chromium-review.googlesource.com/c/1306791 Commit-Queue: Collin Baker <collinbaker@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#604294}
-
Takumi Fujimoto authored
The WebAuth dialog has implemented its own MD-style pulldown button with a down arrow on it. This CL moves the button class into its own file and reuses it as the source picker pulldown button in the Cast dialog. Bug: 893307 Change-Id: I565ce0e991c9f41f3b2ae6747e017dbbfec84523 Reviewed-on: https://chromium-review.googlesource.com/c/1278127 Commit-Queue: Takumi Fujimoto <takumif@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#604293}
-
Xida Chen authored
Currently at ApplyStandardPropertyValue, we give a double to FontVariationAxis which is supposed to take a float, and this could cause float-cast-overflow in corner cases. This CL does static_cast on the double value before given that to the FontVariationAxis. Bug: 900493 Change-Id: Ia4cc4b97bd81296ea1721affead42a2f482b58c0 Reviewed-on: https://chromium-review.googlesource.com/c/1309879Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#604292}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/ed45c57d9818..e6caa9fbf6ef git log ed45c57d9818..e6caa9fbf6ef --date=short --no-merges --format='%ad %ae %s' 2018-10-31 xz4215@gmail.com export RTCRtpTransceiverInit Created with: gclient setdep -r src/third_party/webrtc@e6caa9fbf6ef The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-chromium-autoroll 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=luci.chromium.try:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng BUG=chromium:none TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: If8f960355840670a6c8ff081021fba9e288ff88e Reviewed-on: https://chromium-review.googlesource.com/c/1309550Reviewed-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@{#604291}
-
Lei Zhang authored
It is only used in unit tests. Similarly remove the frame GUID. Change-Id: Idb9bf8f53ce825ea27cfc638ccd437c17e2b0e6f Reviewed-on: https://chromium-review.googlesource.com/c/1308075 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Wei Li <weili@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#604290}
-
Ehsan Chiniforooshan authored
We are merging TBMv1 and TBMv2 metrics and no longer have two versions of every metric. Bug: 890757 Change-Id: Ia304e7fa10d3abcf480d511751b16addab130713 Reviewed-on: https://chromium-review.googlesource.com/c/1309995 Commit-Queue: Ehsan Chiniforooshan <chiniforooshan@chromium.org> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#604289}
-
Antonio Gomes authored
This CL changes password_sync_util.cc|h and its various call sites to work with IdentityManager rather than SigninManager. There should be no functionality change, and the work is part of the identity manager servicification (//services/identity) effort. Specifically about IdentityManager, it wraps internally both SigninManager and OAuth2TokenService objects, exposing a new API. BUG=889874 Change-Id: I2014aff5a3fe4842d23f9c7610adbb9522f9e366 Reviewed-on: https://chromium-review.googlesource.com/c/1306153 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Vaclav Brozek <vabr@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#604288}
-
Clark DuVall authored
This test was added in http://crrev.com/c/1299148 and has been failing on the mojo bot. TBR=timvolodine@chromium.org Change-Id: I536a89fc6a3a39225ddccd4d71c3e5ec954c2198 Reviewed-on: https://chromium-review.googlesource.com/c/1310116Reviewed-by:
Clark DuVall <cduvall@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#604287}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/fa909263570e..a3a8f1e2b37f git log fa909263570e..a3a8f1e2b37f --date=short --no-merges --format='%ad %ae %s' 2018-10-31 rsavitski@google.com Kokoro: stay as root when invoking test script inside a container. Created with: gclient setdep -r src/third_party/perfetto@a3a8f1e2b37f The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll 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=perfetto-bugs@google.com Change-Id: I08962051363be2fe4bce12c21bc8032bfdef1c56 Reviewed-on: https://chromium-review.googlesource.com/c/1309551Reviewed-by:
-
Xiaohan Wang authored
When using BEST_EFFORT, the CDM is registered pretty late that browser tests are failing, and if we specify a site playing encrypted media on command line, the site will find Widevine not supproted (because it's not registered yet). This CL changes the task priority to be USER_VISIBLE to solve this issue. Bug: 900169 Test: Failing tests are passing now. Change-Id: I8530048a07d804f5d99637745a8b7db0851ff221 Reviewed-on: https://chromium-review.googlesource.com/c/1308974 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#604285}
-
Daniel Bratell authored
In jumbo builds many cc files compile together in the same translation unit and then they share the same anonymous namespace. That means that within a (jumbo) build target, also symbols in anonymous namespaces need (build target) unique names. In a recent patch, two GetPromoStringSpecifier functions appeared. They are almost identical but not quite, so this patch just gives them unique names to make all jumbo build configurations happy again. Most jumbo build configurations were already happy because the relevant files did not easily end up in the same jumbo chunk. See https://chromium-review.googlesource.com/c/1298394 TBR=collinbaker@chromium.org,bsep@chromium.org,nyquist@chromium.org,robliao@chromium.org Change-Id: I5f244fc6a4e5b2047f2fdb443d0e40565c8bc103 Reviewed-on: https://chromium-review.googlesource.com/c/1309739Reviewed-by:
Daniel Bratell <bratell@opera.com> Reviewed-by:
Robert Liao <robliao@chromium.org> Commit-Queue: Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#604284}
-
Lukasz Suder authored
Bug: 806868 Change-Id: Ibb115c06bd38f4f5749d15a292d93206dba13d32 Reviewed-on: https://chromium-review.googlesource.com/c/1307410Reviewed-by:
Stephane Zermatten <szermatt@chromium.org> Commit-Queue: Lukasz Suder <lsuder@google.com> Cr-Commit-Position: refs/heads/master@{#604283}
-
Justin Cohen authored
justincohen@ is an author of all code in ios/chrome/browser/ntp Change-Id: I5888fc83e7fa5aabc3187d2c12456adce9ab1da8 Reviewed-on: https://chromium-review.googlesource.com/c/1308994Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#604282}
-
Ken Rockot authored
Gets rid of ForwardingService since it's no longer necessary. Instead, relevant uses of ServiceContext -- whose use necessitated ForwardingService -- have been replaced with the more flexible ServiceBinding. Bug: 891780 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I86011161234afa7395a7bc1240d360bcf97fa851 Reviewed-on: https://chromium-review.googlesource.com/c/1259487Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#604281}
-
Dave Tapuska authored
Mozilla has requested changes to the IDL and either format is identical for Chrome (after we've shipped). They weren't when it was optional. BUG=899734 Change-Id: I85dc7e27c4dc77f5f6443e6ba212f4835ba30e77 Reviewed-on: https://chromium-review.googlesource.com/c/1304615Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#604280}
-
Jared Saul authored
This is a redo of https://chromium-review.googlesource.com/c/chromium/src/+/1300794. That CL was reverted because there were still internal Android dependencies that I originally missed, which I resolved earlier today. Bug: 881343 Change-Id: I99b87e859c0fc147f1c8343f5170d93cad066f7d Reviewed-on: https://chromium-review.googlesource.com/c/1308607Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Commit-Queue: Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#604279}
-
Tim Zheng authored
The default Crostini terminal app is crosh in a Chrome window and the display density toggle doesn't apply. BUG=chromium:839242;chromium:900032 TEST=Manually tested on an eve device Change-Id: I5d7007009803b63590864e107ece80082f9cfb9b Reviewed-on: https://chromium-review.googlesource.com/c/1308416Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Tim Zheng <timzheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#604278}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/2e039e9c4069..ab0e8a3d06fc Created with: gclient setdep -r src-internal@ab0e8a3d06fc The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: I06799bea43c3f66d2fde2a7745996ba68ba1f5fb Reviewed-on: https://chromium-review.googlesource.com/c/1309548Reviewed-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@{#604277}
-
Fredrik Söderquist authored
Per: "A value of zero for either width or height disables rendering of the element and its embedded content." (https://svgwg.org/svg2-draft/embedded.html#Placement) and: "For each descendant graphics element child of parent: * If child is not rendered then continue to the next descendant graphics element." (https://svgwg.org/svg2-draft/coords.html#BoundingBoxes) <foreignObject> / <image> with 'width' or 'height' zero should not contribute to an ancestor container's bounding box. Bug: 643493 Change-Id: I097d9d28faaa52fc793dab251f22bc7b2610d6ad Reviewed-on: https://chromium-review.googlesource.com/c/1309773Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#604276}
-
Morten Stenshorne authored
When an inline object has a block child, we need to split the inline into two "continuations", wrap them inside separate anonymous blocks, and put the actual block child(ren) between those anonymous blocks, also wrapped inside a designated anonymous block. If we later on remove that block child, we don't tear down the continuation chain completely. We get rid of all the anononymous blocks, but keep the two split inlines, that will suddenly become adjacent siblings. They should ideally be joined back into one layout object, but we don't do that. And this fix isn't about that (because I really wish to do other things than just causing and fixing fuzzer crashers for the rest of this year and half of the next). Instead, make sure that we cope with bogus continuation chains. The problem was when an inline that had a bogus continuation chain became relatively positioned, and we ended up marking a subsequent anonymous block as relatively-positioned, although none of its children were DOM children of the inline. This in turn caused trouble for absolute positioning, after https://chromium-review.googlesource.com/c/1301457 , which expects that there be relatively positioned inlines in the vicinity if an anonymous container with block children is relatively positioned. Bug: 900451 Change-Id: Ibde930b12201c10cdf508733d4d182d731e11ecb Reviewed-on: https://chromium-review.googlesource.com/c/1309777 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#604275}
-
Mason Freed authored
This fixes several clusterfuzz issues, which are caused by frames being freed in between calls to CompositeWithRaster(). With this CL, a fresh list of frames is requested and checked for the next one, to avoid using deleted frames. Note that with this CL: https://chromium-review.googlesource.com/c/chromium/src/+/1252141 those calls are asynchronous. Also note that this CL: https://chromium-review.googlesource.com/c/chromium/src/+/1213864 caused this code to get called much more frequently, for all layout tests. Here are the clusterfuzz issues: https://clusterfuzz.com/v2/testcase-detail/5701500434907136 https://clusterfuzz.com/v2/testcase-detail/4929420383748096 https://clusterfuzz.com/v2/testcase-detail/4996950557196288 Bug: 900087, 899465, 899450 Change-Id: I70fc7c723b2118f4796289fe9b7272c42b6e50e5 Reviewed-on: https://chromium-review.googlesource.com/c/1308038Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#604274}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/83d923f38664..73329c8df9d5 git log 83d923f38664..73329c8df9d5 --date=short --no-merges --format='%ad %ae %s' 2018-10-31 reed@google.com remove alignenum flag -- rely on clients to add it as needed Created with: gclient setdep -r src/third_party/skia@73329c8df9d5 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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=luci.chromium.try:android_optional_gpu_tests_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;master.tryserver.blink:linux_trusty_blink_rel TBR=herb@chromium.org Change-Id: I8df114a6bad35b88ecc37ff2a62e656701eead31 Reviewed-on: https://chromium-review.googlesource.com/c/1309549Reviewed-by:
-
James Hawkins authored
The EasyUnlock V1 setup flow is being removed, so this caching mechanism is not necessary at the moment and helps break up the removal change. R=hansberry@chromium.org Bug: 899324 Test: none Change-Id: I7a15a78d95edf5585636dc398f708fa12a7e0bc7 Reviewed-on: https://chromium-review.googlesource.com/c/1306348 Commit-Queue: James Hawkins <jhawkins@chromium.org> Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#604272}
-
James Hawkins authored
Since is the last method on EasyUnlockServiceObserver, that interface is also removed. Bug: 899324 Test: none Change-Id: Idf4c4136ec97431a5ceaa6d9a3c69a5a782b7882 Reviewed-on: https://chromium-review.googlesource.com/c/1307068 Commit-Queue: James Hawkins <jhawkins@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#604271}
-
Nikita Podguzov authored
Bug: 899193 Change-Id: I282a91ea90d5cebe4353a223960712c3b3be9c73 Reviewed-on: https://chromium-review.googlesource.com/c/1308715Reviewed-by:
Alexander Hendrich <hendrich@chromium.org> Reviewed-by:
-
Tim Volodine authored
Make sure the Accept-Language HTTP request header works in Android WebView and fix the corresponding tests. In this patch: - make sure to update the network context using SetAcceptLanguage when the WebView Accept-Language setting changes. - fix tests This patch fixes: - org.chromium.android_webview.test.AcceptLanguageTest#testAcceptLanguage - org.chromium.android_webview.test.AcceptLanguageTest#testAcceptLanguagesWithenUS BUG=893561,841556 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: Ibd745513484466541dffbf9d24b65c1e702dc9b0 Reviewed-on: https://chromium-review.googlesource.com/c/1280857 Commit-Queue: Tim Volodine <timvolodine@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#604269}
-
Robert Sesek authored
Bug: 891697 Change-Id: I49eb77963515637df739c9d2ce83530d4e21cf15 Reviewed-on: https://chromium-review.googlesource.com/c/1308771Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#604268}
-
sczs authored
- Adds message parameter support for TestInfobarDelegate. - Adds Infobar eg test cases for stacked Infobars. Bug: 892376 Change-Id: I9f1f8b16d84ab0bf19111fa50e790319414a9a2d Reviewed-on: https://chromium-review.googlesource.com/c/1308900Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#604267}
-
Collin Baker authored
This will allow the per-profile instance to be fetched from any code that has a Profile*. Bug: 887991 Change-Id: I8394c07033b8e2940cf2a76bb6a697ca09ada452 Reviewed-on: https://chromium-review.googlesource.com/c/1306788 Commit-Queue: Collin Baker <collinbaker@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#604266}
-
Peng Huang authored
Since GLContextVirtual::Initialize() will set GLStateRestorer, so RasterCommandBufferStub and InProcessCommandBuffer don't set it. Bug: None Change-Id: I29f517b03170ce6d5d1bd7182f036aedef090507 Reviewed-on: https://chromium-review.googlesource.com/c/1309881Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#604265}
-
henrika authored
Summary: - getUserMedia({audio: {echoCancellation: {exact: false}}}) => - Adds support for "clean mode" without using any input effects on Android - Avoid switching to COMM mode - Removes usage of AEC white list for Android - Removes spam log messages - Extends logging of audio parameters in audio streams Bug: 866390 Tbr: dalecurtis@chromium.org 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: I45beca7e37c380826cff15ff917233e8379986cb Reviewed-on: https://chromium-review.googlesource.com/c/1304516 Commit-Queue: Henrik Andreasson <henrika@chromium.org> Reviewed-by:Oskar Sundbom <ossu@chromium.org> Reviewed-by:
Max Morin <maxmorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#604264}
-
Francois Doray authored
WorkingSetTrimmer empties the working set of processes in which all frames are frozen. We want to do this to: 1. Track working set growth rate (we empty the working set and record its size x minutes after). 2. Assess impact on global performance. Bug: 885293 Change-Id: I371cc17836b6de5cc749dfb6488b7734a6bf870c Reviewed-on: https://chromium-review.googlesource.com/c/1238819 Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#604263}
-
Owen Min authored
Disable this test on Mac10.12 and Win7 because the test is flaky. Bug: 900326 Change-Id: Iaed7c8a54ec98aedddc7c9584556c202d368294c Tbr: dgozman@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/1310013Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#604262}
-
Owen Min authored
Bug: 899893 Change-Id: I627404e1ab4344da015b593f46ad751404ae3d4f Tbr: raymes@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/1309117Reviewed-by:
-
Xida Chen authored
This variable was added to test whether the touch start / end can be acked and released out of order or not. From the reading of the code, I don't think is true. This CL removes this variable, we will monitor crash reports to ensure this doesn't cause regression. To ensure the same behavior, we need a new variable, which is named |touch_sequence_in_progress_| in this CL. This is specifically for this scenario: we have received touch event ack for touch start, and then JS removes / adds touch event listener, which triggers a OnHasTouchEventHandlers call. In this case, we should not reset touch action if the |touch_sequence_in_pgoress_| is true. We have the unit test OnHasTouchEventHandlersReceivedAfterTouchStart to ensure this behavior. Bug: 900195 Change-Id: I53c3141905b4e2a49ab6e62ea43335c4cee17c15 Reviewed-on: https://chromium-review.googlesource.com/c/1307250 Commit-Queue: Xida Chen <xidachen@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#604260}
-
Dan Harrington authored
When AutoFetchOnNetErrorPage is enabled, the dino page will schedule fetching the page in the background. The UI changes associated with this feature will come in a subsequent CL. Bug: 883486 Change-Id: I308fc93026cf8a126a68b3aa035cf37df39f7871 Reviewed-on: https://chromium-review.googlesource.com/c/1289705 Commit-Queue: Dan H <harringtond@google.com> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Cathy Li <chili@chromium.org> Cr-Commit-Position: refs/heads/master@{#604259}
-
Mostyn Bramley-Moore authored
This change should allow cache hits with goma and ccache. Change-Id: Ie9f1566d6f76a3de5fba5d039ff1a7f5b6f3b2ac Reviewed-on: https://chromium-review.googlesource.com/c/1309778 Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Reviewed-by:
John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#604258}
-
Paul Wankadia authored
https://chromium.googlesource.com/external/github.com/google/re2.git/+log/22caec62055a..46155299edb9 $ git log 22caec620..46155299e --date=short --no-merges --format='%ad %ae %s' 2018-10-31 junyer Don't log/crash when PrefilterTree::KeepNode() sees Prefilter::NONE. Created with: roll-dep src/third_party/re2/src R=mmoroz@chromium.org Bug: 900558 Change-Id: Ic497e7019b2b072a91b0ea3710c138efa6d1ef7a Reviewed-on: https://chromium-review.googlesource.com/c/1309878Reviewed-by:
Max Moroz <mmoroz@chromium.org> Commit-Queue: Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#604257}
-
