- 23 Mar, 2018 40 commits
-
-
Matt Wolenetz authored
Conditionally disables these 2 tests on windows dbg builds. BUG=825038,823003 TBR=tsepez@chromium.org,thestig@chromium.org NO-TRY=True Change-Id: I31b9365a71c29c794af7c1c424d3f6eeed9ea708 Reviewed-on: https://chromium-review.googlesource.com/978940Reviewed-by:
Matthew Wolenetz <wolenetz@chromium.org> Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#545596}
-
Candice Sy authored
Undoing the reordering from https://chromium-review.googlesource.com/969925 because this change created bugs related to starting/stopping the foreground service. Correct usage of notifyDownloadPaused() with isAutoResumable true and of notifyDownloadInterrupted() is to mark downloads as pending if they are not already paused. Bug: 812576,825113,825163 Change-Id: I6c33e035ec501c418c93acff916b7852f2b66cbd Reviewed-on: https://chromium-review.googlesource.com/978612Reviewed-by:
Joy Ming <jming@chromium.org> Commit-Queue: Candice Sy <cmsy@google.com> Cr-Commit-Position: refs/heads/master@{#545595}
-
Amirhossein Simjour authored
This reverts commit a8b44439. This will fix the problem with keyboard not staying after tap. I reopen 820898. Bug: 825175 Change-Id: I41ca2828fe6f81ac8c6569ebe99f97616506bcb5 Reviewed-on: https://chromium-review.googlesource.com/978035Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Amirhossein Simjour <asimjour@chromium.org> Cr-Commit-Position: refs/heads/master@{#545594}
-
Lei Zhang authored
Change-Id: I388cf30b67d43939bfca0f8827d5fe3339ed123d Reviewed-on: https://chromium-review.googlesource.com/978480Reviewed-by:
Wei Li <weili@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#545593}
-
Ian Vollick authored
This updates the system indicators to match the spec. When hovered, a tooltip appears with details. Bug: 824187 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;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Ie5c44bb0e448febe4c253d6ce91a0b43aad50535 Reviewed-on: https://chromium-review.googlesource.com/978614Reviewed-by:
Christopher Grant <cjgrant@chromium.org> Commit-Queue: Ian Vollick <vollick@chromium.org> Cr-Commit-Position: refs/heads/master@{#545592}
-
David Benjamin authored
Bug: none Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I39b362f41eeb9f8d8951bf3f73900bbb39d36df1 Reviewed-on: https://chromium-review.googlesource.com/977005Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#545591}
-
Ryan Landay authored
The logic in StackLayoutBase#setTabLists() has a bug that's causing it to not reduce the number of stacks properly when the number of TabLists is reduced. This does not affect the StackLayout used in production since it always uses a fixed number of stacks, and I apparently didn't notice this when testing the prototype I was working on at the time. Bug: 648314 Change-Id: I93374f398b376ad2f39a09211d928abb407af0b6 Reviewed-on: https://chromium-review.googlesource.com/972199Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Ryan Landay <rlanday@chromium.org> Cr-Commit-Position: refs/heads/master@{#545590}
-
Chromium WPT Sync authored
Using wpt-import in Chromium bf5bef7e. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/14369 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: yhirano@chromium.org, mkwst@chromium.org, japhet@chromium.org: external/wpt/fetch TBR=danyao No-Export: true Change-Id: Ia839d9516782a9a2e07cc647f229d571f927f9df Reviewed-on: https://chromium-review.googlesource.com/978037 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@{#545589}
-
Robert Ma authored
html.idl changed in upstream https://github.com/w3c/web-platform-tests/pull/10110 and imported in https://crrev.com/c/978021 Bug: 825191 Change-Id: Ie1a04e730aabd50c615f1dab079f92eeaa22565a Reviewed-on: https://chromium-review.googlesource.com/978508 Commit-Queue: Robert Ma <robertma@chromium.org> Reviewed-by:
Joshua Bell <jsbell@chromium.org> Cr-Commit-Position: refs/heads/master@{#545588}
-
Michael Giuffrida authored
ShellExtensionsClient::IsScriptableURL() can be called a lot, so it would be better if it didn't log NOTIMPLEMENTED(). We can consider it to be "implemented". Bug: None Change-Id: I4dd42c526d2aa75f3d7146527f1d6dfa7beb20fa Reviewed-on: https://chromium-review.googlesource.com/967602Reviewed-by:
Ben Wells <benwells@chromium.org> Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Cr-Commit-Position: refs/heads/master@{#545587}
-
Xiaoqian Dai authored
ScreenOrientationController is deleted earlier than SplitViewController in Shell during shutdown, so it's possible to have a nullptr dereference crash. Bug: 817520 Change-Id: I0e4ea1730c34dc7d767edffee80e5f09a88bb7af Reviewed-on: https://chromium-review.googlesource.com/976370Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#545586}
-
Kurt Horimoto authored
Bug: 805120 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I610f604d1f20695d028571166e443c12cd11cc20 Reviewed-on: https://chromium-review.googlesource.com/952766 Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#545585}
-
Nasko Oskov authored
When a content script is injected into a document, it can make requests to origins in its manifest. With recent changes to allow Chrome to block cross-site documents, the initiator of content scripts requests was changed to reflect the extension origin. This change meant that SameSite cookies were not attached to those requests. This CL introduces a check for content scripts making subresource requests and tags those requests to allow attaching SameSite cookies. Reland of https://chromium-review.googlesource.com/c/chromium/src/+/896690 Bug: 796480 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I8316f259f8b04c4fad406c398ba5207abef8a00e Reviewed-on: https://chromium-review.googlesource.com/974882 Commit-Queue: Nasko Oskov <nasko@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#545584}
-
Vaclav Brozek authored
The _CheckUmaHistogramChanges checks whether newly added UMA_HISTOGRAM_* calls only use defined histogram names. This CL removes two ways to introduce false positives: * Unrelated macro names which contain UMA_HISTOGRAM as a continuous substring * More than one string literal on the line with the histogram name. While those cases are rare, there seems no downside to fixing them. Bug: 821981 Change-Id: Ie1a803f562883f567d577e742ed2ed87fd0dfe66 Reviewed-on: https://chromium-review.googlesource.com/978245 Commit-Queue: Vaclav Brozek <vabr@chromium.org> Reviewed-by:
-
Paul Irish authored
Change-Id: I8fa0b6e606f828df9677deefbcbf7254359fe5c6 Reviewed-on: https://chromium-review.googlesource.com/974178 Commit-Queue: Paul Irish <paulirish@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Patrick Hulce <phulce@chromium.org> Cr-Commit-Position: refs/heads/master@{#545582}
-
Mike Dougherty authored
Bug: 228355 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I871fe76d7e2d8ce96ce5e70475e503216f711326 Reviewed-on: https://chromium-review.googlesource.com/975996 Commit-Queue: Mike Dougherty <michaeldo@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#545581}
-
Xiaoqian Dai authored
Define SPLITVIEW_ANIMATION_RESTORE_OVERVIEW_WINDOW. And also animate the overview window from its position in overview grid to its snapped postion. Bug: 822486 Change-Id: I64a48b33012c428a5529dfea58f048f30c043e73 Reviewed-on: https://chromium-review.googlesource.com/972562Reviewed-by:
-
Matt Wolenetz authored
Conditionally disables this test on dbg win. BUG=825304,823003 TBR=dpapad@chromium.org NO-TRY=True Change-Id: Ic6950d3d6a2e1fd52a6d30f1f6a549045619ee2f Reviewed-on: https://chromium-review.googlesource.com/978863Reviewed-by:
-
erikchen authored
There are no longer any cross-platform consumers of this code. There are two ChromeOS consumers of the latter, which have been extracted into a function only defined on ChromeOS. Bug: 819289 Change-Id: Iaa0c12c94619ba54be44e23eb45abc5337da4621 Reviewed-on: https://chromium-review.googlesource.com/973943 Commit-Queue: Erik Chen <erikchen@chromium.org> Reviewed-by:
Nick Carter <nick@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#545578}
-
Jao-ke Chin-Lee authored
win_chromium_dbg_ng has had 176 tests running over 3 hours in the 1000 builds between 4670 and 3669. Currently, these show up as infra failures rather than test failures. Increase the hard timeout to 4 hours, as no build in the above sample exceeded 4 hours. BUG=731384 Change-Id: Iff6dba2ffb43920aa991cfef7fdc4bb6e5773eb4 Reviewed-on: https://chromium-review.googlesource.com/978673 Commit-Queue: Jao-ke Chin-Lee <jchinlee@chromium.org> Reviewed-by:
Marc-Antoine Ruel <maruel@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#545577}
-
Weidong Guo authored
Changes: Treat browser back as esc key to navigate back in app list. BUG=819668 Change-Id: Iccccfe8b424570643c851a73d191e86f66271ced Reviewed-on: https://chromium-review.googlesource.com/978546Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#545576}
-
Nate Fischer authored
No change to logic. This adds WebViewClientBoundaryInterface, which will be implemented by WebViewClientCompat on the support library side. These methods will be invoked from the support_library side in later CLs, as we implement each callback. Bug: 781764 Test: N/A Change-Id: I2ba052ff02076cec1c9d4e360aed322a63b511ef Reviewed-on: https://chromium-review.googlesource.com/974898Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Gustav Sennton <gsennton@chromium.org> Commit-Queue: Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#545575}
-
Matt Wolenetz authored
It's flaky on Win7 Tests (dbg)(1). This change disables it in debug Windows builds. BUG=825341,823003 TBR=pkasting@chromium.org NO-TRY=True Change-Id: Ibb04812ca20081ec55cbc1bdbef01464ddb062f3 Reviewed-on: https://chromium-review.googlesource.com/978835Reviewed-by:
-
yucliu authored
Pipeline checks policy before getting the clear audio data back. This also allows pipeline to call proper API for decryption. The patch adds a new enum OutputType for DecryptContextImpl. Pipeline checks the OutputType before decryption. The OEM CDM library still enforces license policy before doing the actual decryption. BUG=internal b/64914604 TEST=cast_cdm_widevine_test, encrypted content Change-Id: I95188b6b025978e0e2e508704216e40c9734db94 Reviewed-on: https://chromium-review.googlesource.com/977039Reviewed-by:
Sergey Volk <servolk@chromium.org> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Yuchen Liu <yucliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#545573}
-
Allen Bauer authored
Bug: 822000 Change-Id: I950ec472da2b5b8f8b52c54b9d14adc0098b3804 Reviewed-on: https://chromium-review.googlesource.com/969092Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Allen Bauer <kylixrd@chromium.org> Cr-Commit-Position: refs/heads/master@{#545572}
-
Xianzhu Wang authored
For <svg style="will-change: left"></svg>, previously we created a transform node just to store the direct compositing reason CompositingReason::kWillChangeCompositingHint, which broke the assumption that only objects having a PaintLayer can be composited. Bug: 825090 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I6eddb3b025ea82c08b38dc4ae0c0efc1048f6854 Reviewed-on: https://chromium-review.googlesource.com/978504Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#545571}
-
Min Qin authored
This CL moves DownloadWorker from content/browser/download to components/download. DownloadWorker is used by ParallelDownloadJob for creating parallel download requests. To remove the dependencies on content/, following changes are made: 1. Introduces URLDownloadHandlerFactory class, this class allows content/ to inject its own URLDownloadHandler implementation (UrlDownloader) when network service is disabled. 2. DownloadManager will always pass a SharedURLLoaderFactory to ResourceDownloader. ResourceDownloader will later return it back to DownloadManager, and DownloadManager will then pass it to the ParallelDownloadJob through DownloadItemImpl. Bug: 803135 Change-Id: I96dfb75daa438049df0dc9fd0fbe74abd5ff1d7a Reviewed-on: https://chromium-review.googlesource.com/974120 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
-
Siddhartha authored
The bitmap resources held by LayerTitleCache for tabs are never cleared by tab strip model. Release resources held for a tab if the tab is marked invisible in tab strip layout. BUG=701976 Change-Id: I3539d5aa546d5f002cb2b1a670de5dea52336722 Reviewed-on: https://chromium-review.googlesource.com/977324Reviewed-by:
-
Findit authored
This reverts commit cdd8661f. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 545521 as the culprit for failures in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2NkZDg2NjFmM2YzNWQ3MDc1MDIwZWFkMDQ5MmNkODNlMThmNzk1ZTkM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests%20%28dbg%29%281%29%2832%29/48976 Sample Failed Step: base_unittests Original change's description: > base: Introduce the new Shared Memory API > > Design doc (public): https://goo.gl/HmBYy6 > > This patch proposes the new API for the Shared Memory based on two > concepts, a shared memory region and a shared memory mapping. The > region controls the platform-specific handle and mediates sharing and > access control. The mapping is just a reference to the memory bytes > which knows how to unmap itself. Ownership is per-instance: neither a > region nor a mapping may be copied, and resources are freed when then > instance dies. This is accomplished through move-only semantics. > > The goal of the new API is to provide a correct cross-platform > implementation that makes the sharing mode explicit via the C++ type > system. > > Thus, this CL introduces two classes for shared memory mappings: > - WritableSharedMemoryMapping owns and provides an access to writable > shared memory > - ReadOnlySharedMemoryMapping for read-only shared memory > > and three classes representing shared memory regions: > - UnsafeSharedMemoryRegion allows to create > WritableSharedMemoryMapping instances and cannot be used for making > read-only mappings > - ReadOnlySharedMemoryRegion, on the contrary, can produce only > ReadOnlySharedMemoryMapping instances and it also creates one writable > mapping in the caller process address space > - WritableSharedMemoryRegion allows to create writable > mappings and can be eventually converted to ReadOnlySharedMemoryRegion. > The important limitation in comparison with UnsafeSharedMemoryRegion > is that the WritableSharedMemoryRegion cannot be duplicated so in > order to pass the writable region to another process, the ownership of > the region must be moved/transferred to this process. > > Bug: 795291 > Change-Id: I7d1d0a24932145cc0f449b01ceb8cafef9b7e407 > Reviewed-on: https://chromium-review.googlesource.com/893458 > Commit-Queue: Alexandr Ilin <alexilin@chromium.org> > Reviewed-by: Matthew Cary <mattcary@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Erik Chen <erikchen@chromium.org> > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Reviewed-by: Ken Rockot <rockot@chromium.org> > Cr-Commit-Position: refs/heads/master@{#545521} Change-Id: I9c5657e80b8a262a42311d4d2a0239dc242ab8f5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 795291 Reviewed-on: https://chromium-review.googlesource.com/978563 Cr-Commit-Position: refs/heads/master@{#545568}
-
Oksana Zhuravlova authored
This change moves thread_priority.mojom from //mojo/common to //mojo/public/mojom/base. Corresponding typemap and struct traits files were also moved. Unit test for ThreadPriority was added to mojo_unittests. Bug: 799482,806965 Change-Id: I47c23a6b5eea8fffd8a6aa9c18f9e8e12045e8e7 Reviewed-on: https://chromium-review.googlesource.com/976621 Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
-
Hector Carmona authored
Test was affected by paper-icon-button migration. This selects the correct element to check for the hidden attribute. Bug: 821759 Change-Id: I1bc67c41e9e9293c5f06ef0120fdde85beb959d6 Reviewed-on: https://chromium-review.googlesource.com/978470Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#545566}
-
Matt Menke authored
NOTRY=true Bug: 822717 Change-Id: I21cd2bed496cef2d8e94d1eebc7354cfa5b90ddd Reviewed-on: https://chromium-review.googlesource.com/978385 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#545565}
-
Avi Drissman authored
BUG=357627 Change-Id: I62ff9157d6d41bd4581ba3d080dca2c309cb3e82 Reviewed-on: https://chromium-review.googlesource.com/978387Reviewed-by:
Sidney San Martín <sdy@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#545564}
-
https://webrtc.googlesource.com/src.git/+log/06e363a6e608..9718711deed8 $ git log 06e363a6e..9718711de --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;master.tryserver.chromium.win:win-msvc-dbg TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I6ad4ab8bec380255169cff3e49e71ebc27abdf20 Reviewed-on: https://chromium-review.googlesource.com/978377Reviewed-by:
webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#545563}
-
Sidney San Martín authored
Bug: 823108 Change-Id: Ie1c4d5ee7fc63ea56f2a25506bab14a7a4137b87 Reviewed-on: https://chromium-review.googlesource.com/973892 Commit-Queue: Sidney San Martín <sdy@chromium.org> Reviewed-by:
Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#545562}
-
mrefaat authored
The reason for the crash is the thread inconsistency of dereferencing the weakptr of the CookieCreationTimeManager object Removed weakptr, as it appears that it's not needed at all. Bug: 813779, 820384 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Iff614621b09b92ae5fa9a41ad90a2e390a1ce6b1 Reviewed-on: https://chromium-review.googlesource.com/976095 Commit-Queue: Mohammad Refaat <mrefaat@chromium.org> Reviewed-by:
-
Matt Wolenetz authored
Adds conditional disabling of these two tests. BUG=825305,823003 TBR=sky@chromium.org,creis@chromium.org NO-TRY=True Change-Id: I9c24882c82db878883a9cd8df9abe3f95723634c Reviewed-on: https://chromium-review.googlesource.com/978666 Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Reviewed-by:
-
Wez authored
These tests were posting delayed tasks to inject video frames, with raw pointers to the target object. The tests would usually pass because the delayed task would be silently dropped during MessageLoop teardown, but on slow systems (e.g. under QEMU) these delayed tasks could end up being executed during the teardown RunUntilIdle(), after the target had been torn down. The target is a ref-counted object, so post delayed tasks to it without using Unretained(), to ensure it remains valid. Bug: 819924 Change-Id: I5a781cbaab1519b5707967753decf2672ba826d5 Reviewed-on: https://chromium-review.googlesource.com/978637Reviewed-by:
Emircan Uysaler <emircan@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#545559}
-
Jay Civelli authored
Changing the WebContentsObserver::SubresourceResponseStarted to SubresourceLoadComplete. This is going to be used to support page load metrics with the network service. Note that SubresourceResponseStarted in frame.mojom is kept, as it is used by the SSLManager. It's implementation is kept in WebContentsImpl and its parameters are changed to only what's needed for that use case. Bug: 816684 Change-Id: I78fa4b0d7f44cfb0a5e98a7fc008fb7a17f8aae4 Reviewed-on: https://chromium-review.googlesource.com/967370 Commit-Queue: Jay Civelli <jcivelli@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Reviewed-by:
Conley Owens <cco3@chromium.org> Cr-Commit-Position: refs/heads/master@{#545558}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 072d5198. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/14368 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: rouslan@chromium.org, mathp@chromium.org: external/wpt/payment-request TBR=danyao No-Export: true Change-Id: I10d399868669fd9cfcbb414325abfaf1b5cfaf51 Reviewed-on: https://chromium-review.googlesource.com/978031 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
-
