- 17 Oct, 2018 40 commits
-
-
Darren Shen authored
No longer working on CSS. Change-Id: I19305cdece49cce61f56225bc3e4774eb114d590 Reviewed-on: https://chromium-review.googlesource.com/c/1287269Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#600593}
-
Eric Roman authored
Bug: 894801 Change-Id: Ieb17a626284296b6f86f4cf9add2fca4359f7d8a Reviewed-on: https://chromium-review.googlesource.com/c/1285649 Commit-Queue: Eric Roman <eroman@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#600592}
-
Jun Mukai authored
Set shards to 3, as normal interactive_ui_tests does. BUG=883523 TEST=none Change-Id: Ia1fe3ec5e109b9d4b619c015525805e504e92d37 Reviewed-on: https://chromium-review.googlesource.com/c/1287197Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#600591}
-
Prashant Malani authored
The bug associated with the delay is no longer valid (the issue hasn't been observed, and indeed the FP H/W architecture has since changed). Additionally, there have been sporadic instances of "missed" StartAuth sessions, where the FP H/W receives the call to put the sensor into match mode, but the AP fails to acknowledge this the successful receipt. A race b/w sensor microcontrollers and AP suspend could lead to this failed acknolwedgement. Thus, we should remove the delay to at least ensure that the StartAuth call gets synchronously handled. BUG=b:113926492 TEST=Flash chrome on nocturne with this change, and suspend + unlock multiple times. Unlock works every time. Change-Id: Ifd41822d6f48ea4a8e162b3bfa30641ed0b5c367 Reviewed-on: https://chromium-review.googlesource.com/c/1285110 Commit-Queue: Prashant Malani <pmalani@chromium.org> Reviewed-by:Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#600590}
-
Robert Sesek authored
This should hopefully make browser_tests stable on Mac. Tbr: ellyjones@chromium.org Bug: 828031 Change-Id: If72e7e6692f48971e88acfb5bd069e1c83c88aaf Reviewed-on: https://chromium-review.googlesource.com/c/1287310Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#600589}
-
Dmitry Titov authored
Bug: 896481 Change-Id: I6e7d05e5673971d2b132e75dcaf1dfe7a858357a Reviewed-on: https://chromium-review.googlesource.com/c/1287290Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Commit-Queue: Dmitry Titov <dimich@chromium.org> Cr-Commit-Position: refs/heads/master@{#600588}
-
Tim Volodine authored
As was observed in https://crbug.com/891722 by jam@ the user_gesture is not being set in the network service code path, resulting in failing user_gesture related instrumentation tests. This patch fixes the issue. Fixes tests: -org.chromium.android_webview.test.ClientOnReceivedHttpErrorTest.testForUserGesture BUG=891722,841556 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I25be23261c178cd43b1e46440dd96dce89f70bf7 Reviewed-on: https://chromium-review.googlesource.com/c/1286821 Commit-Queue: Tim Volodine <timvolodine@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#600587}
-
David Tseng authored
This reverts commit 697307bb. TBR=dmazzoni,chrishall Bug: 885244,893396 Change-Id: I54ea0a3a8e71e7da39078245318608cc66c28c93 Reviewed-on: https://chromium-review.googlesource.com/c/1286815 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#600586}
-
Sunny Sachanandani authored
This reverts commit 90b4fddc. Reason for revert: Causing DC test failures on WinServer2008 bots: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20FYI%20x64%20Release%20%28NVIDIA%29/2903 Original change's description: > Use DirectComposition only if hardware overlays are supported > > Before this change, we were using DirectCompositionSurfaceWin on most > newer systems, but it's not needed unless hardware overlays are > supported. > > The disable_direct_composition workaround increases crash rates because > it's applied before GLSurfaceEGL decides to use direct composition with > ANGLE native EGL surfaces. > > This change moves all direct composition logic from GLSurfaceEGL to > DirectCompositionSurfaceWin, and removes use of direct composition with > ANGLE surfaces. > > Bug: 894675 > 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: I2e9c7c8a7ff36fe7b24868544c03aad0e0b7dad6 > Reviewed-on: https://chromium-review.googlesource.com/c/1278529 > Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> > Reviewed-by: Zhenyao Mo <zmo@chromium.org> > Reviewed-by: Antoine Labour <piman@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#600529} TBR=dcheng@chromium.org,zmo@chromium.org,sunnyps@chromium.org,piman@chromium.org Change-Id: Ic8ff38dddc037934dbf19267c3d027fd81c078eb No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 894675 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 Reviewed-on: https://chromium-review.googlesource.com/c/1287299Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Cr-Commit-Position: refs/heads/master@{#600585}
-
Chromium WPT Sync authored
Using wpt-import in Chromium cabd5a9c. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/26373 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=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: I8a70bb40cdc791740f77b3f2f130a10e75c7c3ed Reviewed-on: https://chromium-review.googlesource.com/c/1287193 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@{#600584}
-
Findit authored
This reverts commit d82a26bd. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 600490 as the culprit for flakes in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vZDgyYTI2YmQwMGM0NmQ1NGUwNzUyMWFmNmZiNTZhOTM0OGVkYWU5MAw Sample Failed Build: https://ci.chromium.org/buildbot/chromium.win/Win%207%20Tests%20x64%20%281%29/44415 Sample Failed Step: browser_tests Sample Flaky Test: SaveCardBubbleViewsFullFormBrowserTest.StrikeDatabase_Local_AddStrikeIfBubbleIgnored Original change's description: > [Autofill] Add strikes when credit card save ignored/denied/fails > > (When a card accrues 3 strikes, future offers to save for that card > should show the omnibox save icon, but NOT pop up the bubble. > On Android, no offer to save is shown at all.) > > Bug: 884817 > Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs > Change-Id: I0d6ad27a4e03d0128970af6e22d2a40a62d0601b > Reviewed-on: https://chromium-review.googlesource.com/c/1269524 > Commit-Queue: Jared Saul <jsaul@google.com> > Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org> > Reviewed-by: Fabio Tirelo <ftirelo@chromium.org> > Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org> > Reviewed-by: Moe Ahmadi <mahmadi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#600490} Change-Id: I7d0536b919212228db19f87c6cc09358928ffd35 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 884817, 896493 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Reviewed-on: https://chromium-review.googlesource.com/c/1287252 Cr-Commit-Position: refs/heads/master@{#600583}
-
Joe DeBlasio authored
Cyrillic letter U+0517 (ԗ) looks somewhat similar to the Latin letter p. This CL adds this character to the set of Cyrillic characters that look like Latin characters. Domains made up entirely of Cyrillic/Latin lookalikes are displayed as punycode in URLs. Bug: 863663 Change-Id: I4340c48d124c9c4cd3d3b5d0f9d3865d709e082d Reviewed-on: https://chromium-review.googlesource.com/c/1286825 Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#600582}
-
Joel Hockey authored
Clean up all checks to use IsParent which is simpler to understand than AppendRelativePath. Also fixes linux files to allow root mount point as valid downloads dir. Change-Id: Ie7d688949a69262b35bf999f0fc661a4633356c2 Reviewed-on: https://chromium-review.googlesource.com/c/1286235 Commit-Queue: Joel Hockey <joelhockey@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
Sam McNally <sammc@chromium.org> Cr-Commit-Position: refs/heads/master@{#600581}
-
Oksana Zhuravlova authored
Bug: 822804 Change-Id: I188fcfad992c7508125bc1ca7d196bb086687f44 Reviewed-on: https://chromium-review.googlesource.com/c/1281886Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#600580}
-
Nate Fischer authored
This enables testExitFullscreenEndsIfAppInvokesCallbackFromOnHideCustomView. This test flaked because DOMUtils was unreliable for clicking links. This CL swaps out DOMUtils for EvaluateJavaScriptWithUserGesture (since the relevant blink code requires user gesture and has no way to turn it off for testing). This plumbs EvaluateJavaScriptWithUserGesture to Java in a testonly target, so the Java method will not be present in production (and so this change should have no security implications). R=boliu@chromium.org --repeat=1000 \ --gtest_filter=AwContentsClientFullScreenTest#testExitFullscreenEndsIfAppInvokesCallbackFromOnHideCustomView Bug: 789306 Test: run_webview_instrumentation_test_apk --break-on-failure \ Change-Id: I4407ce0daa399fbb289d4c9d74e3d9b64bccfcbf Reviewed-on: https://chromium-review.googlesource.com/c/1286031Reviewed-by:Bo <boliu@chromium.org> Commit-Queue: Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#600579}
-
Amos Lim authored
Tests were moved from html/editing/focus to html/interaction/focus/. https://github.com/web-platform-tests/wpt/pull/12801 Bug: 801368 Change-Id: I19731a9ff1f1ab81f8fc5c8118b8d533844212c5 Reviewed-on: https://chromium-review.googlesource.com/c/1286027 Commit-Queue: Amos Lim <eui-sang.lim@samsung.com> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#600578}
-
Weidong Guo authored
Background: In some devices like nocturne, on-screen keyboard pops up when launcher is focused. When launcher is focused, search box textfield will be the initial focused view. SearchBoxViewBase::ContentsChanged() gets called to activate the search box, which triggers the on-screen keyboard. Change: Avoid activate the search box when new contents are empty string. Bug: 895249 Change-Id: I32b0f5a76037b5287d1b179fa20140101ce4f853 Reviewed-on: https://chromium-review.googlesource.com/c/1284754Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#600577}
-
Max Moroz authored
Change-Id: I23b01923afffcd23d6d636c1b4f2a6716769f554 Bug: 896109 Reviewed-on: https://chromium-review.googlesource.com/c/1281224Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#600576}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/594ad5fdcb7b..949d82edba6b Created with: gclient setdep -r src-internal@949d82edba6b 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: I1d41c5f813c21ccf269310d7082bb7b0a396acf8 Reviewed-on: https://chromium-review.googlesource.com/c/1286634Reviewed-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@{#600575}
-
Joshua Bell authored
Chrome's WebSQL implementation handles incognito mode by creating delete-on-exit directories (see crbug.com/779689). When an particular origin is getting deleted (e.g. via devtools) an origin-specific directory needs to get deleted. The logic to compute the directory path differs for incognito and non-incognito sessions, and there was a logic mismatch between the creation and deletion code. Additionally, the actual database files are marked "delete on close" and this necessitated holding onto file handles for the lifetime of the tracker to handle open/close/re-open. But when the origin is deleted, these handles need to be dropped. Refactor so a single method is used to compute the directory path in all cases, and drop handles when the origin is deleted. Bug: 779689, 876659 Change-Id: Idaf53ff407246c4dc3f66cdd84d2d4075b969d8e Reviewed-on: https://chromium-review.googlesource.com/c/1277698Reviewed-by:
Victor Costan <pwnall@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Joshua Bell <jsbell@chromium.org> Cr-Commit-Position: refs/heads/master@{#600574}
-
Michael Lippautz authored
V8 CL: http://crrev.com/c/1256764 Bug: 843903 Change-Id: I347a507cb25ae86d911ca3863ae79efb57cf2f1d Reviewed-on: https://chromium-review.googlesource.com/c/1286147 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Cr-Commit-Position: refs/heads/master@{#600573}
-
Peter Boström authored
Before this change DesktopMediaPickerViews would center itself on the page (and be spoofable), which is in conflict with it being a web-modal dialog that normally attaches to the location bar. This change also removes the close-x which is out of place for web-modal dialogs (and redundant with the existing close button). Bug: chromium:896012 Change-Id: I6d87a626cb5a4a3f2af87667faa3b2a718905202 Reviewed-on: https://chromium-review.googlesource.com/c/1284755 Commit-Queue: Peter Boström <pbos@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#600572}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/51b2312390d4..613fd1daf8d6 git log 51b2312390d4..613fd1daf8d6 --date=short --no-merges --format='%ad %ae %s' 2018-10-17 fmayer@google.com Merge "profiling: Set timeout for sockets in client." Created with: gclient setdep -r src/third_party/perfetto@613fd1daf8d6 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: Ifed414f3c4d3cc24538c1472030870419c20b7ea Reviewed-on: https://chromium-review.googlesource.com/c/1286116Reviewed-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@{#600571}
-
John Abd-El-Malek authored
This is similar to r593703 which fixed the old sign-in flow to work on desktop. The problem is that with network service Set-Cookie headers aren't visible by webRequest API. The fix is to read the cookies using the CookieManager API in C++. Other minor fixes: -FakeGaia doesn't have to set a Path for oauth_code cookie. GAIA doesn't set it in production, and having it set means that the C++ cookie reading code would need to set that path which is unnecessary. -convert SAMLPolicyTest to read cookies through CookieManager Note this doesn't address the Set-Cookie addition in saml_handler.js which still needs to be fixed. Bug: 887061 Change-Id: I902bdf0921f26368d7749838c69f03064e7ea9b4 Reviewed-on: https://chromium-review.googlesource.com/c/1285349Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Roger Tawa <rogerta@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#600570}
-
Ned Nguyen authored
System health test are skipped dynamically, so unlike other Telemetry tests which are disabled with @dectorators.Skip(..), the logic to not skip these tests have be done by the sites that invoke self.skipTest(..) Bug: 893618 Change-Id: I2c2869c5ec75613e797bc68d4c546b698f949869 Reviewed-on: https://chromium-review.googlesource.com/c/1287099Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#600569}
-
John Delaney authored
Currently we are not capturing bytes received by HTML5 media using Media Source Extensions. The metrics capturing bytes received for SRC are not reliably reported. These are desirable metrics for measuring impact of any feature targeting video data usage. Bug: 884899 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: I5892630c60fb635994bb2faffc36ef4dc3ba56d4 Reviewed-on: https://chromium-review.googlesource.com/c/1229213 Commit-Queue: John Delaney <johnidel@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Fredrik Hubinette <hubbe@chromium.org> Cr-Commit-Position: refs/heads/master@{#600568}
-
Matthew Jones authored
This patch reduces the size of a tab in the horizontal tab switcher to 90% of its previous size. The goal of this change is to make tabs on each side more 'tappable'. Bug: 880610 Change-Id: Ia0092aa5739f7ed11518eb5d0129dcfdf9e2cf9e Reviewed-on: https://chromium-review.googlesource.com/c/1286752Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#600567}
-
Peter Qiu authored
extensions::SafeBuiltins have been moved out of ctor of extension::Dispatcher. So explicitly initialize one after extension::Dispatcher is created. Bug: b/117836208 Test: manual verify on cast device Change-Id: Idedba018e9e37ef4a8f7856b00ec6bebab7ae17d Reviewed-on: https://chromium-review.googlesource.com/c/1286279Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Peter Qiu <zqiu@chromium.org> Cr-Commit-Position: refs/heads/master@{#600566}
-
John Williams authored
Bug: 893311 Change-Id: I5479bf30aad7801234144b475ebed6f975760afc Reviewed-on: https://chromium-review.googlesource.com/c/1281878Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Commit-Queue: John Williams <jrw@chromium.org> Cr-Commit-Position: refs/heads/master@{#600565}
-
Xi Han authored
The crash is caused by the callback UIStringOverrider::OverrideUIString() in ApplyUIStringOverrides() is called before resource bundle is initialized. In this CL, we checks whether the resource bundle has been initialized or not. If not, we cache the UI strings to be override in a map owned by the VariationsFieldTrialCreator, and override these UI strings once the full browser starts. Bug: 893707 Change-Id: Ia23bc46a5e13a20ab784ff48d565ce1438a6aa0b Reviewed-on: https://chromium-review.googlesource.com/c/1274001 Commit-Queue: Xi Han <hanxi@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#600564}
-
Alex Newcomer authored
Menu bounds do not fit new wider options if the menu options are added after the menu is initially drawn, and the width of the new options exceeds {screen_width - original_menu_origin.x}. This makes wide menu options unusable in some cases because the menu does not widen to accommodate the new items. Regression started: r573978 Test case: 1. Show a website with link[1] on the far right side of the screen: https://www.google.com/chromebook/ 2. Right click to show a menu on one of the links in the toolbar. The menu should be wide enough to show all contents of the menu. [1]Step 1 must have a link as the source of the menu because links use arc::OpenWithMenu, which can add wider ("Open with ...") options to a context menu. These options are added after the menu is drawn because we show the menu before arc calls back with the new menu options. Test: MenuControllerTest.GrowingMenuMovesLaterallyNotVertically Bug: 882662 Change-Id: Ic848fdb545bbdd05f874041bf1044adb7bd1d094 Reviewed-on: https://chromium-review.googlesource.com/c/1262397 Commit-Queue: Alex Newcomer <newcomer@chromium.org> Reviewed-by:Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#600563}
-
Scott Graham authored
Also, don't import * from fidl to avoid pylint complaining on Gerrit. With a couple tweaks to avoid 80col being exceeded for presubmit. No intended functionality change (and the generated fidl.js is unchanged). Bug: 883496 Change-Id: If4335f92d7630dd4b543dc686d2baf6aca5d2b6e Reviewed-on: https://chromium-review.googlesource.com/c/1258087 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#600562}
-
John Williams authored
Bug: 892797 Change-Id: I6aee5aafb1f68f6eabc37adab8fd91b5ea11cd29 Reviewed-on: https://chromium-review.googlesource.com/c/1278139 Commit-Queue: John Williams <jrw@chromium.org> Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#600561}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/b407163ef346..9aa14a38f434 git log b407163ef346..9aa14a38f434 --date=short --no-merges --format='%ad %ae %s' 2018-10-17 33432579+alan-baker@users.noreply.github.com OpGroupDecorate may not target OpDecorationGroup (#1977) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@9aa14a38f434 The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-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=dsinclair@chromium.org Change-Id: Ib751dc48816ccf7d9ce33a79a31b917d4d975967 Reviewed-on: https://chromium-review.googlesource.com/c/1286114Reviewed-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@{#600560}
-
Kehuang Li authored
In an earlier CL, AEC3 configs are gathered from a JSON string and passed to APM. But will still need to parse two non-AEC3 configs, i.e., pre-amplifier and AGC gains. Bug: 892611 Test: Add new unit tests for the new function and passed all unit tests. Change-Id: Ib52c12d6db18bc8842b64350d1ed8f226758828e Reviewed-on: https://chromium-review.googlesource.com/c/1281913 Commit-Queue: Kehuang Li <kehuangli@chromium.org> Reviewed-by:
Max Morin <maxmorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#600559}
-
Wez authored
We use some unsafe FDIO-internal APIs to implement asynchronous waits on file-descriptors. These APIs have been renamed, and the next SDK roll will remove the old names. Change-Id: I234931903c579c1ae1b46f082d4e526e65b05714 Reviewed-on: https://chromium-review.googlesource.com/c/1285515 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Fabrice de Gans-Riberi <fdegans@chromium.org> Cr-Commit-Position: refs/heads/master@{#600558}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/78bcd2bec371..3a482179dedb git log 78bcd2bec371..3a482179dedb --date=short --no-merges --format='%ad %ae %s' 2018-10-17 syoussefi@chromium.org Vulkan: Implement glFlush 2018-10-17 syoussefi@chromium.org Vulkan: Use a specialized macro for wrapped vulkan calls 2018-10-17 syoussefi@chromium.org Roll clang and other chromium deps 2018-10-17 tikuta@chromium.org Remove unused lambda capture Created with: gclient setdep -r src/third_party/angle@3a482179dedb The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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: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 BUG=chromium:none,chromium:681136 TBR=geofflang@chromium.org Change-Id: I35923d6afed0109c6fe771342aa988e6aa5fcaec Reviewed-on: https://chromium-review.googlesource.com/c/1287013Reviewed-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@{#600557}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/cb047393e8c6..594ad5fdcb7b Created with: gclient setdep -r src-internal@594ad5fdcb7b 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: I91ad74b5705258c5f28165dce691f56b23b0de90 Reviewed-on: https://chromium-review.googlesource.com/c/1286631Reviewed-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@{#600556}
-
Marina Ciocea authored
Audio sandbox policy currently allows ftruncate syscall; adding ftruncate64 used on x86 systems. This fix solves browser_tests failures on 32 bit Linux bots. With ftruncate64 blocked, IPC between audio process and renderer was failing leading to audio render errors. Bug: 890850 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: I6b09429c018468e0eb037498514df048fe79d520 Reviewed-on: https://chromium-review.googlesource.com/c/1282407 Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#600555}
-
Xiaoqian Dai authored
Bug: 895649 Change-Id: I4f4f94639ef7d4a68b37163451afc5eb1dcc77ec Reviewed-on: https://chromium-review.googlesource.com/c/1285086 Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#600554}
-