Remove dependencies to 'chrome' from device/media_transfer_protocol.

Remove dependencies to 'chrome' from device/media_transfer_protocol.

BUG=171742
TEST=none


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=179938

[Lei Zhang, 2013-01-28 21:38:26]

Why don't we just avoid calling MediaTransferProtocolManager::Initialize() and
Shutdown() instead?

[kmadhusu, 2013-01-31 01:20:16]

On 2013/01/28 21:38:26, Lei Zhang wrote:
> Why don't we just avoid calling MediaTransferProtocolManager::Initialize() and
> Shutdown() instead?

Done. Please review patch set 2.

[Lei Zhang, 2013-01-31 01:33:01]

https://codereview.chromium.org/12082017/diff/6001/chrome/browser/chrome_brow...
File chrome/browser/chrome_browser_main_linux.cc (right):

https://codereview.chromium.org/12082017/diff/6001/chrome/browser/chrome_brow...
chrome/browser/chrome_browser_main_linux.cc:143: did_pre_profile_init_ = true;
You have changed the meaning of this variable. In which case, please give it a
new name.

[kmadhusu, 2013-01-31 02:30:40]

https://codereview.chromium.org/12082017/diff/6001/chrome/browser/chrome_brow...
File chrome/browser/chrome_browser_main_linux.cc (right):

https://codereview.chromium.org/12082017/diff/6001/chrome/browser/chrome_brow...
chrome/browser/chrome_browser_main_linux.cc:143: did_pre_profile_init_ = true;
On 2013/01/31 01:33:01, Lei Zhang wrote:
> You have changed the meaning of this variable. In which case, please give it a
> new name.

Done.

https://codereview.chromium.org/12082017/diff/6001/chrome/browser/chrome_brow...
chrome/browser/chrome_browser_main_linux.cc:152: new
chrome::MediaTransferProtocolDeviceObserverLinux());
We don't need to instantiate MediaTransferProtocolDeviceObserverLinux for
testing.

[Lei Zhang, 2013-01-31 02:34:34]

lgtm

[kmadhusu, 2013-01-31 02:45:15]

gdk@: Can you do the OWNER's check for device/ changes?

Thanks.

[miket_OOO, 2013-01-31 18:05:14]

On 2013/01/31 02:45:15, kmadhusu wrote:
> gdk@: Can you do the OWNER's check for device/ changes?
> 
> Thanks.

OWNER LGTM

[commit-bot: I haz the power, 2013-01-31 18:10:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kmadhusu@chromium.org/12082017/12001

[commit-bot: I haz the power, 2013-01-31 18:15:59]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[commit-bot: I haz the power, 2013-01-31 18:45:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kmadhusu@chromium.org/12082017/12001

[commit-bot: I haz the power, 2013-01-31 20:14:09]

Change committed as 179938