- 14 May, 2018 40 commits
-
-
Matt Falkenhagen authored
The test was disabled because Chrome didn't implement updateViaCache and the expected default behavior, but that has been enabled as an experimental feature for a while. Also, delete the corresponding version in http/tests, which used cache-control headers to workaround the lack of updateViaCache/expected behavior. Bug: 675540 Change-Id: Id95750c5c3a6d4c4d86d307bbce4477b9ee53df1 Reviewed-on: https://chromium-review.googlesource.com/1056755Reviewed-by:
Han Leon <leon.han@intel.com> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#558223}
-
Gauthier Ambard authored
This CL tries to fix the EG test for the TabSwitcher by making sure the incognito card of the TabSwitcher is interactable before trying to swipe it. Bug: 842082 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I25aa176aa63f1d228b2604d0b8b9f58f20297cad Reviewed-on: https://chromium-review.googlesource.com/1055388Reviewed-by:
edchin <edchin@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#558222}
-
Gauthier Ambard authored
This CL disable a sync EG test failing almost consistently on iPhone simulator iOS 10 and which is flaky on other configuration. As all the other tests of this test suite are disabled, I am disabling it instead of marking it as flaky, as there is probably something wrong with the whole test suite. Bug: 821490 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I5bada13c540bc3d95c2082109d3e9299b873cc39 Reviewed-on: https://chromium-review.googlesource.com/1054233Reviewed-by:
edchin <edchin@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#558221}
-
Timothy Loh authored
This reverts commit 738768f5. Reason for revert: Times out on MSAN https://ci.chromium.org/buildbot/chromium.memory/Linux%20ChromiumOS%20MSan%20Tests/7143 Original change's description: > Re-enable OpenAudioFiles/FileManagerBrowserTest > > Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. > > Bug: 836254 > Change-Id: Ie6ca28f923dcf70be407c8782985195874f47758 > Reviewed-on: https://chromium-review.googlesource.com/1056748 > Reviewed-by: Stuart Langley <slangley@chromium.org> > Commit-Queue: Noel Gordon <noel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#558168} TBR=noel@chromium.org,fukino@chromium.org,slangley@chromium.org Change-Id: Ib066688090f770c251b333db4446d92f52951a0e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 836254 Reviewed-on: https://chromium-review.googlesource.com/1056657Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558220}
-
Hayato Ito authored
Keep track of failing tests in media/controls for IncrementalShadowDOM. Bug: 776656 Change-Id: Ib0f09da71bdc49935875b2d0814336ea64b90d31 Reviewed-on: https://chromium-review.googlesource.com/1056750 Commit-Queue: Hayato Ito <hayato@chromium.org> Reviewed-by:
Takayoshi Kochi <kochi@chromium.org> Cr-Commit-Position: refs/heads/master@{#558219}
-
Noel Gordon authored
Document these tests, and change test names to match. Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I156a6d5823e86d94b48b5706e3be9ba2c07562e0 Reviewed-on: https://chromium-review.googlesource.com/1056656Reviewed-by:
Sasha Morrissey <sashab@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558218}
-
Noel Gordon authored
When you "create" a folder, it is "new". Ditch the "new" in these test names, make "CreateFolder" the common sub-string. The first test does some sort of selection before calling CreateFolder so prefix "select" in that test's name. No-Presubmit: true Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ie559508c6688d0493ef0866635e35a1944e11cbc Reviewed-on: https://chromium-review.googlesource.com/1056450 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Sasha Morrissey <sashab@chromium.org> Cr-Commit-Position: refs/heads/master@{#558217}
-
Noel Gordon authored
The names of these tests is above and beyond. Provide a commomn prefix and shorten these names to something more sensible. Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I6a903c3fd8fbe91654b4c6fdf67e8f0474bc24e2 Reviewed-on: https://chromium-review.googlesource.com/1056777Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558216}
-
Timothy Loh authored
This reverts commit 6d0559c0. Reason for revert: Breaks ImageBitmapTest.AvoidGPUReadback on MSAN, e.g. https://logs.chromium.org/v/?s=chromium%2Fbb%2Fchromium.memory%2FLinux_MSan_Tests%2F9783%2F%2B%2Frecipes%2Fsteps%2Fwebkit_unit_tests%2F0%2Flogs%2FImageBitmapTest.AvoidGPUReadback%2F0 Original change's description: > Use Skia's new simplified read/write pixels code paths > > Bug: 826819 > Change-Id: Ia6eaca75fe43f9d9ec94f8fe644a44e8e7436675 > Reviewed-on: https://chromium-review.googlesource.com/1054520 > Reviewed-by: Robert Phillips <robertphillips@google.com> > Commit-Queue: Brian Salomon <bsalomon@chromium.org> > Cr-Commit-Position: refs/heads/master@{#557943} TBR=bsalomon@chromium.org,robertphillips@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 826819 Change-Id: Ia4dca1186fbb79569672fbf5cdabd9657c702762 Reviewed-on: https://chromium-review.googlesource.com/1056847Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558215}
-
Hajime Hoshi authored
The task runner is used at HandlePendingEventsAndPromises, and I think an unthorttled task runner is not necessary here. WebGLSync: The unthrottled task runenr was introduced at https://chromium-review.googlesource.com/c/chromium/src/+/906402 WebGLQuery and WebGLTimerQueryExt: The unthrottled task runner was introduced at https://codereview.chromium.org/2341043002 We plan to eliminate kUnthrottled, kUnspecedTimer and kUnspecedLoading and replace them with appropriate kInternal* task types. Task type guideline: https://bit.ly/2vMAsQ4 Bug: 836410 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 Change-Id: I2cc48425aaef973432f8bdee6fbb111b0e9a9188 Reviewed-on: https://chromium-review.googlesource.com/1055369Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#558214}
-
Sam McNally authored
When a user does not have team drives enabled, DriveFS does not create a team_drives directory. If the team drives flag is enabled, file manager attempts to resolve the path to /team_drives within the mount, failing when DriveFS is enabled. Treat a missing /team_drives directory the same as when the team drives flag is disabled and leave teamDriveDisplayRoot_ set to null. Bug: 829274 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: If66fadd3a0e41ac420d43de144b89b489ffde1d6 Reviewed-on: https://chromium-review.googlesource.com/1056933Reviewed-by:
Sasha Morrissey <sashab@chromium.org> Commit-Queue: Sam McNally <sammc@chromium.org> Cr-Commit-Position: refs/heads/master@{#558213}
-
Noel Gordon authored
What it says on the tin, plus minor test comment updates. Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Iefe801b3167629271826d457cf7b92473c902638 Reviewed-on: https://chromium-review.googlesource.com/1056774Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558212}
-
Noel Gordon authored
Document the test, change test names to have a common prefix. Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id2304840000c7fbe69caa281e31f9f79d0cd1e0a Reviewed-on: https://chromium-review.googlesource.com/1056753Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558211}
-
Noel Gordon authored
Use the same prefix for gear tests. Change test names for consistency with the rest of the integration test JS files. Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I6348927f9df141e86fdccf484afae5d432c9b316 Reviewed-on: https://chromium-review.googlesource.com/1056651Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558210}
-
Mike Wasserman authored
Make a shared helper for getting the window type from properties. Apply properties to the window like the old WindowTreeClient. Bug: 837695 Change-Id: Icfe1efcddb52b2c3ef9d3ed218e02c105885e230 Reviewed-on: https://chromium-review.googlesource.com/1056349 Commit-Queue: Michael Wasserman <msw@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#558209}
-
Koji Ishii authored
Following bot results are included. 5407 5409 TBR=eae@chromium.org, mstensho@chromium.org NOTRY=true Bug: 591099 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I9a4b34b49b723b49b4f3153141153fd0205497b2 Reviewed-on: https://chromium-review.googlesource.com/1056907 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#558208}
-
Hitoshi Yoshida authored
Some bindings code generator code depend on the paths of output directories to generate their contents. This CL changes the directory to output temporary files to test, and makes it possible to remove test-only hacks in code generators. Bug: 830255 Change-Id: I67b80489fcb3752679770cb56fa5c442c657b181 Reviewed-on: https://chromium-review.googlesource.com/1056778Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#558207}
-
Ramin Halavati authored
A minor bug in processing safe list in traffic annotation auditor is fixed. Bug: 690323 Change-Id: I76f6d023f88106bf69b6c5546142afb52e8e37c5 Reviewed-on: https://chromium-review.googlesource.com/1054877 Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Georges Khalil <georgesak@chromium.org> Cr-Commit-Position: refs/heads/master@{#558206}
-
afdo-chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 The AutoRoll server is located here: https://afdo-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=gbiv@chromium.org Change-Id: I516955bcf67b5c21b789efad2c90c046c70127ce Reviewed-on: https://chromium-review.googlesource.com/1056794 Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558205}
-
Yutaka Hirano authored
Bug: 570946 Change-Id: Iba0dd0c5af2d44a8baff9c4e5764fb9a624f719c Reviewed-on: https://chromium-review.googlesource.com/1056929Reviewed-by:
Adam Rice <ricea@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#558204}
-
Hajime Hoshi authored
kInternalAnimation was introduced by replacing kUnspecedTimer, which is now kInternalDefault [1]. As we defined the task type guideline [2], the default behavior task runners should use kInternalDefault. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1009448 [2] https://bit.ly/2vMAsQ4, BUG: 836410 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ib5788929dce6331fc8deee8d93df9b351f6d83ca Reviewed-on: https://chromium-review.googlesource.com/1055353Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#558203}
-
Xiaocheng Hu authored
This patch removes the general member functions in RenderedPosition that inspect properties of bidi runs, and replace them by functions providing the exact information needed by SelectionController, which is the only client of RenderedPosition. Details: - Introduced RenderedPosition::BidiBoundaryType, so that we can obtain whether an RP is at bidi boundary without re-comparing its offset. - Replaced almost all existing member functions by - AtBidiBoundary(), replacing AtL/RBoundaryOfBidiRun() - BidiRunContains(other_rp), replacing |other_rp == this->L/RBoundaryOfBidiRun(other_rp.BidiLevelOnR/L())| - IsPossiblyOtherBoundaryOf(other_rp), replacing |this->AtL/RBoundaryOfBidiRun(other_rp.BidiLevelOnL/R())| Bug: 811502 Change-Id: I0b59b76e8060a5953295e09c98fcf7127a636ecc Reviewed-on: https://chromium-review.googlesource.com/1056220Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558202}
-
Noel Gordon authored
Document these tests. Also change them to have a common prefix. Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I0c8d37efe4d9f380c17491b30b38c90d31204098 Reviewed-on: https://chromium-review.googlesource.com/1056752Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558201}
-
Hwanseung Lee authored
the button in toasts of Bookmarks should not have a border. Bug: 842013 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I2d431ba60b598125730eaf56b618fe3e9b1d9ccd Reviewed-on: https://chromium-review.googlesource.com/1055132 Commit-Queue: Hwanseung Lee <hwanseung@chromium.org> Reviewed-by:
calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#558200}
-
Noel Gordon authored
122:47 error Missing semicolon 515:52 error Missing semicolon Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id30d0aa122c0a34ac75b921de484ee5d2f376403 Reviewed-on: https://chromium-review.googlesource.com/1056888Reviewed-by:
Sasha Morrissey <sashab@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558199}
-
Timothy Loh authored
This reverts commit d9bf5454. Reason for revert: Breaks ExtensionApiTestWithManagementPolicy.WebRequestProtectedByPolicy on Linux Debug https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=not_site_per_process_browser_tests&tests=ExtensionApiTestWithManagementPolicy.WebRequestProtectedByPolicy https://logs.chromium.org/v/?s=chromium%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8946590247605008336%2F%2B%2Fsteps%2Fnot_site_per_process_browser_tests%2F0%2Flogs%2FExtensionApiTestWithManagementPolicy.WebRequestProtectedByPolicy%2F0 Original change's description: > [Extensions Click-to-Script] Change default to allowing host permissions > > Change the default behavior of the kRuntimeHostPermissions feature to > allow extensions to have origin access, rather than defaulting to > withholding permission. In the first stage of click-to-script, we don't > want to change any defaults. > > Bug: 841099 > Change-Id: Ibdcbfb0fd9ecc535af73a1bd9048e196eab135e1 > Reviewed-on: https://chromium-review.googlesource.com/1052684 > Commit-Queue: Devlin <rdevlin.cronin@chromium.org> > Reviewed-by: Karan Bhatia <karandeepb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#558079} TBR=rdevlin.cronin@chromium.org,karandeepb@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 841099, 842466 Change-Id: Id37af8c43bbf30d8963852df9b29b58ff3ca5a53 Reviewed-on: https://chromium-review.googlesource.com/1056887 Commit-Queue: Timothy Loh <timloh@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558198}
-
Hitoshi Yoshida authored
The target object is already traced in Oilpan from a static persistent map, but this CL adds a trace to make TraceWrappers() a subset of Trace() for unification. Bug: 841830 Change-Id: I3bec0361aa34ac7a0dc82fcb0cf3d823ea58057a Reviewed-on: https://chromium-review.googlesource.com/1055013 Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#558197}
-
Noel Gordon authored
Consistency: other tests use "Folder" instead of "Directory". Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I8870cc25a39a54c6c06d2d5a2560652e854b5e99 Reviewed-on: https://chromium-review.googlesource.com/1056769Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558196}
-
Noel Gordon authored
Document these tests, and remove redundant "State" from then end of a the geometry maximized test. No-Presubmit: true Bug: 836254 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I261dc8083e19eae5b6252916b6abecb64eef57e1 Reviewed-on: https://chromium-review.googlesource.com/1056749 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Luciano Pacheco (SYD) <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#558195}
-
Nicholas Verne authored
CrostiniRestarterService and CrostiniRestarterServiceFactory are kept local to CrostiniManager (for now) as they don't need to be exposed to other code. There is one CrostiniRestarterService per profile, and it owns CrostiniRestarter objects for that profile. When multiple calls to CrostiniManager::RestartCrostini are made due to user action, only one actual restart flow will run, and when it completes, multiple completion callbacks will run in succession. Bug: 839207 Change-Id: Icbf1da00fd9c937cd51bb77f0ad57bd63476b27b Reviewed-on: https://chromium-review.googlesource.com/1051208 Commit-Queue: Nicholas Verne <nverne@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558194}
-
Ben Wells authored
The comment was incorrect. Bug: None Change-Id: Ifbacb882482dd834198046293b798da27b7b4964 Reviewed-on: https://chromium-review.googlesource.com/1055247 Commit-Queue: Ben Wells <benwells@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#558193}
-
Donna Wu authored
With this CL, the singleton device::MediaTransferProtocolManager is owned by device::mojom::MtpManager, and usage of the former MTP manager has also been converted to Mojo MTP interfaces. Next, we will merge device/media_transfer_protocol/ into the implementation of device::mojom::MtpManager. BUG=769630 Change-Id: I6a2b5d0bf388152e953bb7af3d13e85c2272e570 Reviewed-on: https://chromium-review.googlesource.com/1032237 Commit-Queue: Donna Wu <donna.wu@intel.com> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#558192}
-
Ryan Hamilton authored
QuicClock is already part of the QUIC platform API. It's an abstraction for a clock. We don't gain anything by creating another clock abstraction and wrapping that up in a QuicClock. QuicAlarms are currently implemented using a task runner interface. There's no reason to change the abstraction from alarms to a task runner. Eliminating the task runner interface and using the QUIC alarm abstraction directly makes it easier to use the Quartc QUIC wrappers in a setting with an existing alarm implementation (eg. chromium). This might be useful in implementing RTCQuicStream. Quartc itself would like to use at least one alarm (see b/78013844). Implementing QuicAlarms directly in Quartc makes this easy. Merge internal change: 195335187 Change-Id: I2a0a60044e03c1d1893714ebe13492932ed87baf Reviewed-on: https://chromium-review.googlesource.com/1055938 Commit-Queue: Zhongyi Shi <zhongyi@chromium.org> Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#558191}
-
https://chromium.googlesource.com/chromiumos/chromite.git/+log/193c9144b170..b15c2c28232c $ git log 193c9144b..b15c2c282 --date=short --no-merges --format='%ad %ae %s' 2018-05-09 achuith cros_run_vm_test: Fix remote directory creation. Created with: roll-dep src/third_party/chromite BUG=chromium:829612 The AutoRoll server is located here: https://chromite-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=chrome-os-gardeners@chromium.org Change-Id: Ic04b37f26f6554a286a21836d2f1fb5fa913fedd Reviewed-on: https://chromium-review.googlesource.com/1056791 Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558190}
-
Giovanni Ortuño Urquidi authored
Before, HostedAppBrowserController would set/unset the strict mixed content preference on the main RenderViewHost when a WebContents was attached/detached from a PWA Browser window. This worked when Site Isolation was not enabled because all frames in the WebContents shared the same RenderViewHost. Now that Site Isolation is enabled, cross-origin iframes no longer share the same RenderViewHost resulting in the strict mixed content preference to not be applied to them. With Site Isolation enabled we need to handle two cases: (1) Apply the new preference on existing iframes (2) Apply the new preference on new iframes To handle case (1), this patch switches from UpdateWebKitPreferences to NotifyPreferencesChanged which eventually triggers a preference update on all RenderViewHosts in the WebContents. NotifyPreferencesChanged doesn't take any arguments, it just recomputes the preferences, so this patch modifies ChromeContentBrowserClient to check if the WebContents is in a PWA Browser window and set the preference accordingly. The above changes also handle case (2). When a new iframe is created, OverrideWebkitPrefs is called for its RenderViewHost and if the WebContents is in a PWA Browser window then the preference is set. Bug: 829688 Change-Id: Idb11bf979ec76ae8e1dfc905bf68e3c7ad1dbd6f Reviewed-on: https://chromium-review.googlesource.com/1048885 Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Cr-Commit-Position: refs/heads/master@{#558189}
-
Koji Ishii authored
Following bot results are included. 5346 5349 5356 5363 5367 5369 5377 5378 5380 5386 5389 5392 5393 5394 5395 5398 5402 5403 5404 5405 19 lines were removed and 10 lines were deflaked by consecutive results since 5346. TBR=eae@chromium.org, mstensho@chromium.org NOTRY=true Bug: 591099 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I3235231b21da2bb205ba84be22fca6e1752ce658 Reviewed-on: https://chromium-review.googlesource.com/1053927 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#558188}
-
Timothy Loh authored
Disables: - ScrollEventToOOPIF on TSAN - OverlapSurfaceHitTestTest on Linux (was: TSAN) - HitTestWatermark on Linux (was: TSAN) - HitTestLayerSquashing on Linux - CrossProcessMouseCapture on Linux TBR=kenrb@chromium.org Bug: 833380 Change-Id: Ia8539488479c19e0dbeacf6bc0c16667f325ebd9 Reviewed-on: https://chromium-review.googlesource.com/1056775Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558187}
-
Kevin Marshall authored
The ability to propagate files affords the sandboxing policy logic more precision in propagating the exact capabilities needed by child processes. Also some minor cleanup which changes the LaunchOptions interface to use FilePaths instead of strings for representing paths. Bug: 750938 Change-Id: I38059c39f629bc9234e4f94c0b215a39828f3665 Reviewed-on: https://chromium-review.googlesource.com/1050494 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#558186}
-
Joel Hockey authored
Change-Id: I107242a7118b5eaa0300c5a04c3ac48edf60dc5f Reviewed-on: https://chromium-review.googlesource.com/1056650Reviewed-by:
Nicholas Verne <nverne@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#558185}
-
Kent Tamura authored
Change-Id: I7d22ee25a4b4b55e4de846db6ec79919e703ac3e Reviewed-on: https://chromium-review.googlesource.com/1051185Reviewed-by:
Quinten Yearsley <qyearsley@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#558184}
-