- 16 Nov, 2017 40 commits
-
-
Charlie Andrews authored
Even though they're failing on 5X webview and they're disabled on 5X, there's a bug where 5X webview isn't matching the 5X test condition (https://github.com/catapult-project/catapult/issues/4045). TBR=ashleymarie@chromium.org NOTRY=true Bug: 784540 Change-Id: I100929183b15c1d84ff4e0cecddb15236644bb84 Reviewed-on: https://chromium-review.googlesource.com/775433Reviewed-by:
Charlie Andrews <charliea@chromium.org> Commit-Queue: Charlie Andrews <charliea@chromium.org> Cr-Commit-Position: refs/heads/master@{#517137}
-
Biao She authored
Bug: 762018 Change-Id: I51b9890df2ec4cde768bd08f2e6d5ed54eca999a Reviewed-on: https://chromium-review.googlesource.com/769737Reviewed-by:
agrieve <agrieve@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Commit-Queue: Biao She <bshe@chromium.org> Cr-Commit-Position: refs/heads/master@{#517136}
-
Mark Cogan authored
This CL removes clean/ view controllers from Showcase. Bug: 785925, 710662, 772186 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Iebce9f69b7776aa2755e46ec379d7245075a55a7 Reviewed-on: https://chromium-review.googlesource.com/774465 Commit-Queue: Mark Cogan <marq@chromium.org> Reviewed-by:
Louis Romero <lpromero@chromium.org> Cr-Commit-Position: refs/heads/master@{#517135}
-
btolsch authored
This change fixes two bugs with caching of media sinks and a potential initialization race that could also affect the cache's performance. The first fix is to allow network ID changes of network1->network2 to cache sinks for network1 rather than only caching for network1->disconnected/unknown. The cache originally used the latter logic because net::NetworkChangeNotifier broadcasts a disconnect between any two network changes. However, DiscoveryNetworkMonitor doesn't necessarily forward this behavior to its observers because it actually calculates the current network ID on every net::NetworkChangeNotifier notification. Not caching in the former case could account for some of the cache's low utilization currently seen in metrics. The second fix is to add CastMediaSinkServiceImpl as a DiscoveryNetworkMonitor::Observer from the correct sequence so it receives notifications on the correct sequence. This in turn exacerbates a race between the first network ID update and CastMediaSinkServiceImpl's AddObserver call. The race was therefore also addressed in this change. Bug: 782788, 782859 Change-Id: I064bd184508e66f1806e55dc5afe1eea907836ea Reviewed-on: https://chromium-review.googlesource.com/760667 Commit-Queue: Brandon Tolsch <btolsch@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#517134}
-
Charlie Andrews authored
NOTRY=true TBR=junov@chromium.org Bug: 785286 Change-Id: Ie6ea11ecb9a5ea4153da9aee124012ed9a5626bc Reviewed-on: https://chromium-review.googlesource.com/771853 Commit-Queue: Charlie Andrews <charliea@chromium.org> Reviewed-by:
-
Katie Dektar authored
Bug: 785435 Change-Id: I783aeedf9d67f4a8280d7770b9b276dbfab68b79 Reviewed-on: https://chromium-review.googlesource.com/772882 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#517132}
-
Charles Harrison authored
This CL adds a metric to measure the time spent from URLRequest creation and when Start() is called on it. The purpose is to measure time spent in various resource handlers before sending the request out to the net stack. Bug: 779914 Change-Id: I8e462a7d9fb541de38589aeb926f81047a735d50 Reviewed-on: https://chromium-review.googlesource.com/771549 Commit-Queue: Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#517131}
-
Sylvain Defresne authored
Convert PrintTabHelper to directly track registration with the observed WebState instead of relying on the deprecated code in WebStateObserver. Bug: 775684 Change-Id: I2788f302863f7f23bf35fab7a98a75ced00b17f1 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Reviewed-on: https://chromium-review.googlesource.com/768719 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#517130}
-
Peter K. Lee authored
This change is a precursor to a more extensive change where ExternalAppLauncher API is changed to be asynchronous. This simpler change is to call the decision handler in the implemention of WKNavigateDelegate method webView:decidePolicyForNavigationResponse:decisionHandler: asynchronously on iOS 11 and up. iOS 10 (and possible below) has a bug where JavaScript on the page would not execute if the decision handler is called asynchronously. This change is to make sure that there are no other unforeseen issues with calling decision handler asynchronously on iOS 11. Bug: 774736 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I7a6dfe637cb6be4ce5c7efe5824728b8f06b902a Reviewed-on: https://chromium-review.googlesource.com/773570 Commit-Queue: Peter Lee <pkl@chromium.org> Reviewed-by:
-
Xiaohui Chen authored
gfx::Image object is not threadsafe, we process the image async to avoid blocking UI thread but causes DCHECK error. Now we explicity transfer ownership of the object down to the callback. BUG=None TEST=running with DCHECK on and no error triggered after change. Change-Id: I1b3f678ddd9a411c59328e67de44b4e5db7ae865 Reviewed-on: https://chromium-review.googlesource.com/764373 Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#517128}
-
Jayson Adams authored
This cl contains cleanup that was part of the forthcoming MD tab spinner cl but has been separated out for clarity. Bug: 602558 Change-Id: Ibb58e9283a379f818b4c65ab79ae2e278e13b387 Reviewed-on: https://chromium-review.googlesource.com/769238Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Jayson Adams <shrike@chromium.org> Cr-Commit-Position: refs/heads/master@{#517127}
-
Alexei Filippov authored
BUG=721888 Change-Id: I42ac9339a8091c683c6dce60a857f71e0a9be7b8 Reviewed-on: https://chromium-review.googlesource.com/773626Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#517126}
-
Miriam Gershenson authored
This reverts commit 9a8baf49. These bots don't have enough capacity to handle all net/ CLs, and they've been blocking people from landing changes. Bug: 712733 Change-Id: I270079658a22301553c998acff8ae2395d3e2fa3 Reviewed-on: https://chromium-review.googlesource.com/774958 Commit-Queue: Miriam Gershenson <mgersh@chromium.org> Reviewed-by:
Misha Efimov <mef@chromium.org> Cr-Commit-Position: refs/heads/master@{#517125}
-
Peiyong Lin authored
Page almost idle signal is calculated from main thread task load and network activity. This patch plumbs page almost idle signal to TabLoader to experiment using this signal to schedule tab loading in SessionRestore. Currently, SessionRestore uses timeout and onload as signals to start to load next tab. A Finch Trial will be set up to experiment it. BUG=749789, 778766 Change-Id: Id83b6decc1ade9d3377b5184f0c717ac24bebabb Reviewed-on: https://chromium-review.googlesource.com/764569Reviewed-by:
Zhen Wang <zhenw@chromium.org> Reviewed-by:
-
Philip Rogers authored
These tests were passing and just needed new expectations: paint/overflow/fixed-background-scroll-window.html fast/sub-pixel/repaint-subpixel-layer-in-subpixel-composited-layer.html fast/sub-pixel/should-not-repaint-subpixel-composited-layer.html Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_root_layer_scrolls Change-Id: I2fb8c49e3aa5d1ce054bcd2d63f5f19ef66d1caa Reviewed-on: https://chromium-review.googlesource.com/772813Reviewed-by:
Stefan Zager <szager@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#517123}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 41d5cb07. With Chromium commits locally applied on WPT: 23705631 "service worker: Upstream sandboxed iframe test to WPT." Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/5343 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: hongchan@chromium.org, rtoy@chromium.org: external/wpt/webaudio kojii@chromium.org: external/wpt/css/css-writing-modes meade@chromium.org: external/wpt/web-animations nzolghadr@chromium.org, mustaq@chromium.org: external/wpt/pointerevents rego@igalia.com: external/wpt/css/css-grid smcgruer@chromium.org: external/wpt/css/css-position worker-dev@chromium.org: external/wpt/service-workers TBR=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: I0192a75b5f542020d9e200c88b8b0c7a2a8d2739 Reviewed-on: https://chromium-review.googlesource.com/774898 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@{#517122}
-
Lukasz Anforowicz authored
r499728 has incorrectly translated domAutomationController.sendWithId calls into domAutomationController.send calls - the first argument of the calls should have been dropped. This CL fixes this. Bug: 662543 Change-Id: If06e033ddddfe3e86000c2cbd68e49f9039dd706 Reviewed-on: https://chromium-review.googlesource.com/741921Reviewed-by:
Nick Carter <nick@chromium.org> Reviewed-by:
Roger Tawa <rogerta@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#517121}
-
Robert Ma authored
Only generic all-PASS testharness.js baselines can be safely removed. Removing platform-specific all-PASS baselines can lead to falling back to a different (and wrong) baseline. Bug: 768525 Change-Id: Ia4f2f37068e21cf85535864dbf6e196392fc1533 Reviewed-on: https://chromium-review.googlesource.com/773738 Commit-Queue: Robert Ma <robertma@chromium.org> Reviewed-by:
Quinten Yearsley <qyearsley@chromium.org> Cr-Commit-Position: refs/heads/master@{#517120}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/34d4e5a4d1a7..8b64fd8a85c1 $ git log 34d4e5a4d..8b64fd8a8 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org 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=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I04be5a185c79b3e867371457649823032b3fed64 Reviewed-on: https://chromium-review.googlesource.com/775039Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#517119}
-
Anita Woodruff authored
- This class actually contains methods for both persistent and non persistent notifications, not just persistent ones. TBR=avi@chromium.org Bug: Change-Id: I047d9863592823812d9c56c2dd6f538e4c07b3d1 Reviewed-on: https://chromium-review.googlesource.com/774464 Commit-Queue: Anita Woodruff <awdf@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#517118}
-
Alexei Filippov authored
Drive-by: Fix PRESUBMIT that was not allowing "toolbar::after" string in a CSS file. BUG=769614 Change-Id: I4ac2bf3520acac391df8999d09617e3375308393 Reviewed-on: https://chromium-review.googlesource.com/769990Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
-
Peter Wen authored
Findbugs has been removed. Keeping comments as they are good for context and still contain relevant information on threading etc. Also makes removing easier. Will remove actual file in follow-up CL to reduce revert-churn. TBR for cross-repo refactoring. BUG=777511 TBR=dtrainor@chromium.org,jochen@chromium.org,torne@chromium.org,peter@chromium.org,yusufo@chromium.org,mdjones@chromium.org Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Iec9c044375b9fd68ed605515381026e00a3c5431 Reviewed-on: https://chromium-review.googlesource.com/772399 Commit-Queue: Peter Wen <wnwen@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#517116}
-
Jonathan authored
A new test WebRtcCaptureFromElementBrowserTest.CaptureFromCanvas2DHandlesContextLoss was landed in https://chromium-review.googlesource.com/c/chromium/src/+/756474 It is consistently failing in viz_content_browsertests due to 785023. This change updates the tests filters to include this new test. While also copying the updates which the FYI bot received yesterday. TBR=sky@chromium.org TEST=viz_content_browsertests Bug: 760181 Change-Id: Idf6c9ff0239a7ec8dd871518a65e427aaf927577 Reviewed-on: https://chromium-review.googlesource.com/774978 Commit-Queue: Jonathan Ross <jonross@chromium.org> Reviewed-by:
Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#517115}
-
Jochen Eisinger authored
It's a legacy keyword that was accidentially removed BUG=615608 R=estark@chromium.org Change-Id: I1bf4d87d999f35b30beca644b4dec6712ac2d388 Reviewed-on: https://chromium-review.googlesource.com/772234Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#517114}
-
Gayane Petrosyan authored
Bug: 776071 Change-Id: I14e4aac7e366d9344f4c25e1f752fce193226784 Reviewed-on: https://chromium-review.googlesource.com/737558 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Reviewed-by:
Tao Bai <michaelbai@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#517113}
-
Thomas Tangl authored
The class EditableProfilePhoto is replaced with BadgedProfilePhoto because the editable-option has been dropped from that class. The class is moved to a separate file, so it is easier to maintain and extend (which will happen in the implementation for the DICe UI). Bug: 783152 Change-Id: I25fb7b1f3bf9f168ca23a90390365b13679c4eb6 Reviewed-on: https://chromium-review.googlesource.com/725293 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#517112}
-
Gyuyoung Kim authored
All uses of RefPtr have been removed in blink. Now is time to remove RefPtr.h in blink. As one of the step, this CL replaces RefPtr.h with scoped_refptr.h in platform and modules folder of blink. After landing this CL, new CL will remove all RefPtr.h file and remained uses. Bug: 763844 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: If5cc4480ebf96948573c49b164909072f8322db7 Reviewed-on: https://chromium-review.googlesource.com/773923 Commit-Queue: Gyuyoung Kim <gyuyoung.kim@lge.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#517111}
-
Jérôme Lebel authored
BookmarkPromoController is a helper for view objects that currently displays UI via the dispatcher (eg, -showSignin:). We want to extract presentation actions out of this pseudo-view layer and into the VC layer. Related to crrev.com/c/756362 Bug: Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Id4a97c11f857a448fa8f51a50a3e2c06f4d0426a Reviewed-on: https://chromium-review.googlesource.com/771698Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#517110}
-
Vaclav Brozek authored
Chrome settings on iOS allow selecting a set of saved password entries and deleting them. To select the entries, the view enters the "Edit" mode. If there are any entries left after deletion, the view stays in "Edit", but if there are none, it should go back to the default mode and show a disabled "Edit" button. The code clearly intended to do that, but called |setEdit| by mistake on the view controller instead of on its 'editor' property. This CL fixes that. The CL also fixes the glitch that Edit was not disabled if the last password was deleted from the detailed view. Bug: 755108 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ib28679ecf0502f3231ea7d918acd576ce048489e Reviewed-on: https://chromium-review.googlesource.com/774745 Commit-Queue: Vaclav Brozek <vabr@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#517109}
-
Sylvain Defresne authored
Convert PagePlaceholderTabHelper to directly track the registration with the observed WebState instead of relying on the deprecated code in WebStateObserver. Bug: 775684 Change-Id: I9784be30ecab75126349d401bfe75bc3114dd703 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Reviewed-on: https://chromium-review.googlesource.com/768815 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
-
Mike West authored
We ought to be restricting SharedWorker connections based on the security context of the document that created the worker. Secure contexts cannot connect to non-secure SharedWorkers, and non-secure contexts cannot connect to secure SharedWorkers. https://w3c.github.io/webappsec-secure-contexts/#examples-shared-workers We added metrics for this in 2016, and never followed up. Over the last month, this would have triggered on 0.00001% of page views; it seems safe to remove. Intent to Ship: https://groups.google.com/a/chromium.org/d/msg/blink-dev/NNdqRpSfkow/nvCWkHtRAQAJ Bug: 582813 Change-Id: Iaf9c9304a519b4db1a999e8c91ece5719f8edb7b Reviewed-on: https://chromium-review.googlesource.com/741743Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#517107}
-
Guido Urdaneta authored
The test crashes flakily on various bots. Sample runs: https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/64670 https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/64668 TBR=mstensho@chromium.org Bug: 785610 Change-Id: I3e344e9d361bbfa1b1f31257c75d88bd222021fa Reviewed-on: https://chromium-review.googlesource.com/774886Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#517106}
-
Scott Violet authored
Apparently the main waterfall now runs linux-chromeos-rel, so add content_browsertests --mus to it. BUG=776513 TEST=test only changes Change-Id: I342b2a9ff107c870c15b9b8ffebd413ccc9d38f1 Reviewed-on: https://chromium-review.googlesource.com/773286Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#517105}
-
Guido Urdaneta authored
TBR=zea@chromium.org Bug: 785967 Change-Id: I7f5088c10e16ba77417d801de216f68c6a65322a Reviewed-on: https://chromium-review.googlesource.com/774884Reviewed-by:
-
Gauthier Ambard authored
The LegacyToolbarCoordinator was in toolbar_coordinator.* files. This CL changes the file names to be legacy_toolbar_coordinator.* to avoid confusion. Bug: none Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ic85eb60df6470a70f89e04f3bf0d21937e8e5f5f Reviewed-on: https://chromium-review.googlesource.com/774281Reviewed-by:
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/d8d984ecc99d..24f9c19172e1 $ git log d8d984ecc..24f9c1917 --date=short --no-merges --format='%ad %ae %s' 2017-11-16 brianosman Revert "Fix precision caps and rrect/ellipse effect precisions" 2017-11-16 brianosman Revert "Fold analytic clip FPs into GrReducedClip" 2017-11-16 angle-deps-roller Roll skia/third_party/externals/angle2/ 39644feeb..a3106c585 (1 commit) 2017-11-15 csmartdalton Fold analytic clip FPs into GrReducedClip 2017-11-15 csmartdalton Fix precision caps and rrect/ellipse effect precisions 2017-11-16 angle-deps-roller Roll skia/third_party/externals/angle2/ 4d1469315..39644feeb (2 commits) 2017-11-16 angle-deps-roller Roll skia/third_party/externals/angle2/ 0acb268c9..4d1469315 (1 commit) 2017-11-15 bsalomon Use GrContextFactories that produce a single GrContext in unit tests. 2017-11-15 reed rename overloaded methods as speculative fix/diagnostic 2017-11-15 benjaminwagner Remove Win7/Win8 Perf-Release jobs. 2017-11-15 egdaniel Revert "Add private grpixelconfigs for alpha_8 and alpha_half" 2017-11-15 egdaniel Revert "Nit clean up to add private GrPixelConfig CL" 2017-11-15 benjaminwagner Remove/move remaining Win2k8 GCE jobs. 2017-11-15 egdaniel Nit clean up to add private GrPixelConfig CL 2017-11-15 vmpstr Ensure that removing last extension from GrGLExtension doesn't assert. 2017-11-15 bsalomon Remove support for image load/store 2017-11-15 egdaniel Add private grpixelconfigs for alpha_8 and alpha_half Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.skia.org 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=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel TBR=brianosman@chromium.org Change-Id: I4fef2f5d68f6ccd8295f487ed1cf27c72b36e6c3 Reviewed-on: https://chromium-review.googlesource.com/774684Reviewed-by:
Skia Deps Roller <skia-deps-roller@chromium.org> Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#517102}
-
stkhapugin@chromium.org authored
Currently compilation fails because there's an iOS 11 only constant in a switch-case, so there's a warning -Wunguarded-availability-new. This silences the warning when compiling this case. Bug: Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I3fe6bc7e66a16c3a8590af3472f345f8a883e452 Reviewed-on: https://chromium-review.googlesource.com/774858Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#517101}
-
Guido Urdaneta authored
This reverts commit cff667d9. Reason for revert: Suspect of breaking the following tests on the linux-chromeos-rel bot WebRtcCaptureFromElementBrowserTest.CaptureFromOpaqueCanvas2DHandlesContextLoss WebRtcCaptureFromElementBrowserTest.CaptureFromCanvas2DHandlesContextLoss First failure https://build.chromium.org/p/chromium.chromiumos/builders/linux-chromeos-rel/builds/2653 The bot has been red since. Original change's description: > Use async GPU readbacks in canvas.captureStream() > > This CL replaces the existing blocking call to SkImage::readPixels() with > GLHelper::ReadbackTextureAsync() and GLHelper::CreateReadbackPipelineYUV() > callbacks for texture backed images. > In order to use for these calls, a GLHelper instance is lazily created and > owned by blink::WebGraphicsContext3DProvider. > > Bug: 727385 > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 > Change-Id: Iad8561183afddff0794038213ee22cec3e04ed76 > Reviewed-on: https://chromium-review.googlesource.com/756474 > Commit-Queue: Emircan Uysaler <emircan@chromium.org> > Reviewed-by: Kenneth Russell <kbr@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Reviewed-by: Yuri Wiitala <miu@chromium.org> > Reviewed-by: Justin Novosad <junov@chromium.org> > Reviewed-by: danakj <danakj@chromium.org> > Cr-Commit-Position: refs/heads/master@{#516979} TBR=danakj@chromium.org,miu@chromium.org,kbr@chromium.org,haraken@chromium.org,junov@chromium.org,emircan@chromium.org Change-Id: I9dee0e7e28c2c5e1375578fe6192aeefd9e0cdff No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 727385 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Reviewed-on: https://chromium-review.googlesource.com/775098Reviewed-by:
-
Guido Urdaneta authored
TBR=chenwilliam@chromium.org Bug: 785980 Change-Id: Ib61f79451a99c1a98f061e4f32acd2337d55707a No-Try: true Reviewed-on: https://chromium-review.googlesource.com/774859 Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
-
Koji Ishii authored
Following bot results are included. 2116 2123 2127 2133 2137 2139 2140 2145 2146 2147 2149 2150 2151 2153 2154 2155 2159 2165 2166 2168 282 lines were removed by consecutive passes since 2098. TBR=eae@chromium.org NOTRY=true Bug: 591099 Change-Id: I120acfceac4e9e556c039250db5e90cd5c087ce5 Reviewed-on: https://chromium-review.googlesource.com/774918 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#517098}
-
