commit | c98b9172911cedfff9bcaa200e595e07f0cf09da | [log] [tgz] |
---|---|---|
author | Yuheng Huang <[email protected]> | Thu Jan 20 18:12:47 2022 |
committer | Chromium LUCI CQ <[email protected]> | Thu Jan 20 18:12:47 2022 |
tree | 5bdbf629dc959790e03cf71eabede7ee2eeeec14 | |
parent | 8cbc03680bbd609a35dd05a034c8562ce8e681e5 [diff] [blame] |
Fix OSExchangeData::IsFromPrivileged does not convey on OSExchangeDataProvider::Clone() On WaylandDataDragController[1], OSExchangeData is copied using OSExchangeDataProvider::Clone(). This CL makes the recently added OSExchangeData::IsFromPrivileged() in CL[2] backed by OSExchangeDataProvider so it can convey on OSExchangeDataProvider::Clone(). The change fixes an issue of WebUI tab strip tab dragging not working within the same window in Lacros. [1] https://source.chromium.org/chromium/chromium/src/+/main:ui/ozone/platform/wayland/host/wayland_data_drag_controller.cc;l=224;drc=751bbeb73a0dacc98a2b9486e0cf788fe763356a;bpv=1;bpt=1 [2] https://chromium-review.googlesource.com/c/chromium/src/+/3343455 Bug: 1284996, 1257837, 1283077 Change-Id: If361d2e0f507be0c53ea57c8fd0a418b89291474 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3388844 Reviewed-by: Daniel Cheng <[email protected]> Reviewed-by: Sadrul Chowdhury <[email protected]> Commit-Queue: Yuheng Huang <[email protected]> Cr-Commit-Position: refs/heads/main@{#961529}
diff --git a/ui/base/dragdrop/os_exchange_data_provider_mac.h b/ui/base/dragdrop/os_exchange_data_provider_mac.h index 66518c1..5b2746a 100644 --- a/ui/base/dragdrop/os_exchange_data_provider_mac.h +++ b/ui/base/dragdrop/os_exchange_data_provider_mac.h
@@ -41,6 +41,8 @@ // Overridden from OSExchangeDataProvider: void MarkOriginatedFromRenderer() override; bool DidOriginateFromRenderer() const override; + void MarkAsFromPrivileged() override; + bool IsFromPrivileged() const override; void SetString(const std::u16string& data) override; void SetURL(const GURL& url, const std::u16string& title) override; void SetFilename(const base::FilePath& path) override;