Pass gfx structs by const ref (gfx::Vector2dF)
Avoid unneccessary copy of structures gfx::Vector2dF
by passing them by const ref rather than value.

Any struct of size > 4 bytes should be passed by const ref.
Passing by ref for these structs is faster than passing
by value, especially when invoking function has multiple parameters.


Pass gfx structs by const ref (gfx::Vector2dF)

BUG=159273

Review URL: https://codereview.chromium.org/130443005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246563 0039d316-1c4b-4281-b951-d872f2087c98
Limit use of full paths in file manager commands.

This patch removes all of the full path dependencies which are possible to be removed at this stage in the file manager commands.

Along the way, an unused method has been removed.

TEST=Tested manually.
BUG=320967

Review URL: https://codereview.chromium.org/141433014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246564 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 246563
Add more coverage to PrefHashCalculatorTest.

Adds an empty dictionary in a dictionary to test a legacy path that now has to be maintained (we were previously pruning out empty dictionaries on hash generation for "stability"; this is no longer required as we outlive anything that could modify prefs on shutdown and come in before any such thing on startup... but we have to keep doing this to avoid yet another hash migration path).

This new test ensures that this use case remains solid.

This is required prior to changing this use-case in https://codereview.chromium.org/145013007/ to prove that that CL doesn't affect the hash being generated.

Also took advantage of this test-only change to update all test paths to be consistent in the modified test; generating new hashes is OK in a test-only change.

BUG=330908

Review URL: https://codereview.chromium.org/145213004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246565 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 246565