Remove old web app code it's no longer needed. Simplifies startup a little.

Remove old web app code it's no longer needed. Simplifies startup a little.

Also fixes a regression from the BrowserInit porting work. I was a little over-zealous removing the CommandLine handling. It turns out that it's this that MessageWindow uses to convey the command line of the signalling process to the existing process, and that the existing process re-runs BrowserInit with the signalling process' command line. So I plumbed that back through again.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=9034

[brettw, 2009-02-02 06:26:46]

LGTM

http://codereview.chromium.org/19746/diff/306/58
File chrome/browser/browser.cc (right):

http://codereview.chromium.org/19746/diff/306/58#newcode313
Line 313: const std::wstring& app_name = ComputeApplicationNameFromURL(url);
While not wrong, I think this is better as a std::wstring rather than a const
ref. Since the function returns a real string, you're not saving anything.

http://codereview.chromium.org/19746/diff/306/58#newcode1810
Line 1810: const std::wstring& app_name = ComputeApplicationNameFromURL(url);
Same string comment.