- 13 Apr, 2018 40 commits
-
-
https://skia.googlesource.com/skia.git/+log/c6c5eade823a..21dfd846724a $ git log c6c5eade8..21dfd8467 --date=short --no-merges --format='%ad %ae %s' 2018-04-13 angle-skia-autoroll Roll third_party/externals/angle2/ ed8d5ec73..cc1293775 (1 commit) 2018-04-12 liyuqian Limit our IsBadForDAA for stroke width > 0 and <= 2 2018-04-12 csmartdalton ccpr: Actually blacklist ANGLE 2018-04-12 angle-skia-autoroll Roll third_party/externals/angle2/ 14f4817c4..ed8d5ec73 (8 commits) 2018-04-12 reed can't abort looper loop, as saveCount won't be balanced 2018-04-12 egdaniel Don't create lazy proxys if their width/height is greater than max tex size 2018-04-12 robertphillips Actually generate the gm images on the DDL1 bot 2018-04-12 caryclark son of path is rect bug 2018-04-12 ethannicholas workaround for buggy macOS GLSL shader compiler 2018-04-12 egdaniel Delete getRenderTargetHandle call on SkSurface. 2018-04-12 reed check for non-finite values output by clipper 2018-04-12 brianosman Fail fast when trying to make a texture that's too large 2018-04-12 djsollen Include shadow ops in the debugger. 2018-04-11 liyuqian Reland "Add private fIsBadForDAA bit to SkPath" 2018-04-12 reed add conservative bounds to raster tiling 2018-04-12 reed always clear the looper when we're unrolling it, to not confused devices 2018-04-11 robertphillips Disable CCPR and the small path renderer on DDL1 bot 2018-04-12 caryclark another rect is path fix 2018-04-11 robertphillips Add SK_SUPPORT_LEGACY_BACKEND_OBJECTS 2018-04-11 csmartdalton Implement Sk2f Load2 2018-04-11 csmartdalton ccpr: Blacklist ANGLE again 2018-04-11 robertphillips Allow SkImage_Gpu to fail some requests in DDL mode 2018-04-11 cwallez update_meta_config: Use new LUCI optional GPU trybot on Windows 2018-04-11 csmartdalton Implement Sk4f min/max 2018-04-11 liyuqian Remove obsolete comment 2018-04-11 benjaminwagner Upgrade PixelC and NexusPlayer OS. 2018-04-12 angle-skia-autoroll Roll third_party/externals/angle2/ 2dadd1d08..14f4817c4 (2 commits) 2018-04-11 caryclark fix bm gm 2018-04-12 mtklein skcms→1654786 concat gamut transform matrices together Created with: roll-dep src/third_party/skia BUG=chromium:824145,chromium:821353,chromium:818693,chromium:820245,chromium:820470,chromium:824145,chromium:815092 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;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 TBR=borenet@chromium.org Change-Id: Ic83a9e3addd0c3593736a5bd03a4b4b2d37b7c51 Reviewed-on: https://chromium-review.googlesource.com/1011738Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Eric Boren <borenet@chromium.org> Cr-Commit-Position: refs/heads/master@{#550739}
-
Eugene But authored
At this point almost all callbacks were removed from CRWWebDelegate and the only remaining callbacks are related to Native Content size (moved in this CL) and external app launching (will be moved to PolicyDecider). CRWNativeContentProvider is still heavily used for features like NTP, Old Download Manager, Error Pages, Reading List. It will take more time to get rid of Native Content Provider, so size callbacks can temporary live there. Bug: 674991 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia5c6d565a116c858f3fe5fefa4964364144a8279 Reviewed-on: https://chromium-review.googlesource.com/1010822Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#550738}
-
Nico Weber authored
deqp used to not build with clang, but nowadays it does. We wanted to wait with the switch until we can pin clang on the deqp bots to an older clang if necessary, but we already don't do that on mac and linux, so this doesn't make things any worse, and it removes one blocker for the msvc bot turndown and makes the deqp bots run tests with the compiler that we use for building shipping chromium. Bug: 727437, 786460 Change-Id: I3bd76f7e2939e33b28262ca509259b37e8bbe582 Reviewed-on: https://chromium-review.googlesource.com/1012740Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#550737}
-
Eugene But authored
NSURLSessionTask crashes if download is started for data:// URLs. Creating download task only for HTTP/HTTPS urls is a reasonable limitation, and there is no need to show UI for unsupported downloads. Bug: 832248 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ifec5e3ef1d4e4d76f78c5e310c2ca018e288e8bc Reviewed-on: https://chromium-review.googlesource.com/1011232Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#550736}
-
Avi Drissman authored
BUG=832842 TBR=hbos@chromium.org NOTRY=true Change-Id: I5329fc3c5b0c1e1d330471ceab8edf66ce24dbad Reviewed-on: https://chromium-review.googlesource.com/1012454 Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#550735}
-
Greg Thompson authored
The test harness drives an UpgradeDetectorImpl by calling its UpgradeDetected method. In a normal instance, this method is called by tasks scheduled on detect_upgrade_timer_, which is only started after the instance has determined whether or not updates are enabled. Since the test harness was not waiting for this determination to complete, it was racing with this task. The fix is to disable the update enablement check in the test so that there's no race. BUG=832057 Change-Id: Ie8b69a59e7de29aa7da37fe1edba5d324dbed450 Reviewed-on: https://chromium-review.googlesource.com/1012041 Commit-Queue: Greg Thompson <grt@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#550734}
-
Bret Sepulveda authored
Normally when a button is clicked we assume the mouse is still hovering over it, so we go back to the hovered state and wait for the mouse exit event. However, when the minimize button minimizes the window Windows gives us the mouse exit event before the mouse up event, so the button will get stuck in the hovered state. This patch fixes this by resetting the button state when it receives a window control action, the same way the opaque frame currently solves this problem. Bug: 828591 Change-Id: I6e3f27b8f92bcbf3fb8171555702b8cf5326ba21 Reviewed-on: https://chromium-review.googlesource.com/1006373Reviewed-by:
-
Saman Sami authored
I would like to be able to send a shared memory handle + SkImageInfo and construct an SkBitmap on the receiver's side. TBR=reed@google.com Bug: 831744 Cq-Include-Trybots: luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng Change-Id: If8be0cd2d876100f2430aed6195263cfcc415213 Reviewed-on: https://chromium-review.googlesource.com/1007767 Commit-Queue: Saman Sami <samans@chromium.org> Reviewed-by:
Mike Reed <reed@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#550732}
-
Eugene But authored
DownloadManagerTabHelper does not work without delegate, so make sure that DownloadManagerCoordinator is created before DownloadManagerTabHelper. Previously DownloadManagerCoordinator was created in viewDidLoad which is called after all Tab Helpers are attached to the restored tabs. Bug: 832210 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Id5bcbec42b0723007aef4d8a25a065cb30238484 Reviewed-on: https://chromium-review.googlesource.com/1011182Reviewed-by:
-
Ahmed Fakhry authored
When browser actions are dragged and dropped inside the menu, they used to use the menu's widget theme provider which is of type ui::DefaultThemeProvider. Instead, it should use the browser's frame theme provider, so it can get the correct color. BUG=831510 Change-Id: If7f46e38e44bbfee04b6f67e8377b7d8dd3d9d7f Reviewed-on: https://chromium-review.googlesource.com/1011374Reviewed-by:
-
Jialiu Lin authored
We've launched softer warning to 100%, now it's time to remove the set of scary strings we're not using. Bug: 831244 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I6c37435055154091dc91dcfc096d1295a5091572 Reviewed-on: https://chromium-review.googlesource.com/1010420Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Commit-Queue: Jialiu Lin <jialiul@chromium.org> Cr-Commit-Position: refs/heads/master@{#550729}
-
Avi Drissman authored
This reverts commit c9d9f5f1. Reason for revert: Test failures: EnableDisableSingleClientTest.DisableOneAtATime EnableDisableSingleClientTest.EnableOneAtATime SingleClientAppsSyncTest.InstallSomeApps SingleClientAppsSyncTest.InstallSomeLegacyApps SingleClientAppsSyncTest.InstallSomePlatformApps SingleClientAppsSyncTest.StartWithNoApps SingleClientAppsSyncTest.StartWithSomeLegacyApps SingleClientAppsSyncTest.StartWithSomePlatformApps SingleClientBookmarksSyncTest.BookmarkAllNodesRemovedEvent SingleClientBookmarksSyncTest.DeleteFaviconFromSync SingleClientBookmarksSyncTest.DownloadBookmarkFolder SingleClientBookmarksSyncTest.DownloadDeletedBookmark SingleClientBookmarksSyncTest.DownloadModifiedBookmark SingleClientBookmarksSyncTest.InjectedBookmark SingleClientBookmarksSyncTest.Sanity SingleClientBookmarksSyncTest.SetFaviconHiDPIDifferentCodec SingleClientDictionarySyncTest.Sanity SingleClientDirectorySyncTest.DeleteDirectoryWhenCorrupted SingleClientDirectorySyncTest.StopThenDisableDeletesDirectory SingleClientExtensionsSyncTest.InstallSomeExtensions SingleClientExtensionsSyncTest.StartWithNoExtensions SingleClientExtensionsSyncTest.StartWithSomeExtensions SingleClientExtensionsSyncTest.UninstallWinsConflicts SingleClientPasswordsSyncTest.Sanity SingleClientPreferencesSyncTest.Sanity SingleClientSearchEnginesSyncTest.Sanity SingleClientSessionsSyncTest.NoSessions SingleClientSessionsSyncTest.Sanity on https://ci.chromium.org/buildbot/chromium.win/Win7%20Tests%20(1)/79083 https://ci.chromium.org/buildbot/chromium.win/Win%207%20Tests%20x64%20(1)/37046 https://ci.chromium.org/buildbot/chromium.win/Win10%20Tests%20x64/22395 Original change's description: > Keep poll intervals in prefs. > > Without persisting them, server-side provided intervals are less efficient on > platforms with relatively short app lifetime. > > This also allows lower poll intervals for latency-sensitive use cases in cases > when our notifications are not reliable enough (yet). > > Bug: 832019 > Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs > Change-Id: I69abdc89273dd16301fb32f3a0df6ba51712adf3 > Reviewed-on: https://chromium-review.googlesource.com/1010662 > Commit-Queue: Tim Schumann <tschumann@chromium.org> > Reviewed-by: Marc Treib <treib@chromium.org> > Cr-Commit-Position: refs/heads/master@{#550668} TBR=treib@chromium.org,tschumann@chromium.org Change-Id: I015df172b8f0503553c5bbd9df5efaa97840fd64 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 832019 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/1012818Reviewed-by:
-
Aleks Totic authored
Inline outlines are painted by containing block. Containing block's SelfVisualRect did not include outline rects. The fix: - Refactors inline outline computation into NGOutlineUtils, so it can be shared between paint and layout. - Computes inline outline rects in NGPhysicalBoxFragment constructor. - Propagates outline to Legacy in LayoutNGMixin::AddOverflowFromChildren - Because overflow is flipped blocks, corrects some calls to AddOverflow Lots of work to correct a single failing test. Bug: 728378 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I252515113c8fb4e81dd59f377f4be00471596000 Reviewed-on: https://chromium-review.googlesource.com/1011828 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#550727}
-
The AutoRoll server is located here: https://fuchsia-sdk-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:fuchsia_arm64_cast_audio;master.tryserver.chromium.linux:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: I13af30a066f8ada04b06cc34143cce6c6292f5a9 Reviewed-on: https://chromium-review.googlesource.com/1012590Reviewed-by:
Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#550726}
-
Tien-Ren Chen authored
PaintLayerClipper returns a counter-scrolled rect when kIgnoreOverflowClipAndScroll is specified. Before this CL the counter-scroll was not applied if the scroller is not manually scrollable, e.g. overflow:hidden with a scroll offset. BUG=825033 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I05ad28a584911143e45204c751f679592ab03e19 Reviewed-on: https://chromium-review.googlesource.com/1011214Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Tien-Ren Chen <trchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#550725}
-
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/54a81e45ddfc..58330b0db43d $ git log 54a81e45d..58330b0db --date=short --no-merges --format='%ad %ae %s' 2018-04-06 ehmaldonado roll-dep: Use gclient_eval.GetRevision to get current revision. Created with: roll-dep src/third_party/depot_tools BUG=chromium:760633 The AutoRoll server is located here: https://depot-tools-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. TBR=agable@chromium.org Change-Id: Icab9883fd549a3383689c198168a5d196ab1c855 Reviewed-on: https://chromium-review.googlesource.com/1012342Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#550724}
-
Avi Drissman authored
This was broken for a very long time and the initializers regressed. There was a tool to update the perf_expectations.json file, but it is either broken or has zero documentation. The perf team disavows it. This is unmaintained, so I'm manually hacking this file to get it to a state in which it works. BUG=831951, 832854 NOTRY=true NOPRESUBMIT=true TBR=thakis@chromium.org Change-Id: Idf1bd61d40540d0b85497e6669297fb54c2c2a28 Reviewed-on: https://chromium-review.googlesource.com/1012743 Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
-
Takumi Fujimoto authored
- Add CastDialogSinkButton, which will be used for sink list items in the Harmony Cast dialog. It's a subclass of HoverButton that shows a device icon and an optional status indicator (checkmark or throbber). - Add UIMediaSink, a struct containing data used to populate the button - Add Skia icons used by the button: Cast for EDU, input, speaker, speaker group, and TV Screenshot of the button (two of them) in a WIP dialog: https://drive.google.com/file/d/1M1R9jO-VHEO18aA3pTpoQBI4FHMU5MEf/view?usp=sharing Dimensions and colors are subject to change in later CLs. Bug: 826089, 826393 Change-Id: I0eef58e1c0b9bf2cff3b3d146e911d3a18c7d23c Reviewed-on: https://chromium-review.googlesource.com/996458 Commit-Queue: Takumi Fujimoto <takumif@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
Adam Parker <amp@chromium.org> Cr-Commit-Position: refs/heads/master@{#550722}
-
Avi Drissman authored
This reverts commit 0ade0386. Reason for revert: Unexpected Failures: * bindings/sequence-type.html * custom-elements/spec/define-element.html * external/wpt/custom-elements/CustomElementRegistry.html on https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Mac10.11%20%28dbg%29/ https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20%28dbg%29/ Speculatively reverting this to see if it's the cause. Original change's description: > Web Animations: Fix bugs in procedure to process a keyframes argument > > There were three minor bugs left in the implementation: > > - We threw on lists-in-custom-iterators instead of just ignoring them. > - We returned all properties on the keyframe rather than just those > defined on the keyframe itself (e.g. we would include prototype > properties, against spec). > - We didn't access the properties in ascending unicode order. > > Bug: 827573 > Change-Id: I213ae5b24e1f35d7f28d16625025122950a6ba88 > Reviewed-on: https://chromium-review.googlesource.com/989261 > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Reviewed-by: Yuki Shiino <yukishiino@chromium.org> > Reviewed-by: Robert Flack <flackr@chromium.org> > Commit-Queue: Stephen McGruer <smcgruer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#550641} TBR=flackr@chromium.org,yukishiino@chromium.org,jbroman@chromium.org,haraken@chromium.org,smcgruer@chromium.org Change-Id: I5e8dc0c67599492bd6e90fca4a034e29e334ef88 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 827573 Reviewed-on: https://chromium-review.googlesource.com/1012857Reviewed-by:
-
Matt Menke authored
The method has some safe-for-display safety checks that assume the input is UTF-8 / output is UTF-8. This change makes it at least a little harder to avoid those checks, and makes output no longer vary based on whether passing in a std::string or a string16 (By removing the latter option entirely). Bug: 831321 Change-Id: Ib39a2cccd71861213341e92932525e8ecafc60cd Reviewed-on: https://chromium-review.googlesource.com/1004855Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#550720}
-
Ryan Hansberry authored
Breaks up the UNKNOWN_ERROR enum into 3 enums: 1) UNKNOWN_ERROR, which precisely captures when the host returns an UNKNOWN_ERROR response code. 2) NO_RESPONSE, which captures when the client never received a response at all. This likely indicates a Bluetooth error. 3) INVALID_HOTSPOT_CREDENTIALS, which captures when the host returns a SUCCESS response, but with invalid SSID or password info. Bug: 785514 Change-Id: I0d40283d8b458dd45a1d490fb53eb075e2eb05e4 Reviewed-on: https://chromium-review.googlesource.com/1011595 Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Reviewed-by:
Jeremy Klein <jlklein@chromium.org> Cr-Commit-Position: refs/heads/master@{#550719}
-
Devlin Cronin authored
There was a bug where if the ID were passed as null or undefined (rather than completely omitted), the message would fail to parse. Fix this, and add a unittest. Bug: 828664 Change-Id: I92b0f071a1627322e12e8eaf0f5787ccd0793138 Reviewed-on: https://chromium-review.googlesource.com/1012601Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#550718}
-
Dale Curtis authored
Use WillRepeatedly() instead of WillOnce(). BUG=831566 TBR=chcunningham Change-Id: I2cebaebcfa5ff026c43d3c4a267f5d3653e91dae Reviewed-on: https://chromium-review.googlesource.com/1012699Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#550717}
-
Dave Tapuska authored
Adjust ToggleFullscreen IPC to be two IPCs (Enter/Exit). Pass the options on the Enter IPC. This change is only plumbing necessary to get the options up to the content layer. A follow-on CL will be created that will modify the implementation of fullscreen on Android and plumbing at the WebContents layer. BUG=829937 Change-Id: I76ff73b6435bc6ae2ead57f06a98557a4961258d Reviewed-on: https://chromium-review.googlesource.com/999831 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#550716}
-
Paul Irish authored
Change-Id: If64e6f5fc245da7d9bbb8d3925d004b545338b0a Reviewed-on: https://chromium-review.googlesource.com/993477 Commit-Queue: Paul Irish <paulirish@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#550715}
-
Eric Stevenson authored
This CL adds the --step flag to make debugging large rolls more efficient. Also adds the --apply-patch flag to give users the ability to apply local patches before performing each build (not compatible with --cloud). Note that this doesn't work for patches to supersize; we copy supersize to a temp dir before modifying local state. Side note: using diagnose_bloat.py with CLs before and after the blink rename require "git config merge.renamelimit 20000". Bug: 831601 Change-Id: Ida5d0a879355e48c942dab07afd5489c36bca1ca Reviewed-on: https://chromium-review.googlesource.com/1011681Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Eric Stevenson <estevenson@chromium.org> Cr-Commit-Position: refs/heads/master@{#550714}
-
Yafei Duan authored
Adding user action recording for P2P sharing: - When user shares page loaded in current tab from overflow menu. - When user shares pages from Download Home. - When user clicks on the reload button in the snackbar shown when an untrusted page is loaded. Bug: 758690 Change-Id: I1b5c4dae5d33fb84b363ca2b26b2c56eb58efcf0 Reviewed-on: https://chromium-review.googlesource.com/1009073Reviewed-by:
Peter Williamson <petewil@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Joy Ming <jming@chromium.org> Commit-Queue: Yafei Duan <romax@chromium.org> Cr-Commit-Position: refs/heads/master@{#550713}
-
Alessio Bazzica authored
Removing unneeded files. Bug: webrtc:9076 Change-Id: If54b612ef8c9e5c27f868ab039a340fbbc215445 Reviewed-on: https://chromium-review.googlesource.com/995934Reviewed-by:
-
Daniel Bratell authored
There were two sets of message map macros, one with IsMsgHandled as a function, one with it as a macro. If combined in some translation unit that could cause compilation problems, and it did in some Opera builds. This patch changes the code so that there is only one implementation of the macros and it's the "safe" one, where "safe" means that it can handle that the underlying object is deleted inside a message handler without crashing. 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;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I2f039d8d93a539c8322caa664e642956c5d6d372 Reviewed-on: https://chromium-review.googlesource.com/995896Reviewed-by:
-
Andrey Kosyakov authored
This reverts commit 031b31c4. Reason for revert: triggers an assert on in devtools download interception tests with network service. Original change's description: > DevTools: fix interception for <a download='...'> downloads with network service > > Bug: 721408 > > Change-Id: I60f0cf81a69d7826d2043aa8222d2e69ff9e406a > Reviewed-on: https://chromium-review.googlesource.com/1006231 > Commit-Queue: Andrey Kosyakov <caseq@chromium.org> > Reviewed-by: Min Qin <qinmin@chromium.org> > Reviewed-by: Dmitry Gozman <dgozman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#550690} TBR=dgozman@chromium.org,caseq@chromium.org,qinmin@chromium.org Change-Id: I51f1a7145faa59103ff06b537380745913af2cf2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 721408 Reviewed-on: https://chromium-review.googlesource.com/1012817Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#550710}
-
Becca Hughes authored
The current MediaEnagementClient sends a bit to the renderer if the page has a high MEI score. We would like to use this for other things too but we need to send across more information (this is a precursor CL to crrev.com/c/962803). In order to do this properly this CL renames the existing interfaces to use a generic 'AutoplayConfiguration' name and sends flags instead of a single bool. BUG=821284 Change-Id: I509e35f52c57ae79817dc59eaa4a81fcf51e6310 Reviewed-on: https://chromium-review.googlesource.com/1010716 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#550709}
-
Rohit Rao authored
BUG=805154 TEST=None Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Iaa70b4edc0e0cf69ad8d2fa00be867d56ccc7625 Reviewed-on: https://chromium-review.googlesource.com/1012164 Commit-Queue: Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#550708}
-
Alex Moshchuk authored
This reverts commit 21168346. Reason for revert: These tests were almost certainly fixed by the reverts in https://bugs.chromium.org/p/chromium/issues/detail?id=828529. Original change's description: > Disable flaky WebviewLogin Tests on ChromeOS. > > Disables: > WebviewLoginTest.StoragePartitionHandling > WebviewProxyAuthLoginTest.ProxyAuthTransfer > > Bug: 832608 > Change-Id: Ie5e2d9c8b0fd99fd3c018a004f8b7b0c567d8c38 > TBR: achuith@chromium.org > Reviewed-on: https://chromium-review.googlesource.com/1012036 > Reviewed-by: Ramin Halavati <rhalavati@chromium.org> > Commit-Queue: Ramin Halavati <rhalavati@chromium.org> > Cr-Commit-Position: refs/heads/master@{#550595} TBR=achuith@chromium.org,rhalavati@chromium.org Change-Id: I70215f566a19f05b8b7bea8ce867a5e47e5c417f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 832608 Reviewed-on: https://chromium-review.googlesource.com/1012757Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#550707}
-
Maria Khomenko authored
This is no longer necessary since we only load partner bookmarks on demand now. Bug: 832371 Change-Id: Id6b76f7a8278aace0c682a00cd1d490e11311c73 Reviewed-on: https://chromium-review.googlesource.com/1011582Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Maria Khomenko <mariakhomenko@chromium.org> Cr-Commit-Position: refs/heads/master@{#550706}
-
https://pdfium.googlesource.com/pdfium.git/+log/7afdad5ab7a1..996c93068bfc $ git log 7afdad5ab..996c93068 --date=short --no-merges --format='%ad %ae %s' 2018-04-13 thestig Rename the other CPDF_Color::SetValue() variant. Created with: roll-dep src/third_party/pdfium The AutoRoll server is located here: https://pdfium-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. TBR=dsinclair@chromium.org Change-Id: Ie4bd6cf5456c533a4e8545517401abeed0b05e47 Reviewed-on: https://chromium-review.googlesource.com/1012135 Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#550705}
-
Alexei Filippov authored
BUG=832364 Change-Id: I4baf520742f740bc296eb12ce76d9e82796f9f9a Reviewed-on: https://chromium-review.googlesource.com/1011586Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#550704}
-
Alex Newcomer authored
Whenever an elevation is set (and the BubbleBorder is no longer default) remove the 1px border stroke. This brings the touchable app context menus closer to spec. Bug: 830929 Change-Id: Ia95af9088e2a7cb17b6195700b9cf73e1d0d9c86 Reviewed-on: https://chromium-review.googlesource.com/1008417 Commit-Queue: Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#550703}
-
Becky Zhou authored
Bug: 829377 Change-Id: Icf768d451f9967d954cf0ba40ff42a04c5dbe44d Reviewed-on: https://chromium-review.googlesource.com/1011389Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Becky Zhou <huayinz@chromium.org> Cr-Commit-Position: refs/heads/master@{#550702}
-
Kenneth Russell authored
Fix a collision of Pixel_Video_VP9 on the Mac Intel 10.13 bot. Tbr: ynovikov@chromium.org No-Try: True Bug: 830228 Change-Id: Ib074b0a8adc5961faea3341c082dcdbd2b7ff19e Reviewed-on: https://chromium-review.googlesource.com/1012463Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#550701}
-
Benjamin Pastene authored
https://chrome-internal-review.googlesource.com/608831 should give it enough bots to avoid serious pending times. Will add more bots if needed. TBR=jbudorick@chromium.org Bug: 731962 Change-Id: I5dea57cd599d8f1b24192c458017509ad68d47f4 Reviewed-on: https://chromium-review.googlesource.com/1012597Reviewed-by:
Ben Pastene <bpastene@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#550700}
-
