45573 2010-09-10 16:28:07 -0700 [chromium] Fix incorrect drag node/selection painting. 2010-09-16 12:05:45 -0700 1 1 1 Unclassified WebKit Platform 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED P2 Normal --- 1 dcheng webkit-unassigned commit-queue eric jamesr simon.fraser tony oldest_to_newest 277525 0 dcheng 2010-09-10 16:28:07 -0700 If you dragged a selection near the bottom of the page, the selection would not be renderer correctly (it would be clipped as if the page hadn't been scrolled down). I also updated dragImageForNode to use RenderLayer::paint() as well, since that allows us to remove PaintBehavior from the RTTI helper class. I'm not 100% sure if the new code in Frame::dragImageForNode correct though--it uses document()->renderer() and node->renderer(), which is a little weird. However, if I just used RenderObject* renderer = node->renderer(), it would throw the following assert during the paint: ASSERTION FAILED: rootLayer == m_clipRectsRoot 277528 1 67260 dcheng 2010-09-10 16:31:36 -0700 Created attachment 67260 Patch 279319 2 eric 2010-09-14 18:58:17 -0700 James is really a better reviewer for this than I am. Is this how other ports do this? We seem to have a bunch of code which does manual painting. I know SVG code uses some paint-into-ImageBuffer code for SVGImage and masks. I wonder how many other places do/do-not want scroll clipping and if we should be creating a simple API for all these clients to use instead of grabbing at layers directly. 279328 3 jamesr 2010-09-14 19:07:50 -0700 I still don't understand what exact semantics you are going for here, so it's hard to judge whether the code does what you want. Where are the tests for this? 279370 4 67260 simon.fraser 2010-09-14 21:19:44 -0700 Comment on attachment 67260 Patch I think this will do the wrong thing with iframes and compositing. Iframes pull the paint behavior flags from their parent iframe (see -[WebFrame _drawRect:] in WebKit. Rather than paint via the RenderLayer, why not call m_view->paintContents, which does not intersect with the scrollview's frame? It would also be nice to share more code here with Mac's Frame::imageFromRect(). 279722 5 67716 dcheng 2010-09-15 14:25:59 -0700 Created attachment 67716 Patch 279726 6 67716 dcheng 2010-09-15 14:28:32 -0700 Comment on attachment 67716 Patch (In reply to comment #3) > I still don't understand what exact semantics you are going for here, so it's hard to judge whether the code does what you want. Where are the tests for this? There are no tests. The problem is ScrollView::paint() seemed to be clipping as if the page was never scrolled down. (In reply to comment #4) > (From update of attachment 67260 [details]) > I think this will do the wrong thing with iframes and compositing. Iframes pull the paint behavior flags from their parent iframe (see -[WebFrame _drawRect:] in WebKit. > > Rather than paint via the RenderLayer, why not call m_view->paintContents, which does not intersect with the scrollview's frame? > I didn't know about this function. I've updated the patch. > It would also be nice to share more code here with Mac's Frame::imageFromRect(). I based the implementation of these two functions off of Frame::imageFromRect(). However, they seemed to have a lot of Cocoa-specific magic, so I'm not sure how I could share any more code. Ideas are welcome. 280078 7 67716 commit-queue 2010-09-16 07:36:13 -0700 Comment on attachment 67716 Patch Rejecting patch 67716 from commit-queue. Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--exit-after-n-failures=1', '--wait-for-httpd', '--quiet']" exit_code: 1 Running build-dumprendertree Compiling Java tests make: Nothing to be done for `default'. Running tests from /Projects/CommitQueue/LayoutTests Testing 21296 test cases. websocket/tests/handshake-fail-by-sub-protocol-mismatch.html -> failed Exiting early after 1 failures. 21268 tests run. 529.38s total testing time 21267 test cases (99%) succeeded 1 test case (<1%) had incorrect layout 33 test cases (<1%) had stderr output Full output: http://queues.webkit.org/results/4063026 280238 8 tony 2010-09-16 11:44:36 -0700 Trying commit queue again because it doesn't seem that this code is run by websocket tests. 280252 9 67716 commit-queue 2010-09-16 12:05:39 -0700 Comment on attachment 67716 Patch Clearing flags on attachment: 67716 Committed r67645: <http://trac.webkit.org/changeset/67645> 280253 10 commit-queue 2010-09-16 12:05:45 -0700 All reviewed patches have been landed. Closing bug. 67260 2010-09-10 16:31:36 -0700 2010-09-15 14:25:56 -0700 Patch bug-45573-20100910163135.patch text/plain 4611 dcheng ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg NTA3NDkwMzVmZGMxNmEyNWYxMTVlYTJmNTIyYjg3YTQxODQzMDc1OC4uYmY2YTdmZmU3MjMzZmQy YWViMWQxMzllZmJlNmIxODhkYmEwODgwYSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwyMCBAQAorMjAxMC0wOS0xMCAgRGFu aWVsIENoZW5nICA8ZGNoZW5nQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBO T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbY2hyb21pdW1dIEZpeCBpbmNvcnJlY3QgZHJhZyBu b2RlL3NlbGVjdGlvbiBwYWludGluZy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTQ1NTczCisKKyAgICAgICAgRHJhZyBzZWxlY3Rpb25zIHdlcmUgYmVp bmcgaW5jb3JyZWN0bHkgY2xpcHBlZCB3aGVuIGRyYWdnaW5nIGEgc2VsZWN0aW9uCisgICAgICAg IG5lYXIgdGhlIGJvdHRvbSBvZiBhIHBhZ2UgdGhhdCByZXF1aXJlZCBzY3JvbGxpbmcuIFRvIGZp eCB0aGlzLCB3ZSB1c2UKKyAgICAgICAgUmVuZGVyTGF5ZXI6OnBhaW50KCkgZGlyZWN0bHkgaW5z dGVhZCBvZiBTY3JvbGxWaWV3OjpwYWludCgpLgorCisgICAgICAgIE5vIG5ldyB0ZXN0cy4KKwor ICAgICAgICAqIHBhZ2UvY2hyb21pdW0vRnJhbWVDaHJvbWl1bS5jcHA6CisgICAgICAgIChXZWJD b3JlOjpGcmFtZTo6bm9kZUltYWdlKToKKyAgICAgICAgKFdlYkNvcmU6OkZyYW1lOjpkcmFnSW1h Z2VGb3JTZWxlY3Rpb24pOgorCiAyMDEwLTA5LTA3ICBKYW4gRSBIYW5zc2VuICA8amhhbnNzZW5A c2VuY2hhLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBBbmRyZWFzIEtsaW5nLgpkaWZmIC0t Z2l0IGEvV2ViQ29yZS9wYWdlL2Nocm9taXVtL0ZyYW1lQ2hyb21pdW0uY3BwIGIvV2ViQ29yZS9w YWdlL2Nocm9taXVtL0ZyYW1lQ2hyb21pdW0uY3BwCmluZGV4IDQxNDZiMzlhN2JjYmI1ZDY1MDc4 NzgxZDc0Yzc5MTlmZmUxODkzNjcuLjkxZmZkNTE5ZmVkMTMyZDJhYTk2ZjAyMjg5YzVhNWZiY2Iz ZDQ5NTUgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcGFnZS9jaHJvbWl1bS9GcmFtZUNocm9taXVtLmNw cAorKysgYi9XZWJDb3JlL3BhZ2UvY2hyb21pdW0vRnJhbWVDaHJvbWl1bS5jcHAKQEAgLTI5LDYg KzI5LDcgQEAKICNpbmNsdWRlICJEb2N1bWVudC5oIgogI2luY2x1ZGUgIkZsb2F0UmVjdC5oIgog I2luY2x1ZGUgIkltYWdlQnVmZmVyLmgiCisjaW5jbHVkZSAiUmVuZGVyTGF5ZXIuaCIKICNpbmNs dWRlICJSZW5kZXJWaWV3LmgiCiAjaW5jbHVkZSAiU2V0dGluZ3MuaCIKIApAQCAtNDAsNyArNDEs NiBAQCBzdHJ1Y3QgU2NvcGVkU3RhdGUgewogICAgIFNjb3BlZFN0YXRlKEZyYW1lKiB0aGVGcmFt ZSwgUmVuZGVyT2JqZWN0KiB0aGVSZW5kZXJlcikKICAgICAgICAgOiBmcmFtZSh0aGVGcmFtZSkK ICAgICAgICAgLCByZW5kZXJlcih0aGVSZW5kZXJlcikKLSAgICAgICAgLCBwYWludEJlaGF2aW9y KGZyYW1lLT52aWV3KCktPnBhaW50QmVoYXZpb3IoKSkKICAgICAgICAgLCBiYWNrZ3JvdW5kQ29s b3IoZnJhbWUtPnZpZXcoKS0+YmFzZUJhY2tncm91bmRDb2xvcigpKQogICAgIHsKICAgICB9CkBA IC00OSwxNCArNDksMTIgQEAgc3RydWN0IFNjb3BlZFN0YXRlIHsKICAgICB7CiAgICAgICAgIGlm IChyZW5kZXJlcikKICAgICAgICAgICAgIHJlbmRlcmVyLT51cGRhdGVEcmFnU3RhdGUoZmFsc2Up OwotICAgICAgICBmcmFtZS0+dmlldygpLT5zZXRQYWludEJlaGF2aW9yKHBhaW50QmVoYXZpb3Ip OwogICAgICAgICBmcmFtZS0+dmlldygpLT5zZXRCYXNlQmFja2dyb3VuZENvbG9yKGJhY2tncm91 bmRDb2xvcik7CiAgICAgICAgIGZyYW1lLT52aWV3KCktPnNldE5vZGVUb0RyYXcoMCk7CiAgICAg fQogCiAgICAgRnJhbWUqIGZyYW1lOwogICAgIFJlbmRlck9iamVjdCogcmVuZGVyZXI7Ci0gICAg UGFpbnRCZWhhdmlvciBwYWludEJlaGF2aW9yOwogICAgIENvbG9yIGJhY2tncm91bmRDb2xvcjsK IH07CiAKQEAgLTY0LDIxICs2MiwyMCBAQCBzdHJ1Y3QgU2NvcGVkU3RhdGUgewogCiBEcmFnSW1h Z2VSZWYgRnJhbWU6Om5vZGVJbWFnZShOb2RlKiBub2RlKQogewotICAgIFJlbmRlck9iamVjdCog cmVuZGVyZXIgPSBub2RlLT5yZW5kZXJlcigpOwotICAgIGlmICghcmVuZGVyZXIpCisgICAgUmVu ZGVyT2JqZWN0KiByZW5kZXJlciA9IGRvY3VtZW50KCktPnJlbmRlcmVyKCk7CisgICAgaWYgKCFy ZW5kZXJlciB8fCAhbm9kZS0+cmVuZGVyZXIoKSkKICAgICAgICAgcmV0dXJuIDA7CiAKICAgICBj b25zdCBTY29wZWRTdGF0ZSBzdGF0ZSh0aGlzLCByZW5kZXJlcik7CiAKICAgICByZW5kZXJlci0+ dXBkYXRlRHJhZ1N0YXRlKHRydWUpOwotICAgIG1fdmlldy0+c2V0UGFpbnRCZWhhdmlvcihzdGF0 ZS5wYWludEJlaGF2aW9yIHwgUGFpbnRCZWhhdmlvckZsYXR0ZW5Db21wb3NpdGluZ0xheWVycyk7 CiAgICAgLy8gV2hlbiBnZW5lcmF0aW5nIHRoZSBkcmFnIGltYWdlIGZvciBhbiBlbGVtZW50LCBp Z25vcmUgdGhlIGRvY3VtZW50IGJhY2tncm91bmQuCiAgICAgbV92aWV3LT5zZXRCYXNlQmFja2dy b3VuZENvbG9yKGNvbG9yV2l0aE92ZXJyaWRlQWxwaGEoQ29sb3I6OndoaXRlLCAxLjApKTsKICAg ICBtX2RvYy0+dXBkYXRlTGF5b3V0KCk7CiAgICAgbV92aWV3LT5zZXROb2RlVG9EcmF3KG5vZGUp OyAvLyBFbmFibGUgc3BlY2lhbCBzdWItdHJlZSBkcmF3aW5nIG1vZGUuCiAKICAgICBJbnRSZWN0 IHRvcExldmVsUmVjdDsKLSAgICBJbnRSZWN0IHBhaW50aW5nUmVjdCA9IHJlbmRlcmVyLT5wYWlu dGluZ1Jvb3RSZWN0KHRvcExldmVsUmVjdCk7CisgICAgSW50UmVjdCBwYWludGluZ1JlY3QgPSBu b2RlLT5yZW5kZXJlcigpLT5wYWludGluZ1Jvb3RSZWN0KHRvcExldmVsUmVjdCk7CiAKICAgICBP d25QdHI8SW1hZ2VCdWZmZXI+IGJ1ZmZlcihJbWFnZUJ1ZmZlcjo6Y3JlYXRlKHBhaW50aW5nUmVj dC5zaXplKCkpKTsKICAgICBpZiAoIWJ1ZmZlcikKQEAgLTg2LDcgKzgzLDkgQEAgRHJhZ0ltYWdl UmVmIEZyYW1lOjpub2RlSW1hZ2UoTm9kZSogbm9kZSkKICAgICBidWZmZXItPmNvbnRleHQoKS0+ dHJhbnNsYXRlKC1wYWludGluZ1JlY3QueCgpLCAtcGFpbnRpbmdSZWN0LnkoKSk7CiAgICAgYnVm ZmVyLT5jb250ZXh0KCktPmNsaXAoRmxvYXRSZWN0KDAsIDAsIHBhaW50aW5nUmVjdC5yaWdodCgp LCBwYWludGluZ1JlY3QuYm90dG9tKCkpKTsKIAotICAgIG1fdmlldy0+cGFpbnQoYnVmZmVyLT5j b250ZXh0KCksIHBhaW50aW5nUmVjdCk7CisgICAgUmVuZGVyTGF5ZXIqIGxheWVyID0gcmVuZGVy ZXItPmVuY2xvc2luZ0xheWVyKCk7CisgICAgbGF5ZXItPnBhaW50KGJ1ZmZlci0+Y29udGV4dCgp LCBwYWludGluZ1JlY3QsCisgICAgICAgICAgICAgICAgIFBhaW50QmVoYXZpb3JOb3JtYWwgfCBQ YWludEJlaGF2aW9yRmxhdHRlbkNvbXBvc2l0aW5nTGF5ZXJzLCByZW5kZXJlcik7CiAKICAgICBS ZWZQdHI8SW1hZ2U+IGltYWdlID0gYnVmZmVyLT5jb3B5SW1hZ2UoKTsKICAgICByZXR1cm4gY3Jl YXRlRHJhZ0ltYWdlRnJvbUltYWdlKGltYWdlLmdldCgpKTsKQEAgLTk0LDExICs5MywxNCBAQCBE cmFnSW1hZ2VSZWYgRnJhbWU6Om5vZGVJbWFnZShOb2RlKiBub2RlKQogCiBEcmFnSW1hZ2VSZWYg RnJhbWU6OmRyYWdJbWFnZUZvclNlbGVjdGlvbigpCiB7CisgICAgUmVuZGVyT2JqZWN0KiByZW5k ZXJlciA9IGRvY3VtZW50KCktPnJlbmRlcmVyKCk7CisgICAgaWYgKCFyZW5kZXJlcikKKyAgICAg ICAgcmV0dXJuIDA7CisKICAgICBpZiAoIXNlbGVjdGlvbigpLT5pc1JhbmdlKCkpCiAgICAgICAg IHJldHVybiAwOwogCiAgICAgY29uc3QgU2NvcGVkU3RhdGUgc3RhdGUodGhpcywgMCk7Ci0gICAg bV92aWV3LT5zZXRQYWludEJlaGF2aW9yKFBhaW50QmVoYXZpb3JTZWxlY3Rpb25Pbmx5KTsKICAg ICBtX2RvYy0+dXBkYXRlTGF5b3V0KCk7CiAKICAgICBJbnRSZWN0IHBhaW50aW5nUmVjdCA9IGVu Y2xvc2luZ0ludFJlY3Qoc2VsZWN0aW9uQm91bmRzKCkpOwpAQCAtMTA5LDcgKzExMSw4IEBAIERy YWdJbWFnZVJlZiBGcmFtZTo6ZHJhZ0ltYWdlRm9yU2VsZWN0aW9uKCkKICAgICBidWZmZXItPmNv bnRleHQoKS0+dHJhbnNsYXRlKC1wYWludGluZ1JlY3QueCgpLCAtcGFpbnRpbmdSZWN0LnkoKSk7 CiAgICAgYnVmZmVyLT5jb250ZXh0KCktPmNsaXAoRmxvYXRSZWN0KDAsIDAsIHBhaW50aW5nUmVj dC5yaWdodCgpLCBwYWludGluZ1JlY3QuYm90dG9tKCkpKTsKIAotICAgIG1fdmlldy0+cGFpbnQo YnVmZmVyLT5jb250ZXh0KCksIHBhaW50aW5nUmVjdCk7CisgICAgUmVuZGVyTGF5ZXIqIGxheWVy ID0gcmVuZGVyZXItPmVuY2xvc2luZ0xheWVyKCk7CisgICAgbGF5ZXItPnBhaW50KGJ1ZmZlci0+ Y29udGV4dCgpLCBwYWludGluZ1JlY3QsIFBhaW50QmVoYXZpb3JTZWxlY3Rpb25Pbmx5LCByZW5k ZXJlcik7CiAKICAgICBSZWZQdHI8SW1hZ2U+IGltYWdlID0gYnVmZmVyLT5jb3B5SW1hZ2UoKTsK ICAgICByZXR1cm4gY3JlYXRlRHJhZ0ltYWdlRnJvbUltYWdlKGltYWdlLmdldCgpKTsK 67716 2010-09-15 14:25:59 -0700 2010-09-16 12:05:39 -0700 Patch bug-45573-20100915142558.patch text/plain 2040 dcheng ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg ZTU5ODI3NzNhYTQxYTA0ZjFiZjRjYzQxNWE2M2UwYjcxMDM5ZmQxMy4uMDg0NjE2ZjA4NGU5Njg4 MTVkMzFhOTM4YjA1MGM0ZDQxZWIyYTdkNiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxOCBAQAorMjAxMC0wOS0xNSAgRGFu aWVsIENoZW5nICA8ZGNoZW5nQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBO T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbY2hyb21pdW1dIEZpeCBpbmNvcnJlY3QgZHJhZyBu b2RlL3NlbGVjdGlvbiBwYWludGluZy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTQ1NTczCisKKyAgICAgICAgRHJhZyBzZWxlY3Rpb25zIHdlcmUgYmVp bmcgaW5jb3JyZWN0bHkgY2xpcHBlZCB3aGVuIGRyYWdnaW5nIGEgc2VsZWN0aW9uCisgICAgICAg IG5lYXIgdGhlIGJvdHRvbSBvZiBhIHBhZ2UgdGhhdCB3YXMgdGFsbGVyIHRoYW4gdGhlIHZpZXdw b3J0LiBUbyBmaXgKKyAgICAgICAgdGhpcywgd2UgdXNlIHBhaW50Q29udGVudHMoKSBpbnN0ZWFk IG9mIHBhaW50KCkuCisKKyAgICAgICAgKiBwYWdlL2Nocm9taXVtL0ZyYW1lQ2hyb21pdW0uY3Bw OgorICAgICAgICAoV2ViQ29yZTo6RnJhbWU6Om5vZGVJbWFnZSk6CisgICAgICAgIChXZWJDb3Jl OjpGcmFtZTo6ZHJhZ0ltYWdlRm9yU2VsZWN0aW9uKToKKwogMjAxMC0wOS0xNCAgUGF2ZWwgRmVs ZG1hbiAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBZdXJ5 IFNlbWlraGF0c2t5LgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9wYWdlL2Nocm9taXVtL0ZyYW1lQ2hy b21pdW0uY3BwIGIvV2ViQ29yZS9wYWdlL2Nocm9taXVtL0ZyYW1lQ2hyb21pdW0uY3BwCmluZGV4 IGI5MDk3ZDA3NTYxYzY5OTBjYzRiODU5OWUxZjE1ZjNjOTRjODcxMDcuLjE1ZmY4YjQ3OGM2M2Jh MTRjNzNmYjFiZWM0NjgzYmIxZWZhZjBmYTAgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcGFnZS9jaHJv bWl1bS9GcmFtZUNocm9taXVtLmNwcAorKysgYi9XZWJDb3JlL3BhZ2UvY2hyb21pdW0vRnJhbWVD aHJvbWl1bS5jcHAKQEAgLTg2LDcgKzg2LDcgQEAgRHJhZ0ltYWdlUmVmIEZyYW1lOjpub2RlSW1h Z2UoTm9kZSogbm9kZSkKICAgICBidWZmZXItPmNvbnRleHQoKS0+dHJhbnNsYXRlKC1wYWludGlu Z1JlY3QueCgpLCAtcGFpbnRpbmdSZWN0LnkoKSk7CiAgICAgYnVmZmVyLT5jb250ZXh0KCktPmNs aXAoRmxvYXRSZWN0KDAsIDAsIHBhaW50aW5nUmVjdC5yaWdodCgpLCBwYWludGluZ1JlY3QuYm90 dG9tKCkpKTsKIAotICAgIG1fdmlldy0+cGFpbnQoYnVmZmVyLT5jb250ZXh0KCksIHBhaW50aW5n UmVjdCk7CisgICAgbV92aWV3LT5wYWludENvbnRlbnRzKGJ1ZmZlci0+Y29udGV4dCgpLCBwYWlu dGluZ1JlY3QpOwogCiAgICAgUmVmUHRyPEltYWdlPiBpbWFnZSA9IGJ1ZmZlci0+Y29weUltYWdl KCk7CiAgICAgcmV0dXJuIGNyZWF0ZURyYWdJbWFnZUZyb21JbWFnZShpbWFnZS5nZXQoKSk7CkBA IC0xMDksNyArMTA5LDcgQEAgRHJhZ0ltYWdlUmVmIEZyYW1lOjpkcmFnSW1hZ2VGb3JTZWxlY3Rp b24oKQogICAgIGJ1ZmZlci0+Y29udGV4dCgpLT50cmFuc2xhdGUoLXBhaW50aW5nUmVjdC54KCks IC1wYWludGluZ1JlY3QueSgpKTsKICAgICBidWZmZXItPmNvbnRleHQoKS0+Y2xpcChGbG9hdFJl Y3QoMCwgMCwgcGFpbnRpbmdSZWN0LnJpZ2h0KCksIHBhaW50aW5nUmVjdC5ib3R0b20oKSkpOwog Ci0gICAgbV92aWV3LT5wYWludChidWZmZXItPmNvbnRleHQoKSwgcGFpbnRpbmdSZWN0KTsKKyAg ICBtX3ZpZXctPnBhaW50Q29udGVudHMoYnVmZmVyLT5jb250ZXh0KCksIHBhaW50aW5nUmVjdCk7 CiAKICAgICBSZWZQdHI8SW1hZ2U+IGltYWdlID0gYnVmZmVyLT5jb3B5SW1hZ2UoKTsKICAgICBy ZXR1cm4gY3JlYXRlRHJhZ0ltYWdlRnJvbUltYWdlKGltYWdlLmdldCgpKTsK