Registration + Password Forgotten forms as IFRAME

First off, thanks very much for this solution. It will be implemented by our team in the coming weeks.

I would like support to implement the following feature for this module:
Currently, when the user wants to register on a slave website, the user is redirected to the master website (http://master.domain/user/register?external_dest=http...). When the user has registered, they are redirected back to the slave website.
Allow for an Admin API setting such that the Registration Form from the master site is loaded in an IFRAME on the slaves' "user/register" website.

To implement, I can visualize the following module changes:

• services_sso_server_helper.module::hook_menu(). Add 2 menu callbacks to load the registration form and the password forgotten form (bypassing all page HTML/theming from the master site). Destination URL is a new menu item (see point 2).
• services_sso_server_helper.module::hook_menu(). Add a new page with a DRUPAL behaviors javascript function call that will refresh the parent (non-IFRAME) window.
• services_sso_client/includes/services_sso_client.admin.inc::services_sso_client_user_admin_form(). Add an option variable. The admin may choose between an IFRAME solution or the redirect method.
• services_sso_client.module::hook_form_FORM_ID_alter(). Edit form array for "user_register" and "user_password" forms. Override the theme functions for form (if the setting in the API is enabled). Use themes "theme_services_sso_client_user_register" and "theme_services_sso_client_user_password" functions instead of "theme_user_register".and "theme_user_password" (default form definitions).
• services_sso_client.module::hook_theme(). Add definition for theme functions (see previous point).
• services_sso_client.module::theme_services_sso_client_user_password()
  

        theme_services_sso_client_user_password(&$variables){
          return '<iframe src="' . $master_site . '/user/password">[FORM HTML]</iframe>';
        }
      

• services_sso_client.module::theme_services_sso_client_user_register(). See previous point.
• services_sso_client.module::services_sso_client_menu_alter_callback(). If the IFRAME solution has been chosen: return immediately.

Could a maintainer give their opinion. Is there any forseeable problems with this solution?