Reland r265807: "Moves the notification icon out of th..."

Reland r265807: "Moves the notification icon out of th..."

> Moves the notification icon out of the status area overflow.
>
> This only happens the first time it is created per data dir,
> so that the user can change back their setting if desired.
>
> This approach uses COM to talk with Windows Explorer to
> change the user's notification area preferences.
>
> R=dimich@chromium.org
> BUG=347693
>
> Review URL: https://codereview.chromium.org/110693004
>
> git-svn-id: svn://svn.chromium.org/chrome/trunk/src@265807 0039d316-1c4b-4281-b951-d872f2087c98

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

[dewittj, 2014-04-24 20:05:44]

PTAL, needed to disable the test that actually measures tray size changes on
Vista since new/active icons are already shown by default on that OS.

[sky, 2014-04-24 20:42:53]

LGTM

[dewittj, 2014-04-24 20:45:12]

The CQ bit was checked by dewittj@chromium.org

[commit-bot: I haz the power, 2014-04-24 21:15:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dewittj@chromium.org/252513004/20001

[commit-bot: I haz the power, 2014-04-24 22:46:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-24 22:46:51]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

[dewittj, 2014-04-24 23:16:52]

The CQ bit was checked by dewittj@chromium.org

[commit-bot: I haz the power, 2014-04-24 23:17:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dewittj@chromium.org/252513004/20001

[commit-bot: I haz the power, 2014-04-25 02:35:06]

Change committed as 266091

[Nico, 2014-07-18 16:19:53]

https://codereview.chromium.org/252513004/diff/20001/chrome/browser/ui/views/...
File chrome/browser/ui/views/status_icons/status_tray_win.h (right):

https://codereview.chromium.org/252513004/diff/20001/chrome/browser/ui/views/...
chrome/browser/ui/views/status_icons/status_tray_win.h:19: class
StatusTrayStateChangerProxy {
Since this class has subclasses and is deleted polymorphically (there's a
scoped_ptr<StatusTrayStateChangerProxy> below but it always points at a
subclass), this class should have a virtual destructor – else the subclass
destructor won't be called and subclass members won't be destructed correctly.