- 10 May, 2018 40 commits
-
-
Tibor Goldschwendt authored
This change lets us scroll one page in the tabs view. 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.linux:linux_vr Change-Id: I72d4bc22a73c0c871c8d233d69335ee58a10e93e Reviewed-on: https://chromium-review.googlesource.com/1050845Reviewed-by:
Biao She <bshe@chromium.org> Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#557508}
-
Justin Novosad authored
This CL is a step towards removing OffscreenCanvasResourceProvider BUG=788439 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I78972f67aa8849de59690f591d69c534bdc08a42 Reviewed-on: https://chromium-review.googlesource.com/1053173Reviewed-by:
Justin Novosad <junov@chromium.org> Reviewed-by:
Olivia Lai <xlai@chromium.org> Commit-Queue: Justin Novosad <junov@chromium.org> Cr-Commit-Position: refs/heads/master@{#557507}
-
Ryan Landay authored
We currently log the UMA event MobileTabSwitched when a user switches tabs in the Android tab switcher, but we don't log one if the user stays on the same tab. This CL starts logging a new MobileTabReturnedToCurrentTab event for this case. Bug: 828224 Change-Id: Iaeedf9f0c7493e68ae2fde335eaba5ad6bd0405d Reviewed-on: https://chromium-review.googlesource.com/1050926Reviewed-by:
Brian White <bcwhite@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Ryan Landay <rlanday@chromium.org> Cr-Commit-Position: refs/heads/master@{#557506}
-
Ryan Landay authored
This CL adds logging for a new UMA event MobileToolbarStackViewButtonInStackView to log when a user in the Android tab switcher taps on the tab switcher button on the toolbar to exit the tab switcher. Bug: 828224 Change-Id: I1b7528544daba0c2f23746f309d4b97cf98ef826 Reviewed-on: https://chromium-review.googlesource.com/1050965Reviewed-by:
-
Noel Gordon authored
Tbr: slangley Bug: 841659 Change-Id: I2f7e55e103f91c563600a29632c59fade44fba3b Reviewed-on: https://chromium-review.googlesource.com/1053676Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#557504}
-
Sam McNally authored
Bug: 823956 Change-Id: I6dbb43bb2a9af2723798f00f5ee1cd0cdc1dfa95 Reviewed-on: https://chromium-review.googlesource.com/1039114 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Stuart Langley <slangley@chromium.org> Cr-Commit-Position: refs/heads/master@{#557503}
-
Roger Tawa authored
Bug: 834939 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Iaa64d4a386da00914b0f69a33573f8e049833b82 Reviewed-on: https://chromium-review.googlesource.com/1035805Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Joe Mason <joenotcharles@chromium.org> Commit-Queue: Roger Tawa <rogerta@chromium.org> Cr-Commit-Position: refs/heads/master@{#557502}
-
Nick Harper authored
Bug: 841685 Change-Id: I1deb65b345058cf9317122ffb5bb4a236dd5446c Reviewed-on: https://chromium-review.googlesource.com/1053787 Commit-Queue: Nick Harper <nharper@chromium.org> Commit-Queue: Bernhard Bauer <bauerb@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Cr-Commit-Position: refs/heads/master@{#557501}
-
Bence Béky authored
Some SpdyStream methods are conceptually const, mark them as such. Change-Id: I89935beec587771045554e30dffdb998195e5e64 Reviewed-on: https://chromium-review.googlesource.com/1050190Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#557500}
-
Ethan Kuefner authored
This is the OBBS-side version of https://chromium-review.googlesource.com/c/chromium/tools/build/+/1052920 Bug: 841351 Change-Id: Ie863c647ecd93ebeda326b00ebcad1543db0f7ef Reviewed-on: https://chromium-review.googlesource.com/1053048Reviewed-by:
Simon Hatch <simonhatch@chromium.org> Reviewed-by:
Emily Hanley <eyaich@chromium.org> Commit-Queue: Simon Hatch <simonhatch@chromium.org> Cr-Commit-Position: refs/heads/master@{#557499}
-
Robert Phillips authored
The dependent Skia CL is: https://skia-review.googlesource.com/c/skia/+/127125 (Fix clipping of SkMatrixImageFilter input (take 2)) Change-Id: I971081aebfe333bbaa3b6a6d209038e198c118ff Reviewed-on: https://chromium-review.googlesource.com/1052850Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Robert Phillips <robertphillips@google.com> Cr-Commit-Position: refs/heads/master@{#557498}
-
Jinho Bang authored
This patch adds a use counter to count invalid currency code. The data will be used later when deprecating/removing the member. Bug: 839402 Change-Id: Idbb4f305cf453a3ddddf19983ecfb45bec69d22d Reviewed-on: https://chromium-review.googlesource.com/1053027Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Ganggui Tang <gogerald@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Jinho Bang <jinho.bang@samsung.com> Cr-Commit-Position: refs/heads/master@{#557497}
-
Greg Kraynov authored
Bug: 841735 Change-Id: I38872f08a406c208b65f15845293b2ad55d20b19 Reviewed-on: https://chromium-review.googlesource.com/1053767Reviewed-by:
Alex Clarke <alexclarke@chromium.org> Commit-Queue: Greg Kraynov <kraynov@chromium.org> Cr-Commit-Position: refs/heads/master@{#557496}
-
Tsuyoshi Horo authored
When this feature is enabled: - The Accept header of navigation requests will have "application/signed-exchange;v=b0". - If the response HTTP header has a valid Origin-Trial header for "SignedHTTPExchange", and the Content-Type header is "application/signed-exchange;v=b0", Chrome handles the response as SignedHTTPExchange. - Otherwise, the navigation loading will fail. Bug: 827939 Change-Id: Ib4906388cd6939df51244478d3e7a31df6c43652 Reviewed-on: https://chromium-review.googlesource.com/985993Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Commit-Queue: Tsuyoshi Horo <horo@chromium.org> Cr-Commit-Position: refs/heads/master@{#557495}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 47084bae. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/17237 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: bjonesbe@adobe.com: external/wpt/css/css-shapes TBR=markdittmer No-Export: true Change-Id: I08f07061ebc283afb41038664889d579a07c4ce6 Reviewed-on: https://chromium-review.googlesource.com/1053575 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@{#557494}
-
Bence Béky authored
Currently when SpdyStream::Delegate calls SpdyStream::Cancel(), that calls SpdySession::ResetStream(), which calls SpdySession::ResetStreamIterator(), which calls SpdySession::CloseActiveStreamIterator() with ERR_SPDY_PROTOCOL_ERROR, which calls SpdySession::DeleteStream() with ERR_SPDY_PROTOCOL_ERROR, which calls SpdyStream::OnClose() with ERR_SPDY_PROTOCOL_ERROR, which calls SpdyStream::Delegate::OnClose() with ERR_SPDY_PROTOCOL_ERROR. After this CL, SpdyStream::Cancel() takes an int error argument, which is passed to SpdySession::ResetStream() then to SpdySession::ResetStreamIterator() then to SpdySession::CloseActiveStreamIterator() then to SpdySession::DeleteStream() then to SpdyStream::OnClose() then to SpdyStream::Delegate::OnClose(). This way Delegate can directly control what error its own OnClose() method will be called with. This will be useful if I end up moving HTTP/2 pushed header validation to SpdyHttpStream, so that it can use a dedicated error code to signal to HttpNetworkTransaction if the pushed stream does not match. Bug: 554220 Change-Id: Id6395d834395d182b7af4a45e1bb1186c1b82feb Reviewed-on: https://chromium-review.googlesource.com/1051007Reviewed-by:
-
Max Morin authored
It checks that the stream is allowed and forwards the request to the relevant ForwardingAudioOutputStreamFactory if so. This will cause the stream to be served by the audio service. The old RenderFrameAudioOutputStreamFactory which creates streams living in content/ is renamed to OldRenderFrameAudioOutputStreamFactory. Since the class was renamed, the files were moved (by adding "old_" to the beginning). No need to review those files. Also note that replacement is diffed against the previous implementation. It's probably best to just ignore the diff and review render_frame_audio_output_stream_factory{.cc,.h,_unittest.cc} as new files. A flag is added to switch between the old factory and the new one. Approximate diagram of stuff: https://docs.google.com/drawings/d/1_ZIKj6lihGKRjq4Mflduitmkn_REqpHFeqVNelBGHHk/edit Tbr since there was a "verbal" LGTM in cl comments. Tbr: nasko Bug: 830493 Change-Id: I38887bb97c817cc9182d71edd89d5ff0193b2504 Reviewed-on: https://chromium-review.googlesource.com/1032751 Commit-Queue: Max Morin <maxmorin@chromium.org> Reviewed-by:Olga Sharonova <olka@chromium.org> Cr-Commit-Position: refs/heads/master@{#557492}
-
Max Morin authored
We cannot retry anything yet, so make sure the renderer knows we're gone. Drive-by remove handling of navigations, since it was crashing. Bug: 787806, 840345 Change-Id: Icbaf49dda9ca12094ada2e9decc57622cd1feff6 Reviewed-on: https://chromium-review.googlesource.com/1047209 Commit-Queue: Max Morin <maxmorin@chromium.org> Reviewed-by:
-
Takashi Toyoshima authored
Current PreflightController implementation uses an existing error code to report disallowed preflight redirects tentatively, and results in wrong console messages. This change defines a dedicated error code and use GetErrorString() to generate a console message even for legacy code path. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I1af9b8570c2811a9a575e4504b88d6b37db688ea Bug: 803766 Tbr: kinuko@chromium.org Reviewed-on: https://chromium-review.googlesource.com/1027558Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org> Cr-Commit-Position: refs/heads/master@{#557490}
-
Yutaka Hirano authored
This reverts commit 189d1033. Reason for revert: "ParallelDownloadTest.NoPartialResponse" is flaky. See https://crbug.com/841666. Original change's description: > Cancel parallel download request if server doesn't send partial response > > If the server doesn't support partial responses, Chrome should cancel > parallel download requests. > These requests result in the wrong write offset and makes the file larger > than the original > > BUG=838627 > > Change-Id: I1fa878854579a1572e6e2f90f3bb932b59b333ba > Reviewed-on: https://chromium-review.googlesource.com/1048130 > Commit-Queue: Min Qin <qinmin@chromium.org> > Reviewed-by: Xing Liu <xingliu@chromium.org> > Cr-Commit-Position: refs/heads/master@{#557252} TBR=qinmin@chromium.org,xingliu@chromium.org Change-Id: I995c3c48fe39c535275a353130b10204009c5e30 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 838627, 841666 Reviewed-on: https://chromium-review.googlesource.com/1053470Reviewed-by:
-
Stuart Langley authored
Some cleanup work for chrome/browser/chromeos/file_manager/ briging it in line with recomended practices/c++ modernization. Modernize clang-tidy checks that were used: -- modernize-use-default -- modernize-make-unique -- modernize-raw-string-literal -- modernize-use-emplace -- modernize-use-nullptr This CL introduces NO logic changes. Bug: 841659 Change-Id: Icb8803941bd7f2230cb0174c8e307d712ed0e10a Reviewed-on: https://chromium-review.googlesource.com/1053668Reviewed-by:
-
Mohamed Amir Yosef authored
The functionality of handling remote creation of bookmarks has been introduced in this patch https://chromium-review.googlesource.com/c/chromium/src/+/1017201 which supports remote creation of bookmarks as well as folders in the bookmark tree. This CL 1- Enables DownloadBookmarkFolder test for USS. 2- Removes the redudant DownloadBookmark test. It has been recently introduced however it tests the same behavior as InjectedBookmark test case. 3- Enables InjectedBookmark test for USS instead of the removed DownloadBookmark test. Bug: 516866 Change-Id: Ida235e4817fab122448b88f063dbabdbe7ef7712 Reviewed-on: https://chromium-review.googlesource.com/1052247Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Cr-Commit-Position: refs/heads/master@{#557487}
-
Anita Woodruff authored
- Enables the mojo pathway by default and removes the legacy IPC code path for persistent notifications. - Removes the NotificationsWithMojo feature flag which previously guarded this code-path during incremental development. R=kinuko@chromium.org Bug: 796991 Change-Id: Ia4dac0ab4b98b7a6df342b7df1b424e3af67bf05 Reviewed-on: https://chromium-review.googlesource.com/1027843Reviewed-by:
Peter Beverloo <peter@chromium.org> Commit-Queue: Anita Woodruff <awdf@chromium.org> Cr-Commit-Position: refs/heads/master@{#557486}
-
Mikel Astiz authored
The patch adds a button in chrome://sync-internals that triggers logic to get updates from the sync server, similar to the time-based polls in place (in case invalidations don't work as expected). This is primarily expected to help testing teams. Bug: 839836 Change-Id: I1060026a6e019d0531b4489331294efb6e5aff9c Reviewed-on: https://chromium-review.googlesource.com/1047868 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#557485}
-
Hajime Hoshi authored
We plan to eliminate kUnthrottled, kUnspecedTimer and kUnspecedLoading and replace them with appropriate kInternal* task types. Task type guideline: https://bit.ly/2vMAsQ4 Bug: 830489 Change-Id: I465769461f4db57031e49c391f0efe715e2dc067 Reviewed-on: https://chromium-review.googlesource.com/1046730 Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Reviewed-by:
-
Rune Lillesveen authored
Instead of modifying ComputedStyle in place for margins, clone the style if necessary before setting the new margins. UpdateMarginsAndContent() has been modified to call ComputePreferredLogicalWidths() in the case of dirty preferred widths because that method will call UpdateContent() and UpdateMargins(). We had problems with UpdateMargins() being re-entered causing the style member of LayoutObject to change multiple times during the process when MinPreferredLogicalWidth() was called during UpdateMargins(). Bug: 813068 Change-Id: I902de039f6291ca56c47f88b1f69eee06afb18ca Reviewed-on: https://chromium-review.googlesource.com/1051807 Commit-Queue: Rune Lillesveen <futhark@chromium.org> Reviewed-by:
cathie chen <cathiechen@tencent.com> Cr-Commit-Position: refs/heads/master@{#557483}
-
Hitoshi Yoshida authored
SVGAnimatedProperty::context_element_ and SVGPropertyTearOff::context_element_ were stored as UntracedMember, because they were thought to have a reference cycle involving Persistent. However, SVGInterpolation was removed and the cycle was cut. https://codereview.chromium.org/1607633003 And hence we no longer need to store them as UntracedMember. Bug: 528275 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Iefdda45d9d56a7e814be43a871cbaa37d55e3b2a Reviewed-on: https://chromium-review.googlesource.com/1053619Reviewed-by:
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/baf831d074e7..d31b2f674acc $ git log baf831d07..d31b2f674 --date=short --no-merges --format='%ad %ae %s' 2018-05-10 skia-bookmaker Update markdown files 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;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=egdaniel@chromium.org Change-Id: I2533641be20a9cce435dfac8c5f6330cbcae6c28 Reviewed-on: https://chromium-review.googlesource.com/1053689 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#557481}
-
Rune Lillesveen authored
-webkit-user-modify is not set on the input element by disabled or readonly attributes, but on the inner editor based on the input attributes. The style recalc is triggered from StyleDidChange on LayoutTextControl for the input element. However, if the input element style does not change StyleDidChange is not called, and the inner editor style is not updated. Trigger the style update when readonly and disabled attributes change. Bug: 840833 Change-Id: Iad291b0a079efd197074366e127d1aa3a1f757e7 Reviewed-on: https://chromium-review.googlesource.com/1051236Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#557480}
-
junweifu authored
Add nose landmark enum in the mojom definition and the IDL to match the Spec[1]. Get nose information with Vision Framework on Mac and GMS core on Android platform. Link Mac 10.13 build bots [2] and Face Detection demo[3] here. [1] https://github.com/WICG/shape-detection-api/pull/38 [2] https://ci.chromium.org/buildbot/chromium.fyi/Chromium%20Mac%2010.13/ [3] https://codepen.io/miguelao/pen/PmJWro BUG=835078 Cq-Include-Trybots: luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng Change-Id: I0818bf817bf047a19ec7b34b232c93e4d08e9fa2 Reviewed-on: https://chromium-review.googlesource.com/1020385Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Junwei Fu <junwei.fu@intel.com> Cr-Commit-Position: refs/heads/master@{#557479}
-
Trent Apted authored
The dialog may outlive the WebContents, but ExternalProtocolDialogDelegate::DoAccept() is missing a check on the result of tab_util::GetWebContentsByID() before dereferencing the WebContents. Add a check and a regression test. In order to achieve coverage of the crashing lines, update the test harness and the static method involved to use a ExternalProtocolHandler::Delegate. This interface already exists and is used for testing, but wasn't in the codepaths invoked from the crashing code in ExternalProtocolDialogDelegate::DoAccept(). Bug: 835216 Change-Id: If7d4593090a1db9a96bc829d1e1c253860d0cf20 Reviewed-on: https://chromium-review.googlesource.com/1049774Reviewed-by:
Ben Wells <benwells@chromium.org> Commit-Queue: Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#557478}
-
Ben Wells authored
The CrostiniManager can send multiple error callbacks. Only the first one should be shown to the user. Bug: 841158 Change-Id: If21222f81b609559a462a73af8c5a7bc911592fb Reviewed-on: https://chromium-review.googlesource.com/1051406 Commit-Queue: Ben Wells <benwells@chromium.org> Reviewed-by:
Nicholas Verne <nverne@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#557477}
-
Felix Ekblom authored
BUG=b:78171027 Change-Id: I08a32e347c103f790e399264d8d045b36848d492 Reviewed-on: https://chromium-review.googlesource.com/1042576 Commit-Queue: Felix Ekblom <felixe@chromium.org> Reviewed-by:
Toni Barzic <tbarzic@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Cr-Commit-Position: refs/heads/master@{#557476}
-
https://webrtc.googlesource.com/src.git/+log/e53ac0463daa..b4e0e50a2bbf $ git log e53ac0463..b4e0e50a2 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=chromium:None 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: Iea5aada68e7bbfa582f99f9b910aee4704ead592 Reviewed-on: https://chromium-review.googlesource.com/1053587Reviewed-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@{#557475}
-
Chromium WPT Sync authored
Using wpt-import in Chromium afe927cd. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/17225 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 TBR=qyearsley No-Export: true Change-Id: Ia5465a7ed1514f1bc740b2c95b8bca815e707b36 Reviewed-on: https://chromium-review.googlesource.com/1053570 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
-
Matt Falkenhagen authored
Reland "service worker: Make provider hosts for reserved clients first-class members of SWContextCore." Relands 0ce272e0. Original code review: https://chromium-review.googlesource.com/1051145 Reverted in: https://chromium-review.googlesource.com/1053469 The difference is this initializes ServiceWorkerNavigationHandleCore's |provider_id_| to -1. So the dtor won't use unintitialized memory when a provider was never created. Bug: 841070 Change-Id: I2ec537f76f093dd35a99f14c9033d75d50a7313d Reviewed-on: https://chromium-review.googlesource.com/1051145Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#557423} Reviewed-on: https://chromium-review.googlesource.com/1053609 Cr-Commit-Position: refs/heads/master@{#557473}
-
Aleks Totic authored
Bug: 714962 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: I94905e3ed6e80663b62dd63cc5f3de5b89317917 Reviewed-on: https://chromium-review.googlesource.com/1053372 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#557472}
-
Noel Gordon authored
Bug: 833834 Change-Id: I208207b46af243ce36fc810363b3e727eee41a2d Reviewed-on: https://chromium-review.googlesource.com/1053331 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Reviewed-by:
-
Owen Rodley authored
This is a reland of e481b7b5 This broke the index pack because it introduces absolute paths. Now that https://crrev.com/c/1053044 fixes the handling of absolute paths, this should be safe to reland. Original change's description: > Avoid searching the compilation database for every file. > > The compilation database for chromium is ~180MB of JSON. The current > implementation of "run_tool.py --all" reads the whole database, pulls out > only the filenames, and then has the tool rescan the compilation > database for the flags. We can instead extract the flags as well, and > pass them directly to the tool, to avoid the rescanning. > > A local run of translation_unit over the whole repository took 44m15s > before this change, and 17m23s after. > > Change-Id: I373238980654a5586e32f5a516f0953a4eec03c3 > Reviewed-on: https://chromium-review.googlesource.com/1039203 > Commit-Queue: Owen Rodley <orodley@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#556653} Change-Id: I28c564fa02d3baf16358170ee4fd5985c44506ba Reviewed-on: https://chromium-review.googlesource.com/1053647Reviewed-by:
-
Xiaohan Wang authored
Bug: 840586 Change-Id: I0d91bde150d6e3755df3a3577862149298eaa61f Reviewed-on: https://chromium-review.googlesource.com/1050934Reviewed-by:
Frank Liberato <liberato@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#557469}
-
