> ========================================================================
> http://mondrian.corp.google.com/file/9879962///depot/googleclient/gears/opensource/gears/ui/common/permissions_dialog.html_m4?a=3
> File 
> //depot/googleclient/gears/opensource/gears/ui/common/permissions_dialog.html_m4
>  (snapshot 3)
> ------------------------------------
> Line 424: // Do not resize on Opera Mobile, let the browser resize 
> automatically
> Most of the time you just check for browser.opera, not browser.wince &&
> browser.opera. Are you sure those other checks are all correct (e.g. the
> wrapDomain() ones). Just checking.
Good point. I messed this up merging Deepak's changes to tip of tree.
Have fixed the checks on wrapDomain().

> ========================================================================
> http://mondrian.corp.google.com/file/9879962///depot/googleclient/gears/opensource/gears/ui/common/shortcuts_dialog.html_m4?a=3
> File 
> //depot/googleclient/gears/opensource/gears/ui/common/shortcuts_dialog.html_m4
>  (snapshot 3)
> ------------------------------------
> Line 34: <title></title>
> Empty title? What for?
We set the title from JS. On Opera, you can't set document.title
unless the element is created in HTML.

New snapshot uploaded.

Reply via email to