- 17 Oct, 2018 40 commits
-
-
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}
-
Kenneth Russell authored
The WebGL context attempts to optimize dirty notifications to the canvas and compositor, but the logic was fragile if the compositor skips a frame due to policy decisions. There were two almost-redundant state variables governing this. Remove animation_frame_in_progress_, and reuse marked_canvas_dirty_ for its purpose, clearing marked_canvas_dirty_ in FinalizeFrame. Verified with the test case from the bug and switching spaces 30 times on macOS Mojave. Bug: 895423 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: I456194746a1d20f4f2874b13c6cf6ba54dcf4ea1 Reviewed-on: https://chromium-review.googlesource.com/c/1281963Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#600553}
-
Xiaoqian Dai authored
Also disable other windows hide/show animation when tab-dragging process starts/ends. Bug: 888173 Change-Id: Iee22d20832cefd61ad7ebdea30b5a65e75b9f204 Reviewed-on: https://chromium-review.googlesource.com/c/1282162 Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#600552}
-
Fabrice de Gans-Riberi authored
C++ Compare operators need to follow a strict weak ordering requirement. The implementation used in SymbolMap::Populate didn't meet this requirement. Bug: 893369 Change-Id: Id1b7e4e7c46feb3bf2098ff810656d1d003aef8c Tested: Flakiness is gone. Reviewed-on: https://chromium-review.googlesource.com/c/1286763 Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#600551}
-
Garrett Beaty authored
Change-Id: I208be8bfb845218d88608a877405bd8b6c192066 Reviewed-on: https://chromium-review.googlesource.com/c/1287095Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#600550}
-
Josh Karlin authored
Ad scripts loaded in ad contexts can be detected by the context alone. Reserve tracking individual scripts to non-ad contexts. Bug: 894080 Change-Id: I90ec62e7e816fc83540f0daa232b105d03eb1553 Reviewed-on: https://chromium-review.googlesource.com/c/1276785 Commit-Queue: Josh Karlin <jkarlin@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#600549}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/e92aae6b3612..0f3d2a60105b git log e92aae6b3612..0f3d2a60105b --date=short --no-merges --format='%ad %ae %s' 2018-10-17 kjlubick@google.com Add new Codec fuzzers to FuzzMain 2018-10-17 rmistry@google.com Fix CT documentation 2018-10-17 brianosman@google.com Reland "Remove ancient support for kUnknown_SkColorType in SkBitmapDevice" Created with: gclient setdep -r src/third_party/skia@0f3d2a60105b The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel TBR=egdaniel@chromium.org Change-Id: I4d5f3ad3d55033e8d1046b108c489996c8692c5f Reviewed-on: https://chromium-review.googlesource.com/c/1286117Reviewed-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@{#600548}
-