- 10 May, 2018 40 commits
-
-
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:
Noel Gordon <noel@chromium.org> Commit-Queue: Stuart Langley <slangley@chromium.org> Cr-Commit-Position: refs/heads/master@{#557488}
-
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:
Stuart Langley <slangley@chromium.org> Cr-Commit-Position: refs/heads/master@{#557471}
-
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}
-
Fumitoshi Ukai authored
release clobber debug clobber debug Bug: 835731 Change-Id: I253025e12c36fe2ee6bad941721bc79658de8062 Reviewed-on: https://chromium-review.googlesource.com/1029569Reviewed-by:
Shinya Kawanaka <shinyak@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Fumitoshi Ukai <ukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#557468}
-
Hiroki Nakagawa authored
According to a recent spec change, this CL changes the default credentials mode on WorkletOptions from 'omit' to 'same-origin'. Spec issue: https://github.com/w3c/css-houdini-drafts/issues/756 Feature dashboard: https://www.chromestatus.com/feature/6383899018002432 Bug: 838540 Change-Id: I341f9c61e11387fe7192d75e616b4003fa36128f Reviewed-on: https://chromium-review.googlesource.com/1051307Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#557467}
-
Adam Rice authored
The http/tests/websocket/workers/no-onmessage-in-sync-op.html layout test had a 2 second busy loop. Reduce it to 0.2 seconds. It would be possible to avoid the busy loop by using sync XHR instead, but this might change the semantics of the test. It is particularly important not to change any semantics as this is a regression test for https://bugs.webkit.org/show_bug.cgi?id=76521. The length of the delay has been removed from the test output so that the expectations won't need to be changed if the delay is changed again. No other changes have been made to the test. In particular, it hasn't been updated to a modern style. BUG=386044 Change-Id: I38c317d89b96e7e3729225603c742d99359959e9 Reviewed-on: https://chromium-review.googlesource.com/1046686 Commit-Queue: Adam Rice <ricea@chromium.org> Reviewed-by:
-
Matt Falkenhagen authored
This CL makes it so ServiceWorkerSubresourceLoaderFactory uses a factory created from a ChildURLLoaderFactoryBundle rather than the direct network factory for "network fallback" after service worker does not provide a response to a FetchEvent. The direct network factory didn't work in the case of chrome-extension:// URLs, since network service does not understand such URLs. To create the fallback factory, we use the default ChildURLLoaderFactoryBundle of the RenderFrameImpl, except remove the "default factory" from that bundle. The default factory can be something like the AppCache loader factory, which we don't want to hit when performing network fallback. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: If6ae1d6fc161cae8ec4574417a0d8954215f878e Bug: 836129 Reviewed-on: https://chromium-review.googlesource.com/1032354 Commit-Queue: Matt Falkenhagen <falken@chromium.org> Reviewed-by:
-
Hitoshi Yoshida authored
- Drops two unnecessary TraceWrappers(). They just call TW() of parent classes. - Add a trace in Trace(). Bug: 581412 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.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I726fcea1872488ab98541b8a7cce233793815da8 Reviewed-on: https://chromium-review.googlesource.com/1053336Reviewed-by:
-
Maksim Sisov authored
This is a tiny patch, which adds a check if platform_window has already been destroyed by the time ::Close was called. Change-Id: I3413fae38f9574cea4f806755e6738289efda1f5 Reviewed-on: https://chromium-review.googlesource.com/1051827Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Maksim Sisov <msisov@igalia.com> Cr-Commit-Position: refs/heads/master@{#557463}
-
John Abd-El-Malek authored
The FrameTreeNode's initial RenderFrameHost might change during navigations if a cross-origin redirect happens. So caching the initial RPH/RFH IDs would be incorrect. For the navigation case, the only thing that is needed is the BrowserContext so pass that in directly. The security check (AllowExtensionResourceLoad) isn't needed for navigations, as that's done in ExtensionNavigationThrottle per the comment in the file. This makes ExtensionURLLoaderFactory pass in -1 for the procss ID when the network service is running, to match what ExtensionProtocolHandler does for the non-network-service case. Bug: 840600 Change-Id: Ie23ed9e78f69fd4a8bb7a1a4aa964303db2102cf Reviewed-on: https://chromium-review.googlesource.com/1050812 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#557462}
-
Ryo Hashimoto authored
BUG=837161 TEST=build Change-Id: I99299e9f4066dd445458325c264743211e1c6645 Reviewed-on: https://chromium-review.googlesource.com/1051546Reviewed-by:
Dan Erat <derat@chromium.org> Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#557461}
-
