|
|
Move the NOTIFICATION_APP_TERMINATING notification from content to chrome, since it's fired by chrome.
I removed the code watching it in the workers devtools code. It's not clear if this code is needed or not, so removing it for now. If we get crash reports or memory leaks on bots, then we'll have a repro that we can use to fix this.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=158566
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+33 lines, -61 lines) |
Patch |
 |
M |
chrome/browser/background/background_mode_manager.cc
|
View
|
|
3 chunks |
+2 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/browser/chromeos/extensions/power/power_api_manager.cc
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/chromeos/input_method/browser_state_monitor.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/chromeos/input_method/browser_state_monitor_unittest.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/chromeos/power/power_button_observer.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/lifetime/application_lifetime.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/notifications/notification_ui_manager_impl.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/prerender/prerender_contents.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/tab_contents/background_contents.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/info_bubble_window.mm
|
View
|
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/extensions/shell_window.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/panels/panel.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/uma_browsing_activity_observer.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/common/chrome_notification_types.h
|
View
|
|
1 chunk |
+7 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/debugger/worker_devtools_manager.cc
|
View
|
|
4 chunks |
+1 line, -26 lines |
0 comments
|
Download
|
|
M |
content/public/browser/notification_types.h
|
View
|
|
1 chunk |
+0 lines, -9 lines |
0 comments
|
Download
|
Total messages: 2 (0 generated)
|
Chromium Code Reviews
Issue
10968064:
Move the NOTIFICATION_APP_TERMINATING notification from content to chrome, since it's fired by chro… (Closed)
