- 19 Dec, 2019 40 commits
-
-
John Budorick authored
instead of pool:Chrome-CrOS-VM. Bug: 1011474 Change-Id: I27b1be5840cc4f017eeb96cfc7560a87aab14d74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976553Reviewed-by:
Ben Pastene <bpastene@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#726573}
-
Kelvin Jiang authored
This CL leverages crrev.com/c/1923611 by using the navigation ID within WebRequestInfo to correlate a main-frame navigation request originating from the previously active WebContents to the new WebContents loaded from that request. This is accomplished by adding the actions matched with a request to a pending set of actions for a given navigation ID if the request is associated main-frame navigation. Once the navigation completes, the actions matched for the navigation are added to the badge text and cleared from the pending set. Bug: 992251 Change-Id: Iac7b3502a3598c9d06ab3261ea33f30d2cb20ee7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1956110 Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#726572}
-
Ian Kilpatrick authored
IsLogicalHeightTreatAsAuto -> IsLogicalHeightTreatedAsAuto There should be no behaviour change. Change-Id: I074dba0f468981f901869fa1bb96c098157c7ba3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977059Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Aleks Totic <atotic@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#726571}
-
Ian Kilpatrick authored
There should be no behaviour change. Change-Id: Ic9b22f18358779f9808c7e2aa2f37422e9696936 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976939Reviewed-by:
-
Manuel Rego Casasnovas authored
There was a Mozilla test failing in Chromium because of "will-change: contain" is not creating a stacking context. This has been clarified in the following CSSWG issue: https://github.com/w3c/csswg-drafts/issues/4612 The fix is just adding the "contain" property to HasPropertyThatCreatesStackingContext() in computed_style.cc. BUG=881057 TEST=css/vendor-imports/mozilla/mozilla-central-reftests/contain/contain-paint-stacking-context-001b.html Change-Id: I52a50cfac7bc46a74557760006dc46abba0632ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975913Reviewed-by:
Oriol Brufau <obrufau@igalia.com> Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#726569}
-
Natalie Chouinard authored
Replace usages of java_files with sources in all //chrome/android BUILD.gn files, and automatically format. Also include "sources" in the lists of forwarded variables alongside "java_files" where necessary. Bug: 1035074 Change-Id: I5b570a3a0115993b2ba517362b47a31f64edcc1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973243Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Commit-Queue: Natalie Chouinard <chouinard@chromium.org> Cr-Commit-Position: refs/heads/master@{#726568}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=jsbell@chromium.org Bug: 1023677 Change-Id: Id1edd2e303e6227cfd80aa983a05bf15134e457e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957526 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Joshua Bell <jsbell@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726567}
-
Eric Karl authored
Update the SharedImageBackingFactoryAHB to add lock in the destructor as well as in ProduceLegacyMailbox(). TBR: vikassoni@chromium.org Bug: 1034007, 1003686 Change-Id: I4f6109cdc05f4287ddd45e3e85ec2c03ee15edeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974855 Commit-Queue: Eric Karl <ericrk@chromium.org> Commit-Queue: vikas soni <vikassoni@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#726566}
-
Jorge Lucangeli Obes authored
Metrics were added in https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1958504. Enums were added in https://chromium-review.googlesource.com/c/chromium/src/+/1976310. BUG=985492 TEST=validate_format.py, pretty_print.py. Change-Id: I9ed97d931fbe7877b7d1deb33a5426e9ef8c4011 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976436 Commit-Queue: Jorge Lucangeli Obes <jorgelo@chromium.org> Reviewed-by:
Christopher Thompson <cthomp@chromium.org> Cr-Commit-Position: refs/heads/master@{#726565}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 4bce4c14. With Chromium commits locally applied on WPT: c15c1083 "Add test that datachannels close when connection does." 9899425d "[css-layout-api] IntrinsicSizes Implementation" d75fede5 "[EventTiming] Fix crossiframe test" 93b03639 "Roll WebGPU CTS" 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 NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Ia745a7edc3344d0492ba8901bbe7a8924bbcafcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976844Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#726564}
-
Torne (Richard Coles) authored
When we're on O+ and loading from the WebView implementation, we can use the WebView's declared sandboxed service processes instead of the client's, which will cause them to be forked from the WebView zygote, saving some startup time and memory. Fixed: 1025602 Change-Id: I902cc7bd5685ebddfa4681d7d87bb45fc8330dbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974840 Commit-Queue: Richard Coles <torne@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Auto-Submit: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#726563}
-
Kent Tamura authored
... if attribute's owner element is an HTML element and its owner document is an HTML document and it has no namespaceURI. The new behavior matches to Firefox. Bug: 179453 Change-Id: I499f809c8ad2620da2ef2c78fe6d301feec6f67b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975494Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#726562}
-
Bruce Dawson authored
The feature flag that I added to control whether the Windows timer interval on DC power should be 4 ms or 8 ms had a race condition in that we might query the feature flag before the feature-flag system was configured. This change fixes that by adding the ReadMinTimerIntervalLowResMs function which is called after the feature system is initialized. While working on this I realized that ActivateHighResolutionTimer and EnableHighResolutionTimer were extremely subtle so I simplified them. When calling timeBeginPeriod you *must* eventually call timeEndPeriod to undo your request, and you must call it with exactly the same value. With the raised timer frequency now having three possible values (1, 4, or 8 ms) the logic was getting far too messy so I now do the simpler task of just recording the value passed to timeBeginPeriod and using it in timeEndPeriod. This is more obviously correct. The new rule is that timeBeginPeriod and timeEndPeriod should always be called with g_last_interval_requested as the parameter. I also tidied up and expanded on some comments - I always get confused by the purpose and semantics of ActivateHighResolutionTimer and EnableHighResolutionTimer. I tested with --disable-highres-timer, with and without --enable-features=SlowDCTimerInterruptsWin, using trace_timer_intervals.bat to confirm that all Chrome processes were lower the timer interrupt setting to 4 ms or 8 ms as requested. Bug: 1033108, 927165 Change-Id: I69b9253b456d7e3273ea1823a61c20926b0a4510 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1965846Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Cr-Commit-Position: refs/heads/master@{#726561}
-
Dominic Mazzoni authored
Blink keeps track of a 32-bit integer DOMNodeId for every node in the DOM, when needed. This is already used for the content capture API and for DevTools. Add an option to include these IDs in an accessibility tree, too. Note that this has nothing to do with the ID attribute that an HTML element can have. This ID is for any node and it's internal to Blink. It's a bit confusing. This will be used in order to implement support for exporting tagged PDFs in http://crrev.com/c/1970742 Bug: 607777 Change-Id: I5a7838de4b2a74f86cb68cc9989491d3faf22ce7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970582 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#726560}
-
John Budorick authored
instead of pool:Chrome-GPU. Bug: 1011474 Change-Id: If024ceb3b14914636e762191576321c62a4b7673 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976430Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/a892219ced42..05934953bfd9 git log a892219ced42..05934953bfd9 --date=short --first-parent --format='%ad %ae %s' 2019-12-19 ehmaldonado@google.com gclient: Make smoketests run on Windows. Created with: gclient setdep -r src/third_party/depot_tools@05934953bfd9 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/+/master/autoroll/README.md Bug: chromium:1024683 Tbr: agable@chromium.org Change-Id: Ib126200ff9446ee5b99f2d937feea4afc2c7a888 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976804Reviewed-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@{#726558}
-
Kent Tamura authored
It should handle only XML white spaces; #x20, #x9, #xD, and #xA. We incorrectly handled other white spaces such as U+3000. Bug: 893929 Change-Id: I59708f978ba1c23141f748de7a8f4c58d58c7acf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974713 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=dougarnett@chromium.org Bug: 1023677 Change-Id: I947641610097a19e4c4f1db6c2eca32045869f46 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957492 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726556}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/6d3c7f5526ec..750660e15c3c git log 6d3c7f5526ec..750660e15c3c --date=short --first-parent --format='%ad %ae %s' 2019-12-19 bclayton@google.com Kokoro: Add style presubmit checks Created with: gclient setdep -r src/third_party/swiftshader@750660e15c3c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@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_chromium_msan_rel_ng;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: swiftshader-team+autoroll@google.com Change-Id: I8a5df0f3e970b258e534ffcdf8ce26f1733a13bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976806Reviewed-by:
-
Markus Handell authored
This change fixes omitted propagation of changes in - ready state - enabled state - content hint state to encoded sinks. Bug: 1013590 Change-Id: I97f73bf2b2732b2411779ea7a0d7a56c2fccef57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970553 Commit-Queue: Markus Handell <handellm@google.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#726554}
-
manukh authored
Previous version broke in python3 with the message: 'NameError: name 'unichr' is not defined'. Change-Id: Ia32f7a3b6e2848e72d29ca8126021febbd4389b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976828 Commit-Queue: manuk hovanesian <manukh@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#726553}
-
Michael Forney authored
This allows the builder to specify the path of the ninja binary in case they want to use a specific version, or an alternate implementation like samurai[0]. Also, when calling ninja, make sure build target comes after any options. [0] https://github.com/michaelforney/samurai Change-Id: I4295baa9e0e04e61ae092a9b86ea6ddc943b3c92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975453Reviewed-by:
Nico Weber <thakis@chromium.org> Auto-Submit: Michael Forney <mforney@mforney.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#726552}
-
Rakib M. Hasan authored
Bug: 986447 Change-Id: I4b5989c23643defc10ce52a329f326cdaaf28ac8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975864 Commit-Queue: Rakib Hasan <rmhasan@google.com> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#726551}
-
zhaoyangli authored
Added sync after some ChromeEarlGrey operations where a UI change in app might be triggered. This should eliminate undesirable racing conditions before UI change finishes at app. TabUsageRecorderTestCase is an example and this CL fixes it. Bug: 1034196, 1033879 Change-Id: Ib260414371b832eed517d8bf72b2ee0c10f5cfd7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970914 Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#726550}
-
Xiaoqian Dai authored
Turns out the bubble dialog native window is included in the MRU window list, which seems a bug - filed a separate bug for it crbug.com/1035998. The fix in this CL still makes sense though. Bug: 1034653 Change-Id: I079f2256631fdd39f7102a61bd81ea4a6e88a9f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975086Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#726549}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=shivanisha@chromium.org Bug: 1023677 Change-Id: I5c0076283f6820c7f65d307a6f5280630093afc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957532 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Shivani Sharma <shivanisha@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726548}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/addc9bfc6cd1..1c9b1c07affb git log addc9bfc6cd1..1c9b1c07affb --date=short --first-parent --format='%ad %ae %s' 2019-12-19 thestig@chromium.org Fix bitmap size calculation in EmbedderTest::HashBitmap(). 2019-12-19 tsepez@chromium.org De-virtualize CFFL_FormFiller::GetFocusBox(). 2019-12-19 tsepez@chromium.org Remove unused argument from CFFL_FormFiller::WndtoPWL() and FFFLtoWND(). Created with: gclient setdep -r src/third_party/pdfium@1c9b1c07affb If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pdfium-autoroll Please CC pdfium-deps-rolls@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 Bug: None Tbr: pdfium-deps-rolls@chromium.org Change-Id: Iceb1f7a33f395f17bd210f8701a82b3f18940005 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976805Reviewed-by:
-
manukh authored
- Replaced 'Building, Running, and Debugging' section - Previously, this section was incomplete and only described how to build chromium. - Added instructions to run and visually debug as well. - Updated build instructions to support debugging. - Simplified recommended CMakeLists.txt file - Simplified instructions for creating a desktop entry - Increased recommended VM option Xmx from 5g to 10g - Marked optional steps as optional - Punctuation fixes - Added link to clion.md to docs/README.md Change-Id: Id0514b1498517e0c10fafb1a9639aeda44717921 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976280 Commit-Queue: manuk hovanesian <manukh@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#726546}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=shivanisha@chromium.org Bug: 1023677 Change-Id: I4ecb846b64a36236013f36b7d16a663c134f3412 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1955082 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
-
rbpotter authored
Size comparison: Polymer 2: app.crisper.js: 142k app.vulcanized.html: 90k lazy_load.crisper.js: 30k lazy_load.vulcanized.html: 26k Total: 288k Polymer 3: history.rollup.js: 108k lazy_load.rollup.js: 55k shared.rollup.js: 125k Total: 288k Note that Polymer 2 version also pulls in 2 non-generated files (constants.*, around 2k) that are bundled in Polymer 3. Also compared load times, and found that the additional JS import of shared.rollup.js does not slow down page load; Polymer 3 version of the page appears to load 3-4% faster. Bug: 1022212 Change-Id: I0d32864436390b9226bd05abfb6af4cbbf40de5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975080Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#726544}
-
Peter Qiu authored
We're currently using GenericPendingReceiver::interface_name() when forwarding the bind request to the system connector, which is returning a reference string. However, the |interfae_name_| is resetted in GenericPendingReceiver::PassPipe(), which is invoked in the same bind request. This results in system connector getting corrupted interface name. So make a copy of the interface name, and use it for the bind request. Bug: 146448872 Test: Verify no crash when CastAudioOutputStream uses ServiceConnector to request MultiroomManager interface. Change-Id: Ia6987045e4c6ab39f46fe097fb98e6dfa5e00468 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977023 Commit-Queue: Peter Qiu <zqiu@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Reviewed-by:Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#726543}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=zmo@chromium.org Bug: 1023677 Change-Id: Idd0b624a2f277dab6dc1b900e315fc3d4966df1b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1954579 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726542}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=bsazonov@chromium.org Bug: 1023677 Change-Id: Id4f3e848e4d7dd6e30fad14238fcfa0313a29b7b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957497 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726541}
-
Albert Chaulk authored
Cropping with a SurfaceDrawQuad is done a little differently than with a texture quad, it need to be scaled up and positioned in the target space and then clipped using the clip_rect. Bug: b/143612326 Test: manually Change-Id: I38e78fd5ff4c60df2c44f9054136d8b27ac55ddc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972259Reviewed-by:
Alex Sakhartchouk <alexst@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Commit-Queue: Albert Chaulk <achaulk@chromium.org> Cr-Commit-Position: refs/heads/master@{#726540}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=pkotwicz@chromium.org Bug: 1023677 Change-Id: I1ad33980e6f5e05694ef1ec080049b5043078112 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957544 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Peter Kotwicz <pkotwicz@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726539}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/a509bcf2af35..5acad7aa14e2 Created with: gclient setdep -r src-internal@5acad7aa14e2 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 Bug: None Tbr: jbudorick@google.com Change-Id: I0406d771ab807bf5b590dc8fd1dd995f1fc38dfd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976664Reviewed-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@{#726538}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/0340c945..1ba8de1a Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I12900322ec49bf86dcc9db3a8572dd7adb1e2a02 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976840Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#726537}
-
Olivier Li authored
This test had brittle behavior that was hidden by the way HistoryService used to be coded. The feature will soon be flipped to ENABLED_BY_DEFAULT. This will make HistoryServiceBackend's task runner dispatch to the ThreadPool instead of a dedicated base::Thread. The tests need to be prepared to stay functional when that happens. This is done as the last mile of the chrome-wide effort to coalesce as much work as possible to the ThreadPool. Bug: 661143 Change-Id: I4f3bae453c93454eb8cecc6cd3046cfd07958bfe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976150 Auto-Submit: Oliver Li <olivierli@chromium.org> Commit-Queue: Brandon Tolsch <btolsch@chromium.org> Reviewed-by:
Brandon Tolsch <btolsch@chromium.org> Cr-Commit-Position: refs/heads/master@{#726536}
-
Mikel Astiz authored
The API is refactored to isolate the flow for incoming encryption keys. This better separates the API from other functions that are intended to be used by actual UI (e.g. user-entered passphrases), and sets the foundation for triggering fetches initiated by the TrustedVaultClient implementation itself (useful on Android). There is a behavioral difference as a result of rewriting the logic in SyncServiceCrypto, which is arguably desirable: trusted vault keys are not propagated to the sync engine (and hence the Nigori keybag unless there is an actual need to do so, which excludes, most importantly, users that are not in the relevant passphrase type). Bug: 1012659 Change-Id: I009e02ebd0370eb98e95c50368299b0bf947cfb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973823 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#726535}
-
Joshua Peraza authored
b411976ca5f4 [log minidump] add tool to encode/decode minidump log 38f4bae3ad84 chromeos: fix minidump-dir-for-tests flag Change-Id: Ibd0d5fc15aa286951de9da2f3e923b45ad44fe13 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976831 Commit-Queue: Joshua Peraza <jperaza@chromium.org> Commit-Queue: Mark Mentovai <mark@chromium.org> Auto-Submit: Joshua Peraza <jperaza@chromium.org> Reviewed-by:
Mark Mentovai <mark@chromium.org> Cr-Commit-Position: refs/heads/master@{#726534}
-
