[rAC] remove duplicate code.

TooltipIcon was defined twice, once in the dialog, once in a separate
field.

BUG=none
R=estade@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230659 0039d316-1c4b-4281-b951-d872f2087c98
Relanding this with fixes for the SyncPolicyTest.TestEvent and SyncPolicyTest.TestEventReadOnly tests.

Replace the CreateEvent/OpenEvent patches with their Nt counterparts like
NtOpenEvent and NtCreateEvent.

Reason being :- We patch these APIS via the Export table patch which does not
work with bound imports. This results in our
patched functions never getting called.

This should fix the GPU process hang with the XP presentation path.

BUG=305815
TBR=cpu,rvargas,thakis

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230660 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 230660
Remove reliance on GetActiveDesktop from AppListService::Get()

chrome::GetActiveDesktop() should only be used in tests. On Windows 8
split-screen there are cases where it picks the incorrect desktop
resulting in the incorrect app launcher being used.

Callers of AppListService::Get() are now required to specify the correct
HostDesktopType to increase the likelihood that the correct app list is
returned.

BUG=303524
TEST=Have app launcher & exit from Chrome completely.
- Open Chrome on the Desktop, leave browser window there.
- Open Chrome from Windows 8 Mode. Open and close App Launcher in Ash.
- Go into split screen with Ash in a small part of the screen on the left.
- Open Windows Desktop on the right side. Open App Launcher from the taskbar.
-> App launcher should show in Desktop Mode, not Ash.

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

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