Thanks for the patch Deepak. Just a few very minor comments. If it's OK with you, I'll make the changes and submit on your behalf.
======================================================================== http://mondrian.corp.google.com/file/10347104///depot/googleclient/gears/opensource/gears/ui/common/html_dialog.js?a=1 File //depot/googleclient/gears/opensource/gears/ui/common/html_dialog.js (snapshot 1) ------------------------------------ Line 33: Trailing whitespace ------------------------------------ Line 34: if(browser.wince && browser.opera) { if (browser ... ------------------------------------ Line 54: buttonRowElem.style.display = 'block'; Tab ======================================================================== http://mondrian.corp.google.com/file/10347104///depot/googleclient/gears/opensource/gears/ui/common/settings_dialog.html_m4?a=1 File //depot/googleclient/gears/opensource/gears/ui/common/settings_dialog.html_m4 (snapshot 1) ------------------------------------ Line 212: m4_ifelse(PRODUCT_BROWSER,~IEMOBILE~,m4_dnl m4_ifelse(PRODUCT_OS,~wince~,m4_dnl ======================================================================== -- To respond, reply to this email or visit http://mondrian.corp.google.com/10347104
