Browser Plugin: Rename old colliding files to old_*.

Browser Plugin: Rename old colliding files to old_*.

Some builds don't like two files in a project with the same name.

This addresses the issue. 

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

[Fady Samuel, 2012-08-07 23:22:53]

Adding creis@ to review as suggested by jam@.

[Charlie Reis, 2012-08-07 23:58:28]

LGTM.  Which builds had trouble with the duplicate file names?

http://codereview.chromium.org/10830204/diff/1/content/browser/browser_plugin...
File content/browser/browser_plugin/old/old_browser_plugin_host.h (right):

http://codereview.chromium.org/10830204/diff/1/content/browser/browser_plugin...
content/browser/browser_plugin/old/old_browser_plugin_host.h:37: class
BrowserPluginHost : public WebContentsObserver,
I suppose we don't need to rename the class itself, since it's going to be
removed shortly anyway.  It won't cause name conflicts because of the "old"
namespace, correct?

[Fady Samuel, 2012-08-08 00:41:35]

On 2012/08/07 23:58:28, creis wrote:
> LGTM.  Which builds had trouble with the duplicate file names?
> 
>
http://codereview.chromium.org/10830204/diff/1/content/browser/browser_plugin...
> File content/browser/browser_plugin/old/old_browser_plugin_host.h (right):
> 
>
http://codereview.chromium.org/10830204/diff/1/content/browser/browser_plugin...
> content/browser/browser_plugin/old/old_browser_plugin_host.h:37: class
> BrowserPluginHost : public WebContentsObserver,
> I suppose we don't need to rename the class itself, since it's going to be
> removed shortly anyway.  It won't cause name conflicts because of the "old"
> namespace, correct?

win, mac, linux_rel, win_rel. Committing now thanks. Yea, no other name
conflicts due to the old namespace.

[commit-bot: I haz the power, 2012-08-08 00:41:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fsamuel@chromium.org/10830204/1

[commit-bot: I haz the power, 2012-08-08 02:02:57]

Try job failure for 10830204-1 (retry) on linux_rel for step
"interactive_ui_tests".
It's a second try, previously, step "interactive_ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-08-08 02:13:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fsamuel@chromium.org/10830204/1

[commit-bot: I haz the power, 2012-08-08 04:40:15]

Change committed as 150498