Fix invalid constructor arguments to tsan_analyze.py

The recent changes in the memory tools scripts in r226634
(https://codereview.chromium.org/25815003) breaks some use of
the TSan RV tool. Examples:
http://build.chromium.org/p/client.webrtc.fyi/builders/Linux%20TsanRV

The removal of the source dir parameter to the constructor of
the TsanAnalyzer class was update in one place of valgrind_test.py.
This CL fixes the remaining places.

BUG=none
NOTRY=True
TESTED=Successful local test using:
 GYP_DEFINES=build_with_tool=tsan build/gyp_chromium
 ninja -C out/Release base_unittests
 tools/valgrind/chrome_tests.sh --tool=tsan_rv -b out --target Release -t base_unittests

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229147 0039d316-1c4b-4281-b951-d872f2087c98
Refactor code paths into the Create Shortcuts dialog.

Removed BrowserWindow::ShowCreateChromeAppShortcutsDialog -- this no
longer makes sense because the dialog can also be launched from the App
Launcher which does not use BrowserWindow.

chrome::ShowCreateChromeAppShortcutsDialog is now available on all
platforms (Views, Cocoa and GTK, not just Views).

The New Tab Page now calls this function directly, instead of going
through BrowserWindow. This is consistent with how the App Launcher
opens this dialog.

BUG=307837

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229148 0039d316-1c4b-4281-b951-d872f2087c98
Fixed always setting RTP to true when answering a
peerconnection call

In messages_handlings.js true was passed on as a third
paremeter when setting up a peerconnection which always 
enabled RTP data channels when answering a call. The 
reason for this is that the when initiating a 
peerconnection it takes place in peerconnection.js while 
answer is handled by message_handling.js.

I had to create a global variable in message_handling.js 
which is initiallty true due to browser tests awlays 
assume RTP data channels is being used. A function is 
called in peerconnection.js when the RTP check box on 
peerconnection.html is ticked that sets this global 
variable. This function is also called when the 
window.unload event fires to in order to sync the 
check box and the global variable.

BUG=307986
TEST=Manual tests on peerconnection.html and WebRTC browsertests
NOTRY=TRUE

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

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