Incorporating feedback from C++ readability review.

Incorporating feedback from C++ readability review.

This changelist includes some minor fixes suggested during a C++ readability review. There are no code logic changes -- all edits are cosmetic.

Review URL on Rietveld code review tool: http://codereview.appspot.com/1737041/show
Buganizer URL for readability review: http://b/issue?id=2786642

BUG=none
TEST=none

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

[ncarter (slow), 2010-07-02 23:35:35]

http://codereview.chromium.org/2838042/diff/1/3
File chrome/test/live_sync/live_sync_test.h (right):

http://codereview.chromium.org/2838042/diff/1/3#newcode19
chrome/test/live_sync/live_sync_test.h:19: class Profile; // Declared in
"chrome/browser/profile.h".
Nowhere in Chrome are such "Declared in" comments in use; this is inconsistent
with Chrome style.  If this was a readability reviewer suggestion, please point
them at searches of the Chrome codebase to show that this isn't local
convention.

[Raghu Simha, 2010-07-02 23:41:39]

http://codereview.chromium.org/2838042/diff/1/3
File chrome/test/live_sync/live_sync_test.h (right):

http://codereview.chromium.org/2838042/diff/1/3#newcode19
chrome/test/live_sync/live_sync_test.h:19: class Profile; // Declared in
"chrome/browser/profile.h".
On 2010/07/02 23:35:35, ncarter wrote:
> Nowhere in Chrome are such "Declared in" comments in use; this is inconsistent
> with Chrome style.  If this was a readability reviewer suggestion, please
point
> them at searches of the Chrome codebase to show that this isn't local
> convention.

Done.

Alphabetized the forward declarations while I was at it.

[ncarter (slow), 2010-07-03 01:45:40]

LGTM