diff options
author | Shawn O. Pearce <sop@google.com> | 2009-03-03 16:49:50 -0800 |
---|---|---|
committer | Shawn O. Pearce <sop@google.com> | 2009-03-03 16:49:50 -0800 |
commit | ef1407814acecd62aa336893e5011f843e79699b (patch) | |
tree | e3c9525afbb5b358c0511933cff76666bbc357a9 | |
parent | 6c3e7e71315410a754f72e7e197a04acde47140a (diff) |
Generate the id for the iframe used during OpenID login
We don't really care what the iframe is called, just that it must
have a unique name within the entire document. Letting GWT create
the name for us simplifies that long-term, as there is no longer a
magic string in the code.
Signed-off-by: Shawn O. Pearce <sop@google.com>
-rw-r--r-- | src/main/java/com/google/gerrit/client/openid/OpenIdLoginPanel.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/gerrit/client/openid/OpenIdLoginPanel.java b/src/main/java/com/google/gerrit/client/openid/OpenIdLoginPanel.java index b5c4d54db7..5ec93a61c7 100644 --- a/src/main/java/com/google/gerrit/client/openid/OpenIdLoginPanel.java +++ b/src/main/java/com/google/gerrit/client/openid/OpenIdLoginPanel.java @@ -88,7 +88,7 @@ public class OpenIdLoginPanel extends Composite implements FormHandler { formBody.add(discoveryCbField = new Hidden(OpenIdUtil.P_DISCOVERY_CB)); formBody.add(new Hidden(OpenIdUtil.P_SIGNIN_MODE, mode.name())); - providerFrame = new NamedFrame("openid_provider"); + providerFrame = new NamedFrame(DOM.createUniqueId()); providerFrame.setVisible(false); form = new FormPanel(providerFrame); |