LGTM ======================================================================== http://mondrian.corp.google.com/file/9464817///depot/googleclient/gears/opensource/gears/base/opera/opera_utils.cc?a=3 File //depot/googleclient/gears/opensource/gears/base/opera/opera_utils.cc (snapshot 3) ------------------------------------ Line 134: HWND opera_window = ::FindWindow(L"Opera_MainWndClass", NULL); On 2:10 pm, 18 Dec, steveblock wrote: > Is it true that we can't search by window title because this is set to the page > title? Also, we should consider adding a 'search by class name' option to > ProcessRestarter. Currently we search only by process name. See > installer/opera/setup.cc and installer/common/process_restarter.h.
That's a good point. You could add an issue about this so we don't forget about it (well Baran won't forget, anyway :)). ======================================================================== -- To respond, reply to this email or visit http://mondrian.corp.google.com/9464817
