- 31 Oct, 2018 40 commits
-
-
Orin Jaworski authored
The code eliminated was a switch constant that wasn't being used. The .cc, .h, and related inclusions were eliminated with no effect. Bug: 900255 Change-Id: I76521fb8acce5b7eb5aa9e38fa748953468105ea Reviewed-on: https://chromium-review.googlesource.com/c/1308040Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Orin Jaworski <orinj@chromium.org> Cr-Commit-Position: refs/heads/master@{#604303}
-
Dmitry Gozman authored
This reverts commit a58614dd. Reason for revert: the issue should be fixed by https://chromium-review.googlesource.com/c/chromium/src/+/1309213 Original change's description: > Disable WorkerDevToolsSanityTest.PauseInSharedWorkerInitialization on Linux > > Change-Id: Ibac0faa988dfa05b727967251d0f51cfe0b474d1 > Bug: 900432 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > TBR: dgozman@chromium.org > Reviewed-on: https://chromium-review.googlesource.com/c/1309322 > Commit-Queue: Tarun Bansal <tbansal@chromium.org> > Reviewed-by: Tarun Bansal <tbansal@chromium.org> > Cr-Commit-Position: refs/heads/master@{#604089} TBR=dgozman@chromium.org,tbansal@chromium.org Change-Id: Ib776370c42fffc4c44ddec9eca10c8093c6db01b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 900432 Reviewed-on: https://chromium-review.googlesource.com/c/1310355Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#604302}
-
Stephen Martinis authored
It appears that this isn't being set anywhere in recipes, so this should maybe be safe to remove? Bug: 718157 Change-Id: Iec1934638ec6f1acaf5b1b96c1e8fdb129457c39 Reviewed-on: https://chromium-review.googlesource.com/c/1308912 Commit-Queue: Stephen Martinis <martiniss@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#604301}
-
Min Qin authored
Currently when history db fails to load, we creates a temporary download and won't persist it back to the history db. When in-progress DB is enabled, we should do the same. if history service won't return a valid id for a new download, this download should only live with the current browser sessions. BUG=898859 Change-Id: I1b09b59358b043541c4642d2080040c947ce8560 Reviewed-on: https://chromium-review.googlesource.com/c/1305877Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Reviewed-by:
Changwan Ryu <changwan@chromium.org> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#604300}
-
Dmitry Gozman authored
When pausing on initialization, we issue Debugger.enable immediately followed by Debugger.pause and expect the pause to happen. Because Debugger.pause can interrupt JS, it may actually run before Debugger.enable on the backend. That makes pause noop, and we never pause. The solution is to await for Debugger.enable before asking to pause. Bug: 775132, 900432 Change-Id: Ic2f504875b98312839235c6e32a10403757444f2 Reviewed-on: https://chromium-review.googlesource.com/c/1309213 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#604299}
-
Rayan Kanso authored
The spec states that SyncManager::register should be rejected with an InvalidStateError if there is no active ServiceWorker, and NotAllowedError if the Background Sync has been disabled. https: //wicg.github.io/BackgroundSync/spec/#sync-manager-interface Change-Id: I7ef1ef99a388a61949bd0a7bfa2555ba9b7d6ad8 Reviewed-on: https://chromium-review.googlesource.com/c/1309790Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Commit-Queue: Rayan Kanso <rayankans@chromium.org> Cr-Commit-Position: refs/heads/master@{#604298}
-
Bo Liu authored
The default for an "is empty" value should be true to better match reality. This is breaking the case where an offscreen webview is directly translated on-screen without another onDraw recording on the UI thread. The comparison in BVR::OnParentDrawConstraintsUpdated is supposed to catch this case and cause another invalidate. This CL refers to internal bug b/117560442 Change-Id: Ic03f91631780a2057931cb39cdecd04d32f52b1d Reviewed-on: https://chromium-review.googlesource.com/c/1295315 Commit-Queue: Bo <boliu@chromium.org> Reviewed-by:
Changwan Ryu <changwan@chromium.org> Cr-Commit-Position: refs/heads/master@{#604297}
-
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:
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@{#604286}
-
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:
Eugene But <eugenebut@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:
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@{#604273}
-
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:
Ryan Hansberry <hansberry@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:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Nikita Podguzov <nikitapodguzov@google.com> Cr-Commit-Position: refs/heads/master@{#604270}
-
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}
-