Stylistic fixes for BrowsingDataFileSystemHelper

Stylistic fixes for BrowsingDataFileSystemHelper.

The only functional change is to drop the `profile_` property from the
CannedBrowsingDataFileSystemHelper, which just stored it without ever
touching it.  I don't like keeping pointers around if I don't need them, so
I'm dropping it here.

BUG=None
TEST=Read the code. Can you understand it?


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

[Mike West, 2011-06-01 15:33:53]

This is incredibly low priority, so feel free to get back to me on it in a week
or three.  Really, no rush.

BrowsingDataFileSystemHelper seems like a big enough chunk of self-sufficient
code to ask for a readability review.  You three have been through the review
process (and I ask you questions every few minutes anyway), so if you've got
some free time, I'd appreciate you giving these cleanups a once-over so I can
submit something to the reviewers with a reasonable expectation of success.

Thanks (and again, low prio)!

-Mike

[jochen (gone - plz use gerrit), 2011-06-01 19:55:41]

LGTM

[Mike West, 2011-06-09 17:06:00]

I lied.  There's a functional change: I can drop the `profile_` property from
the CannedBrowsingDataFileSystemHelper, and fewer raw pointers are better.

This, of course, de-LGTMs things.  Whenever you have a few free minutes, take
another look, please. :)

[jochen (gone - plz use gerrit), 2011-06-09 17:16:13]

still LGTM

[commit-bot: I haz the power, 2011-06-10 07:55:58]

Change committed as 88647