- 18 Sep, 2019 40 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/dd0dff1c35d2..e3671013e5a1 git log dd0dff1c35d2..e3671013e5a1 --date=short --no-merges --format='%ad %ae %s' 2019-09-18 jvanverth@google.com Reland "Add ETC2 support to Metal backend." Created with: gclient setdep -r src/third_party/skia@e3671013e5a1 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 brianosman@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=brianosman@google.com Bug: None Change-Id: Ib87a3662c50d4ee2733082ccf257e2d20e732b95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810186Reviewed-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@{#697660}
-
Eric Roman authored
Bug: 1004532 Change-Id: I1a883ddc6c7a0459706afdadf099e6658801e829 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810042Reviewed-by:
Matt Mueller <mattm@chromium.org> Commit-Queue: Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#697659}
-
Jesse Schettler authored
This fixes an issue where the IPC nesting level wasn't being decremented correctly when printing a PDF. Bug: b:140576300 Test: Print a PDF Change-Id: I63579581f2c8a5b9afb231e6057ef8f2f7425ca7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809973Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#697658}
-
Lei Zhang authored
- Change Linux changelog links to https. - Update the links to point to googleblog.com. - Reorder the cases to stable/beta/dev. The updated URLs are short enough to fit on line, so debchange won't cut them off anymore. Bug: 428613 Change-Id: I5aeb151c10fb202ab1f3b77ff4554ed6b486c84e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810458Reviewed-by:
Michael Moss <mmoss@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#697657}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/ad7f2c5c4c7f..36c1c0e22fab git log ad7f2c5c4c7f..36c1c0e22fab --date=short --no-merges --format='%ad %ae %s' 2019-09-17 52076061+digit-google@users.noreply.github.com Fix Fuchsia build. (#2868) 2019-09-16 raun.krisch@gmail.com Adding valilidation checks for OpEntryPoint duplicate names and execution mode (#2862) 2019-09-16 alanbaker@google.com Extra resource interface validation (#2864) 2019-09-13 alanbaker@google.com Split capability tests (#2866) 2019-09-13 alanbaker@google.com SPIRV-Tools support for SPIR-V 1.5 (#2865) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@36c1c0e22fab If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,geofflang@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 TBR=radial-bots+chrome-roll@google.com,geofflang@chromium.org Bug: None Change-Id: I129f542843f54f1de5e396268a59cb5a550dd3e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809984 Commit-Queue: Geoff Lang <geofflang@chromium.org> Reviewed-by:
-
Thomas Tangl authored
Screenshot: https://drive.google.com/file/d/13LQ5jov5tzc__Q1X-xUq-FX9Ptet2i9y/view?usp=sharing Flag: profile-menu-revamp Note: String is subject to change. Bug: 995720 Change-Id: I2658c66897fe899a16355bdced923e32d8c4cf87 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1811117 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#697655}
-
Jun Mukai authored
IconLoader::ClearImage should also be called on removing the icon, otherwise it will return incorrect (empty) image after the app reinstallation. Bug: b/139496595 Test: manually Change-Id: I833adf45bd62ee4f3c64c039ae5fdb7364d04cbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809750Reviewed-by:
Long Cheng <lgcheng@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#697654}
-
Peng Huang authored
Bug: none Change-Id: I987547cfa1a5007ea3e3844afcc6f7c9d8ad9cd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810762 Auto-Submit: Peng Huang <penghuang@chromium.org> Commit-Queue: Jonathan Backer <backer@chromium.org> Reviewed-by:
Jonathan Backer <backer@chromium.org> Cr-Commit-Position: refs/heads/master@{#697653}
-
Deepanjan Roy authored
Change-Id: If88a9f663337ee91aee816459e9f4ae35ebaaaec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810184 Commit-Queue: Annie Sullivan <sullivan@chromium.org> Reviewed-by:
Annie Sullivan <sullivan@chromium.org> Cr-Commit-Position: refs/heads/master@{#697652}
-
Pete Schaffner authored
search-focused "New Search" and "Incognito Search" actions. This replaces the force-touch shortcuts to open new tabs with the omnibox focused. This also updates the current quick action icon set to be more harmonious and in sync with the Material icon set. Bug: 957837 Change-Id: I6f40f98a675b2a92031d40f3f38efbe3d19ff4d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1712873Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Nik Bhagat <nikunjb@chromium.org> Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#697651}
-
Yutaka Hirano authored
This reduces memory copies. Bug: 991501 Change-Id: Icfd1c0c973d52a4528ec6a7364d7616328a882a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809038Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Adam Rice <ricea@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#697650}
-
Sylvain Defresne authored
The ownership of the "observer_" member field of CookieNotificationBridge was unclear and after a casual look at the code, it was unclear whether the code caused a retain cycle or not. Rename the member field, mark it as __strong and add documentation to make it clear that the ownership model is correct, and that there is no retain cycle caused by this object. Bug: 1005153 Change-Id: I8f49e39e8355a0a2c758848ef34c45b4d5335b66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809376 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#697649}
-
Deepanjan Roy authored
It wasn't obvious to me that leaving the description blank means it automatically gets the text "See the link to graphs below." Updating docs. Change-Id: I20b00383e05a7aebea27735c4e14e690817a3f77 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809113Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/859cb8a7e034..8c80da317efc git log 859cb8a7e034..8c80da317efc --date=short --no-merges --format='%ad %ae %s' 2019-09-18 treehugger-gerrit@google.com Merge "perfetto-ui: Fix touch left scroll" 2019-09-18 lalitm@google.com trace_processor: add benchmarks for bitvector methods Created with: gclient setdep -r src/third_party/perfetto@8c80da317efc If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: chromium:135177627 Change-Id: I373c4a4b818e828f02029fb3f1bca0360745810c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810182Reviewed-by:
-
yilkal authored
This CL corrects SupervisedUserNavigationThrottleTest::BlockHost to not overwrite supervised_users::kContentPackManualBehaviorHost settings. Bug: 1003871 Change-Id: I93092b980848c995ffbcb2c204c753e3f5710fd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803901Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Commit-Queue: Yilkal Abe <yilkal@chromium.org> Cr-Commit-Position: refs/heads/master@{#697646}
-
Alex Cooper authored
Inline sessions with options now require user activation or they are rejected. This changes pages with inline sessions to default request an inline session without any options and to request the options in a tap gesture (since Android is currently the only platform that really supports modes other than viewer) Bug: 1005159 Change-Id: Ia6941200d14c29f962f95cd5bfe9b662a9b0eec8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810757 Auto-Submit: Alexander Cooper <alcooper@chromium.org> Commit-Queue: Jacob DeWitt <jacde@chromium.org> Reviewed-by:
Jacob DeWitt <jacde@chromium.org> Cr-Commit-Position: refs/heads/master@{#697645}
-
Nikita Podguzov authored
This helper method is needed to fully complete TestingPrefServiceSimple initialization in unit tests. Bug: 992889 Change-Id: I752b2a3333375177448cbe37e4945e08cfdbbdc8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810917 Commit-Queue: Nikita Podguzov <nikitapodguzov@google.com> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#697644}
-
rbpotter authored
This is the last remaining file in the PDF viewer that is not closure compiled. Bug: 721073 Change-Id: I9093804ffcd961a8c8a2d6e80b6888996f08f4e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808072 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
-
James Cook authored
Settings search relies on the HTML attribute associated-control to show yellow bubbles when a search matches a sub-page. Change the settings-subpage for external storage management to have an associated-control at the top level of the settings page, so the bubble appears on the Device > Storage row for a matching search. This also prevents JS exceptions in the search code (the previous selector didn't match any elements). Test: On a device with Google Play Store support, open OS settings and search for "stor". Verify: JS console has no errors. Search field spinner eventually stops spinning. Yellow bubble appears over the Device > Storage item. Clicking search (X) clear bubble removes all yellow bubbles. Bug: 1003970 Change-Id: I752eccebca599b1275fe505cff7cf561dc7ed66c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808179Reviewed-by: -
Antonio Gomes authored
... down into blink/renderer. CL also moves the following pure virtual classes: network_list_manager.h and network_list_observer.h to the same location, and adds a TODO to verify their existence altogether. BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: Iabba2936c745d028f5388153d5fe0d10ec19a8b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806977 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#697641}
-
Anna Malova authored
for force dark behaviour (force dark vs media query). Rename corresponding methods. Bug: 1001516 Change-Id: I7af3bb835ddd2f9761ffe350e626ef10d7321087 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796673 Commit-Queue: Anna Malova <amalova@chromium.org> Reviewed-by:
Tobias Sargeant <tobiasjs@chromium.org> Cr-Commit-Position: refs/heads/master@{#697640}
-
Niels Möller authored
Intended to ease deletion of default implementations of methods in the abstract PeerConnectionInterface. See webrtc cl https://webrtc-review.googlesource.com/c/src/+/140860 Bug: webrtc:10716 Change-Id: I57d5b72b1650c3f1558773e9e86f1b77831e9f59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807826Reviewed-by:
Mirko Bonadei <mbonadei@chromium.org> Reviewed-by:
Olga Sharonova <olka@chromium.org> Commit-Queue: Niels Möller <nisse@chromium.org> Cr-Commit-Position: refs/heads/master@{#697639}
-
Chinglin Yu authored
Switching to running the perf collector on a dedicated sequence triggers a crash in DCHECK_CALLED_ON_VALID_THREAD when the THREAD_CHECKER is checked on 2 different threads in the same thread pool at different times. Switch to SEQUENCE_CHECKER to fix the crash. Bug: 904785 Test: Turn dcheck_always_on = true and revert https://crrev.com/c/1764319 Verify on DUT that collections doesn't crash the system UI. Change-Id: I9972baa568a2f7bad373777750085a160c6deb23 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810146Reviewed-by:
Gabriel Marin <gmx@chromium.org> Commit-Queue: Gabriel Marin <gmx@chromium.org> Cr-Commit-Position: refs/heads/master@{#697638}
-
Anastasiia Nikolaienko authored
If addition of secondary accounts is disabled, an error page should be displayed instead of Gaia sign-in screen. Bug: 987218 Change-Id: I2954adddf8ece06ca9a0cf355539781e23817d42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1731891Reviewed-by:
Kush Sinha <sinhak@chromium.org> Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#697637}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/61a2765940a7..e70432574670 git log 61a2765940a7..e70432574670 --date=short --no-merges --format='%ad %ae %s' 2019-09-18 chrisforbes@google.com Fix cf_x86_phone boot Created with: gclient setdep -r src/third_party/swiftshader@e70432574670 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 TBR=swiftshader-team+autoroll@google.com Bug: chromium:b/141209513 Change-Id: I61e58792143ddcdc4bb3b7cc18ff03836c6ffb35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810181Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/a84a61e89596..dd281eec08fe Created with: gclient setdep -r src-internal@dd281eec08fe 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: I1cbe5bc91131a709511a8481620cd8a99a53271a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810183Reviewed-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@{#697635}
-
Miyoung Shin authored
This CL converts PublicIpAddressGeolocationProviderPtr in components to the new Mojo type. Bug: 955171 Change-Id: I92b9e37fbdeb105893c45a99952e430e7b11b9c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809198Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Cr-Commit-Position: refs/heads/master@{#697634}
-
Antonio Gomes authored
BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: Id4b213f3a43515b3e501d5af9d9ce11fee5c09fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806976 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
-
Nate Fischer authored
No change to logic. The kDisableMetrics switch doesn't exist. I think the intended switch name was kForceEnableMetricsReporting. Bug: 994418 Test: N/A Change-Id: I89e36fcf5f42225953c40c25ea00addcf1993185 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809980 Auto-Submit: Nate Fischer <ntfschr@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#697632}
-
Gyuyoung Kim authored
Making to use IDS_FOO_BAR instead of WebLocalizedString::kFooBar directly, callers of WebLocalizedString::kFooBar need to include content_strings.h from //content. But, it can cause a layer violation issue. So, we need to move conteont_string.grd to blink in order to make the callers include IDS_FOO_BAR from Blink. Major changes are as below, - Move content_string.grd to third_party/blink/public/strings/ - Renames blink_strings.grd - Change content_ prefix with blink_ prefix. TBR=jamesr@chromium.org, pfeldman@chromium.org Bug: 995644 Change-Id: I8587cb4efd0d6a8d95ff7ad95cd416c3b55f7af2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788968 Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com> Reviewed-by:
Robert Flack <flackr@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
Martin Robinson <mrobinson@igalia.com> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#697631}
-
Fredrik Söderquist authored
In the majority of cases SMILTimeContainer::NotifyIntervalsChanged is already called - or doesn't need to be called either because we're about to perform an update already (Reset()), or because there is no time container to notify anyway (constructor). Bug: 998526 Change-Id: I2e36cbd622a79f3f74f2edf170e6e1d34be3582c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810897Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#697630}
-
Anastasiia Nikolaienko authored
Bug: 1001386 Change-Id: I4082dcdb5968d845c6dca0601c5d5f96a84f17a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805664Reviewed-by:
-
David Benjamin authored
https://boringssl.googlesource.com/boringssl/+log/5ce7022394055e183c12368778d361461fe90a6e..a7d9ac2af4684747c4524cbeba9737b04dce3e3e Bug: none Change-Id: I7580cf38646d98994dad027f8aace2fb07e2a342 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808421 Commit-Queue: Steven Valdez <svaldez@chromium.org> Auto-Submit: David Benjamin <davidben@chromium.org> Reviewed-by:
Steven Valdez <svaldez@chromium.org> Cr-Commit-Position: refs/heads/master@{#697628}
-
Shivani Sharma authored
These tests rely on resources to be cached. Once split cache is enabled by default, they will not be cached if there is no network isolation key in the request. Thus populating the key for these requests. Bug: 989067 Change-Id: Ie0edee408bf74dc17234cbc90b67c6ae2899f585 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806844Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Shivani Sharma <shivanisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#697627}
-
Tim van der Lippe authored
This is a reland of b3373919 Original change's description: > Convert ui/ARIAUtils.js to an ES module > > ES modules allow files to explicitly import and export symbols [1]. > This patch migrates ARIAUtils.js to an ES module, while maintaining > backwards compatibility with existing usages of `window.UI.ARIAUtils`. > > The build system is updated to copy these modules verbatim, as it no > longer require pre-processing to be loaded. The `module.json` no longer > includes this file as a script, to make sure it is not double-bundled > into `shell.js`. Instead, it is added to the new field `modules`, which > contains all modules. By doing so, it becomes clear which files are > converted to modules already and which require the legacy references > to the global scope. > > This module and future modules will be imported from `root.js`, which is > added to all existing HTML application files. > > Design Doc: https://docs.google.com/document/d/1h9dOy3nNPNfZ2AtZXzB-DwJqG4Oo37WWvKLCuzCcPzo/edit?usp=sharing > > [1]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import > > TBR=jochen@chromium.org > > Change-Id: Ie79c8f2fce3aff96fa28af0b575eae39bfe8e1a5 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1748949 > Reviewed-by: Jochen Eisinger <jochen@chromium.org> > Reviewed-by: Yang Guo <yangguo@chromium.org> > Commit-Queue: Tim van der Lippe <tvanderlippe@google.com> > Cr-Commit-Position: refs/heads/master@{#697238} Change-Id: I7544cbf4364d0b1a3c2f137a4d17a1afdb1d4aed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810534Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Tim Van der Lippe <tvanderlippe@chromium.org> Cr-Commit-Position: refs/heads/master@{#697626}
-
Peter Kasting authored
This is a partial revert of https://chromium-review.googlesource.com/c/chromium/src/+/1761666 coupled with a string change discussed over email with srahim/markchang. Bug: 1005025,1005026 Change-Id: I4451f9ff741a958906ce5d76174cafd52eff28a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810033Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#697625}
-
Andrew Grieve authored
Change-Id: Ic616ac3d020b0ec8302451408af459bda73a58d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810185Reviewed-by:
Peter Wen <wnwen@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Auto-Submit: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#697624}
-
Aaron Tagliaboschi authored
Also fix a couple of miscounted constants Bug: 1005220 Change-Id: I4eeebc048594030a1a7e86e730a2b01450c97f1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810802Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Aaron Tagliaboschi <aarontag@chromium.org> Cr-Commit-Position: refs/heads/master@{#697623}
-
Will Harris authored
This CL also ensures that mitigation policy is applied before adding any rules that depend on the policy. It also restores the sandbox convention that mitigation policies and rules can always be requested by a sandbox user and the sandbox will try and enforce the maximum set of rules supported by the running OS. i.e. the user should not be aware of OS capabilities, merely sandbox capabilities. BUG=996834 Change-Id: I5a8432d94dc1bcfbe327589f2b21c4c76fb9fedb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810937Reviewed-by:
James Forshaw <forshaw@chromium.org> Commit-Queue: Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#697622}
-
Antonio Gomes authored
This is a follow up of [1]. [1] https://crrev.com/c/1805159 BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ib3198c792586ebe2671b365d17d46201a07d5aa5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806975 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
-
