- 11 Dec, 2018 40 commits
-
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/9a1b0e72..72448bed Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I1b37dea72dce8da3904aa7c6f490b8a0ad88f78a Reviewed-on: https://chromium-review.googlesource.com/c/1372024Reviewed-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@{#615548}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/152255fcc745..c8b5fd3f2752 git log 152255fcc745..c8b5fd3f2752 --date=short --no-merges --format='%ad %ae %s' 2018-12-11 lalitm@google.com Merge "perfetto: fix tracing test on Android ASAN" Created with: gclient setdep -r src/third_party/perfetto@c8b5fd3f2752 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: I079a8f7540ce6d1ac814a6d1c64ad0748d1f10bb Reviewed-on: https://chromium-review.googlesource.com/c/1372085Reviewed-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@{#615547}
-
Josh Karlin authored
Change-Id: I21b6ea7ddeb722bf91ce184c9d21c35baedf8749 Reviewed-on: https://chromium-review.googlesource.com/c/1372046Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Commit-Queue: Josh Karlin <jkarlin@chromium.org> Cr-Commit-Position: refs/heads/master@{#615546}
-
Friedrich Horschig [CET] authored
This reverts commit 9ef757a8. Reason for revert: is again needed after https://chromium-review.googlesource.com/c/chromium/src/+/1362457 was reverted Original change's description: > Revert "Make //third_party/fontconfig explicitly depend on //third_party:freetype_harfbuzz." > > This reverts commit 0baa5bdf. > > Reason for revert: is not needed after https://chromium-review.googlesource.com/c/chromium/src/+/1362457 > > Original change's description: > > Make //third_party/fontconfig explicitly depend on //third_party:freetype_harfbuzz. > > > > Bug: 911836 > > Change-Id: Ia76687b3cb8d329792e5ef0310faee93d05344f2 > > Reviewed-on: https://chromium-review.googlesource.com/c/1361725 > > Reviewed-by: Thomas Anderson <thomasanderson@chromium.org> > > Commit-Queue: Max Moroz <mmoroz@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#613792} > > TBR=bungeman@chromium.org,mmoroz@chromium.org,thomasanderson@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 911836 > Change-Id: Ibc4c88faadd5c1e603fc0f26ce956c282909798e > Reviewed-on: https://chromium-review.googlesource.com/c/1365083 > Reviewed-by: Max Moroz <mmoroz@chromium.org> > Reviewed-by: Thomas Anderson <thomasanderson@chromium.org> > Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> > Cr-Commit-Position: refs/heads/master@{#614534} TBR=bungeman@chromium.org,mmoroz@chromium.org,thomasanderson@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 911836 Change-Id: Id09d7c0394e8c5eb445aa19ba5a62f8fa251ecb0 Reviewed-on: https://chromium-review.googlesource.com/c/1371885Reviewed-by:
Friedrich Horschig [CET] <fhorschig@chromium.org> Reviewed-by:
Max Moroz <mmoroz@chromium.org> Commit-Queue: Friedrich Horschig [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#615545}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/8f66ddbae325..e011cb742d99 git log 8f66ddbae325..e011cb742d99 --date=short --no-merges --format='%ad %ae %s' 2018-12-11 terelius@webrtc.org Move chart proto for event_log_visualizer. 2018-12-11 nisse@webrtc.org Rename EncodedImage::_size to capacity_, make private. 2018-12-11 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision aec72e9e..d5698f68 (615246:615355) Created with: gclient setdep -r src/third_party/webrtc@e011cb742d99 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,chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I5a31afacdbc38767ae24412a821ba8738974fbcc Reviewed-on: https://chromium-review.googlesource.com/c/1371711Reviewed-by:
-
Dan Elphick authored
Replaces many deprecated uses of v8 functions Object::Get/Set, Value::ToString, in extensions/renderer with versions that return Maybe/MaybeLocals or by using casts if there are already checks that ensure the cast will always succeed. Bug: v8:7279, v8:7283, v8:8562, chromium:913942 Change-Id: I4aeec76251bd11dfc274156d8d00586e04e9a99e Reviewed-on: https://chromium-review.googlesource.com/c/1353985 Commit-Queue: Dan Elphick <delphick@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#615543}
-
Nico Weber authored
chrome_nibs.gyp went away a while ago, and gyp is no longer deps'd in. Bug: none Change-Id: I71d7503b0f95ff15575653da7efe35e4e62a6fcd Reviewed-on: https://chromium-review.googlesource.com/c/1371928Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#615542}
-
Michael Ludwig authored
Bug: 913948 Change-Id: I2712119f0ff48a56d1a8b994dc470e76b37250d2 Reviewed-on: https://chromium-review.googlesource.com/c/1371925Reviewed-by:
Brian Salomon <bsalomon@google.com> Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#615541}
-
Ned Nguyen authored
Bug:913625 Change-Id: Id6751f4af3aedac41fd3968a58a797b785cfbd84 TBR=jbudorick@chromium.org, bradhall@chromium.org NOTRY=true # linux_chromium_rel_ng flake Change-Id: Id6751f4af3aedac41fd3968a58a797b785cfbd84 Reviewed-on: https://chromium-review.googlesource.com/c/1371646 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#615540}
-
John Chen authored
Make Add Cookie command compliant to W3C spec. Also added a set of helper functions to help parsing some property types used in W3C spec. ChromeDriver now passes all WPT WebDriver tests related to cookie. Bug: chromedriver:2002 Change-Id: I3779a14e010bca865498a9d40710974e38ede829 Reviewed-on: https://chromium-review.googlesource.com/c/1366925 Commit-Queue: John Chen <johnchen@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#615539}
-
Florian Uunk authored
The AutofillProfileDataTypeController waits for PDM to load all its data before doing data association. However, with Butter, PDM needs to ask the sync service whether the sync feature is enabled to know whether to read the server cards from the local or account database. Currently, PDM only gets the sync service whenever we create an AutofillManager, which isn't always the case on startup. This CL adds a quick fix, where we also pass the sync service to PDM in the AutofillProfileDataTypeController, so we know for sure it can load all data. A better fix would be to have a single place where we pass the sync service to PDM, and remove both the call in AutofillManager and this new call. Bug: 913945 Change-Id: Ia872cb8ce0cb97467237665e5a92b1d70eb1c2bc Reviewed-on: https://chromium-review.googlesource.com/c/1371427 Commit-Queue: Florian Uunk <feuunk@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#615538}
-
Owen Min authored
Disable this data upload as it's not used by server. Change-Id: I7da2e3b82aef19318a3ed5ec6c00f2cabe15fb2b Reviewed-on: https://chromium-review.googlesource.com/c/1370723Reviewed-by:
Marc-André Decoste <mad@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#615537}
-
Jeremy Roman authored
These are the ones that didn't explicitly include wtf/atomics.h and so were missed earlier. Bug: 736037,247328 Change-Id: I407d1ad6f53c058866bb4a98ca910768fe9ce6c5 Reviewed-on: https://chromium-review.googlesource.com/c/1370187Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Commit-Queue: Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#615536}
-
Stephen McGruer authored
This CL adds a README.md to the fast/animations/scroll-animations directory, explaining when that directory should be used vs external/wpt/scroll-animations (aka, rarely). Bug: 911254 Change-Id: Id24ee770dcbe070b1f89ffe925edbacbba4a673c TBR=yigu@chromium.org Change-Id: Id24ee770dcbe070b1f89ffe925edbacbba4a673c Reviewed-on: https://chromium-review.googlesource.com/c/1367825Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Reviewed-by:
Yi Gu <yigu@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#615535}
-
Jesse Doherty authored
There will need to be a follow up CL that adds an implementation for Chrome to provide an actual key. Bug: 906202 Change-Id: Ib471b1124e19b11755b410f384a447f929a1b547 Reviewed-on: https://chromium-review.googlesource.com/c/1351709 Commit-Queue: Jesse Doherty <jwd@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Cr-Commit-Position: refs/heads/master@{#615534}
-
Xiyuan Xia authored
ash::Shell owned by ChromeBrowserMainExtraPartsAsh is released before UiDevToolsServer owned by ChromeBrowserMainExtraPartsViews. This causes a use-after-free in DOMAgentAura dtor when it tries to RemoveObserver from the already released Env of ash::Shell. Fix the issue by explicitly releasing |devtools_server_| in ChromeBrowserMainExtraPartsViews::PostMainMessageLoopRun. Bug: 896977 Change-Id: I07172d8014dc17c1a91a11e220579a13ad0524b0 Reviewed-on: https://chromium-review.googlesource.com/c/1370832Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#615533}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 3e8e3c9e. 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: foolip@chromium.org, robertma@chromium.org, smcgruer@chromium.org: external/wpt/infrastructure NOAUTOREVERT=true TBR=markdittmer No-Export: true Change-Id: I5349c5b8538c4f94cb3341c878c95cbc7b3a73b7 Reviewed-on: https://chromium-review.googlesource.com/c/1371647Reviewed-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@{#615532}
-
John Budorick authored
Bug: 731964,913687 Change-Id: Ie7de1a0aba1316b8916219afb766b18e95dc5549 Reviewed-on: https://chromium-review.googlesource.com/c/1372026Reviewed-by:
Robert Ma <robertma@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#615531}
-
Li Hao authored
Sometimes we expect to build mac target without v8, but there is a warning that v8_current_cpu is set as a build argument, but not appeared in declare_args(). If the declaration file v8_target_cpu.gni is not imported, we should not override it. Change-Id: I04ef533a565fb9272aa1117ccea1bdee022f5c9a Reviewed-on: https://chromium-review.googlesource.com/c/1369645 Commit-Queue: Robert Sesek <rsesek@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
-
Dave Tapuska authored
CSS Font Loading spec refers a font-loading task queue. Add a queue matching that name and update it to use that queue. Change-Id: I885962521f2a17f3b35b181d3dc1066cc86a4e6e Reviewed-on: https://chromium-review.googlesource.com/c/1370707 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#615529}
-
Sami Kyostila authored
This patch fixes a problem in the Mac message pump where we kept calling Delegate::DoDelayedWork even if a previous call to Delegate::DoWork caused the runloop to quit. This matches the behavior outlined in message_pump.h[1] as well as that of the other platform-specific pumps. We also introduce a message pump unit test that verifies behavior across all message pump types. Bug: 891670 [1] https://cs.chromium.org/chromium/src/base/message_loop/message_pump.h?rcl=a28a6097182e739c4a8c600b538fcc07681fc7a3&l=58 Change-Id: I00ccb5c55ac8551259a2febd343ec64fdd55630d Reviewed-on: https://chromium-review.googlesource.com/c/1370367Reviewed-by:
Mark Mentovai <mark@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#615528}
-
David 'Digit' Turner authored
This CL fixes compile_resources.py to ensure that it correctly includes missing localized strings into the final APK. The problem was that some android_resources() targets declared some Filipino resources under values-fil/ directories, and some Indonesian ones under values-in/ 'fil' is a 3-letter ISO 639-2 language code, that was not supported before Android 5.0/Lollipop. Hence, our script only tried to include strings from 2-letter locale qualifiers, excluding the -fil one entirely. Due to the problem above, most packages place their localized strings for Filipino are placed under 'values-tl/' instead, because 'tl' matches Tagalog, a related but lesser spoken language in the Philippines. Except a few packages that still use values-fil/, for examples see [2]. 'in' is a new 2-letter ISO 639-2 language code, but due to compatibility concerns, our script wants to use 'id' instead for the Indonesian language (see [1] for a little bit more details). A few packages provide their localized strings for Indonesian under values-in/, which was ignored [3]. This CL fixes the issue by renaming "newer" locale qualifiers to "older" ones. This is done by extracting the qualifier from values-<locale>, converting it to its Chromium version, then back to its Android version. In practice it also deals with BCP 47 codes, as in: values-fil/ -> values-tl/ values-in/ -> values-id/ values-b+en+US -> values-en-rUS/ The last example above is not relevant, but opens the door to supporting more locales in Chromium in the future. The end result is adding missing localized strings for Filipino and Indonesian for 30 string resources (listed below). This ends up increasing the size of MonochromePublic.apk by about 2000 bytes. BUG=921044 R=agrieve@chromium.org,estevenson@chromium.org,yfriedman@chromium.org,torne@chromium.org [1] https://developer.android.com/reference/java/util/Locale The section in the Locale() constructor that reads: * ISO 639 is not a stable standard; some of the language codes it defines (specifically "iw", "ji", and "in") have changed. This constructor accepts both the old codes ("iw", "ji", and "in") and the new codes ("he", "yi", and "id"), but all other API on Locale will return only the OLD codes. Older Android system releases could only read compiled resources from obsolete config locale qualifier (e.g. 'in' for Indonesian). [2] https://cs.chromium.org/chromium/src/third_party/feed/src/src/main/java/com/google/android/libraries/feed/basicstream/internal/viewholders/res/?q=values-fil&sq=package:chromium&dr [2] https://cs.chromium.org/search/?q=values-fil/&sq=package:chromium&type=cs [3] https://cs.chromium.org/search/?q=values-id/&sq=package:chromium&type=cs [4] List of affected string resources: back_button_content_description cancel_button dialog_button_got_it dialog_button_open_help_center dialog_message_incompatible_phone dialog_message_no_cardboard dialog_title dialog_title_incompatible_phone dialog_title_vr_core_not_enabled dialog_title_vr_core_not_installed dialog_title_warning dialog_vr_core_not_enabled dialog_vr_core_not_installed go_to_playstore_button go_to_vr_listeners_settings_button more_button_label no_browser_text no_suggestions place_your_phone_into_cardboard place_your_viewer_into_viewer_format settings_button_content_description setup_button snackbar_fetch_failed snackbar_fetch_no_new_suggestions suggestions_section_empty switch_viewer_action switch_viewer_prompt zero_state_text_afternoon zero_state_text_evening zero_state_text_morning To obtain this list, process the unoptimized resources archives (where resource names are not obfuscated) as follows: aapt dump resources \ out/Release/obj/chrome/android/monochrome_public_apk.unoptimized.ap_ | \ sed -e 's/d=0x[0-9a-f]*//g' > /tmp/A/Resources.txt Do this for the build before and after this CL is applied and compare the results. The 'sed' part is used to get rid of irrelevant differences in each resource allocation. Change-Id: I4a7d961d68de381106347072a69eda6751d1bf7f Reviewed-on: https://chromium-review.googlesource.com/c/1367730 Commit-Queue: David Turner <digit@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#615527}
-
Justin DeWitt authored
This causes a lint error on the official builders. Bug: 894334,913755,913012 Change-Id: I9dbaa9932873755639d22618b3d758ef87c840cb Reviewed-on: https://chromium-review.googlesource.com/c/1370587Reviewed-by:
Cathy Li <chili@chromium.org> Commit-Queue: Justin DeWitt <dewittj@chromium.org> Cr-Commit-Position: refs/heads/master@{#615526}
-
Oleh Prypin authored
The Android test runner, when retries were enabled, previously would overwrite the isolated-script-test-perf-output JSON file with the latest try, which means that all previous results would be lost. Now it will still write to the provided 'file-name.json' first, but then for following tries it will write to 'file-name_1.json', 'file-name_2.json', etc., which can allow to salvage the results in post-processing. Unrelated change in behavior due to refactoring: --app-data-file previously used this naming scheme for outputs: ['file-name_0.ext', 'file-name_1.ext', ...]; now it's ['file-name.ext', 'file-name_1.ext', ...]. Bug: 755660 Change-Id: Id8ecb506ef91504b925178b433544faa3db8c476 Reviewed-on: https://chromium-review.googlesource.com/c/1371426 Commit-Queue: Oleh Prypin <oprypin@chromium.org> Reviewed-by:
-
Alex Clarke authored
As a temporary measure add an aliased variable to record the size of the queue. Bug: 912541 Change-Id: I874fcc7cb463b3b891fbe875d42539e72d771767 Reviewed-on: https://chromium-review.googlesource.com/c/1371809Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#615524}
-
Maxim Kolosovskiy authored
According to metrics, 0.3% of forms have >100 fields (the current limit), increase the limit to 250 in order to cover all forms. Autofill throttling limits the number of queries/uploads are sent for a given form by a given user. Therefore, uploading huge forms wouldn't flood network. Bug: 857454 Change-Id: Ia92a3d6f2033159df121d8728a263c70ee7914f7 Reviewed-on: https://chromium-review.googlesource.com/c/1369772Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Commit-Queue: Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#615523}
-
Friedrich Horschig [CET] authored
This test is consistently failing on Mac since the last wpt roll. No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 913932, 626703 TBR=robertma@chromium.org Change-Id: I071d68a9003edecf4b24b10b4adfe1c86cf7da61 Reviewed-on: https://chromium-review.googlesource.com/c/1371904 Commit-Queue: Friedrich Horschig [CET] <fhorschig@chromium.org> Reviewed-by:
-
Tim Schumann authored
Change-Id: I1b762a554913d0e3b3df0792c2ff845c4dde5a53 Reviewed-on: https://chromium-review.googlesource.com/c/1370365Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#615521}
-
Friedrich Horschig [CET] authored
This reverts commit d1317fc3. Reason for revert: SyncSchedulerImplTest is failing and this is the most likely culprit. More information in the linked bug. Bug: 913946 Original change's description: > [Reland #3] Use the SequenceManager in ScopedTaskEnvironment > > A reland of https://crrev.com/c/1324391 > > This is necessary because we want content::TestBrowserThreadBundle to > own a BrowserUIThreadScheduler, but that also owns a ScopedTaskEnvironment > and you can't have two SequenceManagers on the same thread. > > This patch allows ScopedTaskEnvironment to optionally work with an > externally owned SequenceManager solving the problem. > > This implements https://docs.google.com/document/d/1y08C6JQ9Yta3EQXzwIqqIIKHq9500WV6CWFZzZfDx7I/edit?usp=drivesdk, > > We now have the ability to mock time on the UI and IO threads. > > RE HttpServiceTest change HttpServiceTest.MultipleRequests was failing > mysteriously on fuschia on the bots only (not locally). IO_MOCK_TIME seems > to have fixed this. > > > TBR=gab@chromium.org,fdoray@chromium.org,wez@chromium.org > > Bug: 863341, 891670, 708584 > Change-Id: I95444a0a50f8b577c5fae62c12c8423c7e8f21d6 > Reviewed-on: https://chromium-review.googlesource.com/c/1361863 > Commit-Queue: Alex Clarke <alexclarke@chromium.org> > Reviewed-by: Alex Clarke <alexclarke@chromium.org> > Reviewed-by: Sami Kyöstilä <skyostil@chromium.org> > Cr-Commit-Position: refs/heads/master@{#615496} TBR=wez@chromium.org,gab@chromium.org,fdoray@chromium.org,skyostil@chromium.org,alexclarke@chromium.org Change-Id: If5ff0362758b5270eb3113d2fd6eeeb7facc37c3 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 863341, 891670, 708584 Reviewed-on: https://chromium-review.googlesource.com/c/1371811Reviewed-by:
-
Andrew Grieve authored
These classes need to be the same across releases. Bug: 913116 Change-Id: I57dc8928f24944f0234863f5fcfc248886c6a7cc Reviewed-on: https://chromium-review.googlesource.com/c/1371244Reviewed-by:
Filip Gorski <fgorski@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#615519}
-
Florent Castelli authored
Intent: https://groups.google.com/a/chromium.org/d/msg/blink-dev/LXJUzctTVfM/p4J-WrzbBgAJ Bug: 908911 Change-Id: I8e82cc588fb991458e2372fe1dcb9db459cd36b5 Reviewed-on: https://chromium-review.googlesource.com/c/1363276Reviewed-by:
Henrik Boström <hbos@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Commit-Queue: Florent Castelli <orphis@chromium.org> Cr-Commit-Position: refs/heads/master@{#615518}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/3c2ff12d419d..152255fcc745 git log 3c2ff12d419d..152255fcc745 --date=short --no-merges --format='%ad %ae %s' 2018-12-11 fmayer@google.com Fix test failure on is_debug. Created with: gclient setdep -r src/third_party/perfetto@152255fcc745 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: If77220fecdba346d3506e552c69fa9d8ed3ffeb8 Reviewed-on: https://chromium-review.googlesource.com/c/1371709Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/757effd70e5f..84629852e33d Created with: gclient setdep -r src-internal@84629852e33d 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: Ic04ded354bf2c823a6c08eec6163ed351980325c Reviewed-on: https://chromium-review.googlesource.com/c/1371708Reviewed-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@{#615516}
-
Rayan Kanso authored
The handler uses the side data blob to store the request body if any. Bug: 774054 Change-Id: I14ec3e992543937bef7a601bcb603103842c6acf Reviewed-on: https://chromium-review.googlesource.com/c/1363205 Commit-Queue: Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Reviewed-by:
Ben Kelly <wanderview@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#615515}
-
Joe Mason authored
The logo now has a title attr in it for screen readers. But when it's set as the background of a <span>, or as an <img> src, screen readers don't look at the tags inside. For the screen reader to view the tag it seems the <svg> element must be part of the DOM tree. So after the <svg> element is loaded from a resource, add it directly to the page. R=hcarmona@chromium.org Bug: 767130 Change-Id: I1de73057ba59df114fdb142baab9fc66f210995f Reviewed-on: https://chromium-review.googlesource.com/c/1370833Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Joe Mason <joenotcharles@google.com> Cr-Commit-Position: refs/heads/master@{#615514}
-
Marc Treib authored
Bug: none Change-Id: I8f0360e7f3ff166aa321a9041b6dd5865dcf5988 Reviewed-on: https://chromium-review.googlesource.com/c/1371425 Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
-
Friedrich Horschig [CET] authored
This reverts commit 05142837. Reason for revert: Added test flakes on most Mac versions. Bug: 913939 Original change's description: > Add autofill_profile integration test for UMA Sync.ModelTypeEntityChange3 > > The new test is similar to the existing PersonalDataManagerSanity test, > but adds checks for the Sync.ModelTypeEntityChange3.AUTOFILL_PROFILE > histogram. > Compared to PersonalDataManagerSanity, the new test skips the step of > adding a duplicate profile, since that seems to produce flaky results. > > Bug: 904390 > Change-Id: Ic85050dcab80e53ea1a5e1fa63f61db5e6c2cb09 > Reviewed-on: https://chromium-review.googlesource.com/c/1369778 > Reviewed-by: Jan Krcal <jkrcal@chromium.org> > Commit-Queue: Marc Treib <treib@chromium.org> > Cr-Commit-Position: refs/heads/master@{#615461} TBR=treib@chromium.org,jkrcal@chromium.org Change-Id: I1da849dfe70d5523b8d211d0d6d592c101672456 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 904390 Reviewed-on: https://chromium-review.googlesource.com/c/1371888Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/ff281cc4..9a1b0e72 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Id3e47c4585c77e1d6a0a9a4a259a13889ec47031 Reviewed-on: https://chromium-review.googlesource.com/c/1371645Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium 447dc435. 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: rego@igalia.com: external/wpt/css/selectors NOAUTOREVERT=true TBR=markdittmer No-Export: true Change-Id: I42c4d57343eac3172b9e4a303d7cff1eac201f89 Reviewed-on: https://chromium-review.googlesource.com/c/1371644Reviewed-by:
-
Moe Ahmadi authored
- Removes dependency of :popup_menu_ui to the tools popup menu and navigation popup menu logic, i.e., PopupMenuCommands, coordinator, and mediator, allowing it to be reused in the new translate infobar for the language selection as well as translate options popup menus. - Moves reusable UI targets to ios/chrome/browser/ui/popup_menu/publicenu. Bug: 910994 Change-Id: I16229bd0d8d64ba4e20498bdb3ea283b780f0678 Reviewed-on: https://chromium-review.googlesource.com/c/1358114 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#615509}
-
![Friedrich Horschig [CET]'s avatar](https://secure.gravatar.com/avatar/f27b033ce4cd5371e6e413eed6db870f?s=80&d=identicon "Friedrich Horschig [CET]")
