RESOLVED FIXED 87453
configure.ac has duplicated AC_MSG_RESULT([$enable_sandbox]) and nothing for seamless iframes
https://bugs.webkit.org/show_bug.cgi?id=87453
Summary configure.ac has duplicated AC_MSG_RESULT([$enable_sandbox]) and nothing for ...
Tim Horton
Reported 2012-05-24 20:13:37 PDT
# check whether to enable HTML5 sandbox iframe support AC_MSG_CHECKING([whether to enable HTML5 sandboxed iframe support]) AC_ARG_ENABLE(sandbox, AC_HELP_STRING([--enable-sandbox], [enable HTML5 sandboxed iframe support [default=yes]]), [],[enable_sandbox="yes"]) AC_MSG_RESULT([$enable_sandbox]) # check whether to enable HTML5 iframe seamless attribute support AC_MSG_CHECKING([whether to enable HTML5 iframe seamless attribute support]) AC_ARG_ENABLE(iframe-seamless, AC_HELP_STRING([--enable-iframe-seamless], [enable HTML5 iframe seamless attribute support [default=yes]]), [],[enable_iframe_seamless="yes"]) AC_MSG_RESULT([$enable_sandbox]) Note the two AC_MSG_RESULTs are the same; I think this is the result of a failed copy and paste.
Attachments
Patch (1.29 KB, patch)
2012-05-24 21:52 PDT, Zan Dobersek
no flags
Zan Dobersek
Comment 1 2012-05-24 21:51:37 PDT
Thanks for spotting this, a stupid mistake made on my side.
Zan Dobersek
Comment 2 2012-05-24 21:52:41 PDT
Eric Seidel (no email)
Comment 3 2012-05-25 12:31:56 PDT
Comment on attachment 143968 [details] Patch OK.
WebKit Review Bot
Comment 4 2012-05-25 12:40:22 PDT
Comment on attachment 143968 [details] Patch Clearing flags on attachment: 143968 Committed r118550: <http://trac.webkit.org/changeset/118550>
WebKit Review Bot
Comment 5 2012-05-25 12:40:26 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.