Fix some uses of global variables in transient pages.

Fix some uses of global variables in transient pages.
- contextMenus.create now fetches a unique ID from the browser process instead
of a process-wide counter.
- tts and socket both keep the transient page alive until their event handlers
signal completion, so their process-wide counter remains valid.

BUG=119614
TEST=no

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

[Matt Perry, 2012-04-04 00:32:42]

http://codereview.chromium.org/9958147/diff/1/chrome/common/extensions/extens...
File chrome/common/extensions/extension_messages.h (left):

http://codereview.chromium.org/9958147/diff/1/chrome/common/extensions/extens...
chrome/common/extensions/extension_messages.h:359:
IPC_MESSAGE_ROUTED3(ExtensionMsg_InlineWebstoreInstallResponse,
I moved the ExtensionMsg's up to their section.

[Yoyo Zhou, 2012-04-04 18:52:24]

LGTM

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/browser/renderer...
File chrome/browser/renderer_host/chrome_render_message_filter.cc (right):

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/browser/renderer...
chrome/browser/renderer_host/chrome_render_message_filter.cc:431: if
(profile_->GetExtensionProcessManager() && service) {
Use this instead: ExtensionProcessManager* process_manager =
ExtensionSystemFactory::GetForProfile(profile_)->process_manager();

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/common/extension...
File chrome/common/extensions/extension_messages.h (right):

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/common/extension...
chrome/common/extensions/extension_messages.h:266: // Response to the renderer
for the above message.
"the above message" was already wrong, but change it to GetAppNotifyChannel.

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/renderer/extensi...
File chrome/renderer/extensions/extension_dispatcher.cc (right):

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/renderer/extensi...
chrome/renderer/extensions/extension_dispatcher.cc:160: bool
IsLazyBackgroundPage(const std::string& extension_id) {
This name seems to be... off.

[Matt Perry, 2012-04-04 19:21:44]

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/browser/renderer...
File chrome/browser/renderer_host/chrome_render_message_filter.cc (right):

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/browser/renderer...
chrome/browser/renderer_host/chrome_render_message_filter.cc:431: if
(profile_->GetExtensionProcessManager() && service) {
On 2012/04/04 18:52:24, Yoyo Zhou wrote:
> Use this instead: ExtensionProcessManager* process_manager =
> ExtensionSystemFactory::GetForProfile(profile_)->process_manager();

Done.

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/common/extension...
File chrome/common/extensions/extension_messages.h (right):

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/common/extension...
chrome/common/extensions/extension_messages.h:266: // Response to the renderer
for the above message.
On 2012/04/04 18:52:24, Yoyo Zhou wrote:
> "the above message" was already wrong, but change it to GetAppNotifyChannel.

Done.

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/renderer/extensi...
File chrome/renderer/extensions/extension_dispatcher.cc (right):

https://chromiumcodereview.appspot.com/9958147/diff/1/chrome/renderer/extensi...
chrome/renderer/extensions/extension_dispatcher.cc:160: bool
IsLazyBackgroundPage(const std::string& extension_id) {
On 2012/04/04 18:52:24, Yoyo Zhou wrote:
> This name seems to be... off.

Changed to IsCurrentContextLazyBackgroundPage