58070 2011-04-07 12:40:26 -0700 HTMLVideoElement::webkitEnterFullscreen does not use new Full Screen API when available. 2011-04-07 15:38:51 -0700 1 1 1 Unclassified WebKit Media 528+ (Nightly build) Other OS X 10.5 RESOLVED FIXED InRadar P2 Normal --- 1 jer.noble jer.noble abarth dglazkov eric webkit.review.bot oldest_to_newest 381582 0 jer.noble 2011-04-07 12:40:26 -0700 HTMLVideoElement::webkitEnterFullscreen does not use new Full Screen API when available. 381583 1 jer.noble 2011-04-07 12:41:09 -0700 <rdar://problem/9251239> 381587 2 88674 jer.noble 2011-04-07 12:45:20 -0700 Created attachment 88674 Patch 381592 3 webkit.review.bot 2011-04-07 12:53:12 -0700 Attachment 88674 did not build on chromium: Build output: http://queues.webkit.org/results/8346689 381599 4 88674 jer.noble 2011-04-07 13:01:17 -0700 Comment on attachment 88674 Patch Patch seems to be missing bits and pieces. Will re-upload. 381609 5 88676 jer.noble 2011-04-07 13:12:26 -0700 Created attachment 88676 Patch 381611 6 88676 dglazkov 2011-04-07 13:14:14 -0700 Comment on attachment 88676 Patch Why are these wired differently from the MediaControlFullscreenButton? Shouldn't we use the same codepaths? 381618 7 jer.noble 2011-04-07 13:18:28 -0700 (In reply to comment #6) > (From update of attachment 88676 [details]) > Why are these wired differently from the MediaControlFullscreenButton? Shouldn't we use the same code paths? Good point. I should check to make sure the settings are enabled first. Thanks! 381628 8 dglazkov 2011-04-07 13:26:07 -0700 (In reply to comment #7) > (In reply to comment #6) > > (From update of attachment 88676 [details] [details]) > > Why are these wired differently from the MediaControlFullscreenButton? Shouldn't we use the same code paths? > > Good point. I should check to make sure the settings are enabled first. Thanks! No worries -- I was just looking at this code :). Sounds like ideally, the code from here: http://codesearch.google.com/codesearch/p#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/rendering/MediaControlElements.cpp&l=950&exact_package=chromium should just migrate to enterFullScreen. 381629 9 88681 jer.noble 2011-04-07 13:26:30 -0700 Created attachment 88681 Patch 381632 10 88681 dglazkov 2011-04-07 13:28:47 -0700 Comment on attachment 88681 Patch Can we also eliminate the same code here: http://codesearch.google.com/codesearch/p#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/rendering/MediaControlElements.cpp&l=950&exact_package=chromium 381640 11 jer.noble 2011-04-07 13:35:42 -0700 Indeed, that would be ideal. 381683 12 88681 eric.carlson 2011-04-07 14:04:36 -0700 Comment on attachment 88681 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=88681&action=review > Source/WebCore/html/HTMLMediaElement.cpp:2464 > +#if ENABLE(FULLSCREEN_API) > + if (document()->settings() && document()->settings()->fullScreenEnabled()) { > + webkitRequestFullScreen(0); > + return; > + } > +#else > ASSERT(!m_isFullscreen); > m_isFullscreen = true; > if (document() && document()->page()) { > + > document()->page()->chrome()->client()->enterFullscreenForNode(this); > scheduleEvent(eventNames().webkitbeginfullscreenEvent); > } > +#endif There isn't any need for the #else ... #endif, we want to continue to support the old API even when ENABLE_FULLSCREEN_API is defined. > Source/WebCore/html/HTMLMediaElement.cpp:2475 > +#if ENABLE(FULLSCREEN_API) > + if (document()->settings() && document()->settings()->fullScreenEnabled()) { > + document()->webkitCancelFullScreen(); > + return; > + } > +#else Same here. 381690 13 jer.noble 2011-04-07 14:12:26 -0700 Committed r83208: <http://trac.webkit.org/changeset/83208> 381728 14 dglazkov 2011-04-07 14:44:43 -0700 (In reply to comment #13) > Committed r83208: <http://trac.webkit.org/changeset/83208> Whoops. Don't you do this twice now, since MediaControlFullscreenButtonElement::defaultEventHandler calls enterFullScreen? 381754 15 jer.noble 2011-04-07 15:01:06 -0700 (In reply to comment #14) > (In reply to comment #13) > > Committed r83208: <http://trac.webkit.org/changeset/83208> > > Whoops. Don't you do this twice now, since MediaControlFullscreenButtonElement::defaultEventHandler calls enterFullScreen? I filed bug #58087 to get rid of the duplicated code. 381792 16 webkit.review.bot 2011-04-07 15:38:51 -0700 http://trac.webkit.org/changeset/83208 might have broken SnowLeopard Intel Release (Tests), GTK Linux 32-bit Release, and GTK Linux 64-bit Debug The following tests are not passing: media/context-menu-actions.html media/media-fullscreen-inline.html media/media-fullscreen-not-in-document.html 88674 2011-04-07 12:45:20 -0700 2011-04-07 13:01:17 -0700 Patch bug-58070-20110407124519.patch text/plain 2206 jer.noble U3VidmVyc2lvbiBSZXZpc2lvbjogODMxOTUKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA1OTdiM2U5ZTUxYmMyYWFk OWU5ZGNmZThjZjE0NjI5MzcwYzhjMmVlLi40YjQxODA2YTk5MGQ4OTAyY2Y5ZTRmOTE5OWZkYjM5 ZTZjYTlhNDdiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291 cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDQtMDcgIEplciBO b2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgSFRNTFZpZGVvRWxlbWVudDo6d2Via2l0RW50ZXJGdWxsc2Ny ZWVuIGRvZXMgbm90IHVzZSBuZXcgRnVsbCBTY3JlZW4gQVBJIHdoZW4gYXZhaWxhYmxlLgorICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTgwNzAKKworICAg ICAgICBNYWtlIHRoZSBIVE1MTWVkaWFFbGVtZW50IGZ1bGwgc2NyZWVuIGZ1bmN0aW9ucyBjYWxs IGludG8gdGhlIG5ldyBGdWxsIFNjcmVlbiBBUEkgd2hlbgorICAgICAgICBGVUxMU0NSRUVOX0FQ SSBpcyBlbmFibGVkLgorCisgICAgICAgICogaHRtbC9IVE1MTWVkaWFFbGVtZW50LmNwcDoKKyAg ICAgICAgKFdlYkNvcmU6OkhUTUxNZWRpYUVsZW1lbnQ6OmVudGVyRnVsbHNjcmVlbik6CisgICAg ICAgIChXZWJDb3JlOjpIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVlbik6CisKIDIwMTEt MDQtMDYgIEplciBOb2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3 ZWQgYnkgTWFjaWVqIFN0YWNob3dpYWsuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9odG1s L0hUTUxNZWRpYUVsZW1lbnQuY3BwIGIvU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MTWVkaWFFbGVt ZW50LmNwcAppbmRleCA2NDI3ZTQ3ODRiMjkxZmIxYjQyNDJkMmUzOGFjZjQwYzdiZTI1NDI4Li4x ZmY2MmViNDYwNWVkNjE5NjQzOWJjMjhkMTZhNjU1NjQ1ZmUyZWE5IDEwMDY0NAotLS0gYS9Tb3Vy Y2UvV2ViQ29yZS9odG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3Jl L2h0bWwvSFRNTE1lZGlhRWxlbWVudC5jcHAKQEAgLTI0NDgsMTkgKzI0NDgsMjUgQEAgYm9vbCBI VE1MTWVkaWFFbGVtZW50Ojppc0Z1bGxzY3JlZW4oKSBjb25zdAogdm9pZCBIVE1MTWVkaWFFbGVt ZW50OjplbnRlckZ1bGxzY3JlZW4oKQogewogICAgIExPRyhNZWRpYSwgIkhUTUxNZWRpYUVsZW1l bnQ6OmVudGVyRnVsbHNjcmVlbiIpOwotCisjaWYgRU5BQkxFKEZVTExTQ1JFRU5fQVBJKQorICAg IHdlYmtpdFJlcXVlc3RGdWxsU2NyZWVuKDApOworI2Vsc2UKICAgICBBU1NFUlQoIW1faXNGdWxs c2NyZWVuKTsKICAgICBtX2lzRnVsbHNjcmVlbiA9IHRydWU7CiAgICAgaWYgKGRvY3VtZW50KCkg JiYgZG9jdW1lbnQoKS0+cGFnZSgpKSB7CisKICAgICAgICAgZG9jdW1lbnQoKS0+cGFnZSgpLT5j aHJvbWUoKS0+Y2xpZW50KCktPmVudGVyRnVsbHNjcmVlbkZvck5vZGUodGhpcyk7CiAgICAgICAg IHNjaGVkdWxlRXZlbnQoZXZlbnROYW1lcygpLndlYmtpdGJlZ2luZnVsbHNjcmVlbkV2ZW50KTsK ICAgICB9CisjZW5kaWYKIH0KIAogdm9pZCBIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVl bigpCiB7CiAgICAgTE9HKE1lZGlhLCAiSFRNTE1lZGlhRWxlbWVudDo6ZXhpdEZ1bGxzY3JlZW4i KTsKLQorI2lmIEVOQUJMRShGVUxMU0NSRUVOX0FQSSkKKyAgICBkb2N1bWVudCgpLT4KKyNlbHNl CiAgICAgQVNTRVJUKG1faXNGdWxsc2NyZWVuKTsKICAgICBtX2lzRnVsbHNjcmVlbiA9IGZhbHNl OwogICAgIGlmIChkb2N1bWVudCgpICYmIGRvY3VtZW50KCktPnBhZ2UoKSkgewpAQCAtMjQ2OSw2 ICsyNDc1LDcgQEAgdm9pZCBIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVlbigpCiAgICAg ICAgIGRvY3VtZW50KCktPnBhZ2UoKS0+Y2hyb21lKCktPmNsaWVudCgpLT5leGl0RnVsbHNjcmVl bkZvck5vZGUodGhpcyk7CiAgICAgICAgIHNjaGVkdWxlRXZlbnQoZXZlbnROYW1lcygpLndlYmtp dGVuZGZ1bGxzY3JlZW5FdmVudCk7CiAgICAgfQorI2VuZGlmCiB9CiAKIFBsYXRmb3JtTWVkaWEg SFRNTE1lZGlhRWxlbWVudDo6cGxhdGZvcm1NZWRpYSgpIGNvbnN0Cg== 88676 2011-04-07 13:12:26 -0700 2011-04-07 13:26:27 -0700 Patch bug-58070-20110407131225.patch text/plain 2231 jer.noble U3VidmVyc2lvbiBSZXZpc2lvbjogODMxOTkKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA1OTdiM2U5ZTUxYmMyYWFk OWU5ZGNmZThjZjE0NjI5MzcwYzhjMmVlLi40YjQxODA2YTk5MGQ4OTAyY2Y5ZTRmOTE5OWZkYjM5 ZTZjYTlhNDdiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291 cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDQtMDcgIEplciBO b2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgSFRNTFZpZGVvRWxlbWVudDo6d2Via2l0RW50ZXJGdWxsc2Ny ZWVuIGRvZXMgbm90IHVzZSBuZXcgRnVsbCBTY3JlZW4gQVBJIHdoZW4gYXZhaWxhYmxlLgorICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTgwNzAKKworICAg ICAgICBNYWtlIHRoZSBIVE1MTWVkaWFFbGVtZW50IGZ1bGwgc2NyZWVuIGZ1bmN0aW9ucyBjYWxs IGludG8gdGhlIG5ldyBGdWxsIFNjcmVlbiBBUEkgd2hlbgorICAgICAgICBGVUxMU0NSRUVOX0FQ SSBpcyBlbmFibGVkLgorCisgICAgICAgICogaHRtbC9IVE1MTWVkaWFFbGVtZW50LmNwcDoKKyAg ICAgICAgKFdlYkNvcmU6OkhUTUxNZWRpYUVsZW1lbnQ6OmVudGVyRnVsbHNjcmVlbik6CisgICAg ICAgIChXZWJDb3JlOjpIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVlbik6CisKIDIwMTEt MDQtMDYgIEplciBOb2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3 ZWQgYnkgTWFjaWVqIFN0YWNob3dpYWsuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9odG1s L0hUTUxNZWRpYUVsZW1lbnQuY3BwIGIvU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MTWVkaWFFbGVt ZW50LmNwcAppbmRleCA2NDI3ZTQ3ODRiMjkxZmIxYjQyNDJkMmUzOGFjZjQwYzdiZTI1NDI4Li5l MWQ1ZjA0OWVhN2MzOGJiOTZkNjcwOWZmMGZkYWYyYjc0Nzg1MTE5IDEwMDY0NAotLS0gYS9Tb3Vy Y2UvV2ViQ29yZS9odG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3Jl L2h0bWwvSFRNTE1lZGlhRWxlbWVudC5jcHAKQEAgLTI0NDgsMTkgKzI0NDgsMjUgQEAgYm9vbCBI VE1MTWVkaWFFbGVtZW50Ojppc0Z1bGxzY3JlZW4oKSBjb25zdAogdm9pZCBIVE1MTWVkaWFFbGVt ZW50OjplbnRlckZ1bGxzY3JlZW4oKQogewogICAgIExPRyhNZWRpYSwgIkhUTUxNZWRpYUVsZW1l bnQ6OmVudGVyRnVsbHNjcmVlbiIpOwotCisjaWYgRU5BQkxFKEZVTExTQ1JFRU5fQVBJKQorICAg IHdlYmtpdFJlcXVlc3RGdWxsU2NyZWVuKDApOworI2Vsc2UKICAgICBBU1NFUlQoIW1faXNGdWxs c2NyZWVuKTsKICAgICBtX2lzRnVsbHNjcmVlbiA9IHRydWU7CiAgICAgaWYgKGRvY3VtZW50KCkg JiYgZG9jdW1lbnQoKS0+cGFnZSgpKSB7CisKICAgICAgICAgZG9jdW1lbnQoKS0+cGFnZSgpLT5j aHJvbWUoKS0+Y2xpZW50KCktPmVudGVyRnVsbHNjcmVlbkZvck5vZGUodGhpcyk7CiAgICAgICAg IHNjaGVkdWxlRXZlbnQoZXZlbnROYW1lcygpLndlYmtpdGJlZ2luZnVsbHNjcmVlbkV2ZW50KTsK ICAgICB9CisjZW5kaWYKIH0KIAogdm9pZCBIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVl bigpCiB7CiAgICAgTE9HKE1lZGlhLCAiSFRNTE1lZGlhRWxlbWVudDo6ZXhpdEZ1bGxzY3JlZW4i KTsKLQorI2lmIEVOQUJMRShGVUxMU0NSRUVOX0FQSSkKKyAgICBkb2N1bWVudCgpLT53ZWJraXRD YW5jZWxGdWxsU2NyZWVuKCk7CisjZWxzZQogICAgIEFTU0VSVChtX2lzRnVsbHNjcmVlbik7CiAg ICAgbV9pc0Z1bGxzY3JlZW4gPSBmYWxzZTsKICAgICBpZiAoZG9jdW1lbnQoKSAmJiBkb2N1bWVu dCgpLT5wYWdlKCkpIHsKQEAgLTI0NjksNiArMjQ3NSw3IEBAIHZvaWQgSFRNTE1lZGlhRWxlbWVu dDo6ZXhpdEZ1bGxzY3JlZW4oKQogICAgICAgICBkb2N1bWVudCgpLT5wYWdlKCktPmNocm9tZSgp LT5jbGllbnQoKS0+ZXhpdEZ1bGxzY3JlZW5Gb3JOb2RlKHRoaXMpOwogICAgICAgICBzY2hlZHVs ZUV2ZW50KGV2ZW50TmFtZXMoKS53ZWJraXRlbmRmdWxsc2NyZWVuRXZlbnQpOwogICAgIH0KKyNl bmRpZgogfQogCiBQbGF0Zm9ybU1lZGlhIEhUTUxNZWRpYUVsZW1lbnQ6OnBsYXRmb3JtTWVkaWEo KSBjb25zdAo= 88681 2011-04-07 13:26:30 -0700 2011-04-07 14:04:36 -0700 Patch bug-58070-20110407132629.patch text/plain 2451 jer.noble U3VidmVyc2lvbiBSZXZpc2lvbjogODMxOTkKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA1OTdiM2U5ZTUxYmMyYWFk OWU5ZGNmZThjZjE0NjI5MzcwYzhjMmVlLi40YjQxODA2YTk5MGQ4OTAyY2Y5ZTRmOTE5OWZkYjM5 ZTZjYTlhNDdiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291 cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDQtMDcgIEplciBO b2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgSFRNTFZpZGVvRWxlbWVudDo6d2Via2l0RW50ZXJGdWxsc2Ny ZWVuIGRvZXMgbm90IHVzZSBuZXcgRnVsbCBTY3JlZW4gQVBJIHdoZW4gYXZhaWxhYmxlLgorICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTgwNzAKKworICAg ICAgICBNYWtlIHRoZSBIVE1MTWVkaWFFbGVtZW50IGZ1bGwgc2NyZWVuIGZ1bmN0aW9ucyBjYWxs IGludG8gdGhlIG5ldyBGdWxsIFNjcmVlbiBBUEkgd2hlbgorICAgICAgICBGVUxMU0NSRUVOX0FQ SSBpcyBlbmFibGVkLgorCisgICAgICAgICogaHRtbC9IVE1MTWVkaWFFbGVtZW50LmNwcDoKKyAg ICAgICAgKFdlYkNvcmU6OkhUTUxNZWRpYUVsZW1lbnQ6OmVudGVyRnVsbHNjcmVlbik6CisgICAg ICAgIChXZWJDb3JlOjpIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVlbik6CisKIDIwMTEt MDQtMDYgIEplciBOb2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3 ZWQgYnkgTWFjaWVqIFN0YWNob3dpYWsuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9odG1s L0hUTUxNZWRpYUVsZW1lbnQuY3BwIGIvU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MTWVkaWFFbGVt ZW50LmNwcAppbmRleCA2NDI3ZTQ3ODRiMjkxZmIxYjQyNDJkMmUzOGFjZjQwYzdiZTI1NDI4Li42 NzMzODE5ZTI0Y2ZhODU1MDU2NDcwYTBkM2NiYWI1YmFmM2YyYWI3IDEwMDY0NAotLS0gYS9Tb3Vy Y2UvV2ViQ29yZS9odG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3Jl L2h0bWwvSFRNTE1lZGlhRWxlbWVudC5jcHAKQEAgLTI0NDgsMTkgKzI0NDgsMzEgQEAgYm9vbCBI VE1MTWVkaWFFbGVtZW50Ojppc0Z1bGxzY3JlZW4oKSBjb25zdAogdm9pZCBIVE1MTWVkaWFFbGVt ZW50OjplbnRlckZ1bGxzY3JlZW4oKQogewogICAgIExPRyhNZWRpYSwgIkhUTUxNZWRpYUVsZW1l bnQ6OmVudGVyRnVsbHNjcmVlbiIpOwotCisjaWYgRU5BQkxFKEZVTExTQ1JFRU5fQVBJKQorICAg IGlmIChkb2N1bWVudCgpLT5zZXR0aW5ncygpICYmIGRvY3VtZW50KCktPnNldHRpbmdzKCktPmZ1 bGxTY3JlZW5FbmFibGVkKCkpIHsKKyAgICAgICAgd2Via2l0UmVxdWVzdEZ1bGxTY3JlZW4oMCk7 CisgICAgICAgIHJldHVybjsKKyAgICB9CisjZWxzZQogICAgIEFTU0VSVCghbV9pc0Z1bGxzY3Jl ZW4pOwogICAgIG1faXNGdWxsc2NyZWVuID0gdHJ1ZTsKICAgICBpZiAoZG9jdW1lbnQoKSAmJiBk b2N1bWVudCgpLT5wYWdlKCkpIHsKKwogICAgICAgICBkb2N1bWVudCgpLT5wYWdlKCktPmNocm9t ZSgpLT5jbGllbnQoKS0+ZW50ZXJGdWxsc2NyZWVuRm9yTm9kZSh0aGlzKTsKICAgICAgICAgc2No ZWR1bGVFdmVudChldmVudE5hbWVzKCkud2Via2l0YmVnaW5mdWxsc2NyZWVuRXZlbnQpOwogICAg IH0KKyNlbmRpZgogfQogCiB2b2lkIEhUTUxNZWRpYUVsZW1lbnQ6OmV4aXRGdWxsc2NyZWVuKCkK IHsKICAgICBMT0coTWVkaWEsICJIVE1MTWVkaWFFbGVtZW50OjpleGl0RnVsbHNjcmVlbiIpOwot CisjaWYgRU5BQkxFKEZVTExTQ1JFRU5fQVBJKQorICAgIGlmIChkb2N1bWVudCgpLT5zZXR0aW5n cygpICYmIGRvY3VtZW50KCktPnNldHRpbmdzKCktPmZ1bGxTY3JlZW5FbmFibGVkKCkpIHsKKyAg ICAgICAgZG9jdW1lbnQoKS0+d2Via2l0Q2FuY2VsRnVsbFNjcmVlbigpOworICAgICAgICByZXR1 cm47CisgICAgfQorI2Vsc2UKICAgICBBU1NFUlQobV9pc0Z1bGxzY3JlZW4pOwogICAgIG1faXNG dWxsc2NyZWVuID0gZmFsc2U7CiAgICAgaWYgKGRvY3VtZW50KCkgJiYgZG9jdW1lbnQoKS0+cGFn ZSgpKSB7CkBAIC0yNDY5LDYgKzI0ODEsNyBAQCB2b2lkIEhUTUxNZWRpYUVsZW1lbnQ6OmV4aXRG dWxsc2NyZWVuKCkKICAgICAgICAgZG9jdW1lbnQoKS0+cGFnZSgpLT5jaHJvbWUoKS0+Y2xpZW50 KCktPmV4aXRGdWxsc2NyZWVuRm9yTm9kZSh0aGlzKTsKICAgICAgICAgc2NoZWR1bGVFdmVudChl dmVudE5hbWVzKCkud2Via2l0ZW5kZnVsbHNjcmVlbkV2ZW50KTsKICAgICB9CisjZW5kaWYKIH0K IAogUGxhdGZvcm1NZWRpYSBIVE1MTWVkaWFFbGVtZW50OjpwbGF0Zm9ybU1lZGlhKCkgY29uc3QK