Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(236)

Side by Side Diff: content/public/browser/web_contents_observer.h

Issue 1117023002: Keep event page alive when there's some Pepper plugin on it. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove DCHECK. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « content/common/frame_messages.h ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
7 7
8 #include "base/process/kill.h" 8 #include "base/process/kill.h"
9 #include "base/process/process_handle.h" 9 #include "base/process/process_handle.h"
10 #include "content/common/content_export.h" 10 #include "content/common/content_export.h"
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
271 const std::string& name) {} 271 const std::string& name) {}
272 272
273 // This methods is invoked when the title of the WebContents is set. If the 273 // This methods is invoked when the title of the WebContents is set. If the
274 // title was explicitly set, |explicit_set| is true, otherwise the title was 274 // title was explicitly set, |explicit_set| is true, otherwise the title was
275 // synthesized and |explicit_set| is false. 275 // synthesized and |explicit_set| is false.
276 virtual void TitleWasSet(NavigationEntry* entry, bool explicit_set) {} 276 virtual void TitleWasSet(NavigationEntry* entry, bool explicit_set) {}
277 277
278 virtual void AppCacheAccessed(const GURL& manifest_url, 278 virtual void AppCacheAccessed(const GURL& manifest_url,
279 bool blocked_by_policy) {} 279 bool blocked_by_policy) {}
280 280
281 // These methods are invoked when a Pepper plugin instance is created/deleted
282 // in the DOM.
283 virtual void PepperInstanceCreated() {}
284 virtual void PepperInstanceDeleted() {}
285
281 // Notification that a plugin has crashed. 286 // Notification that a plugin has crashed.
282 // |plugin_pid| is the process ID identifying the plugin process. Note that 287 // |plugin_pid| is the process ID identifying the plugin process. Note that
283 // this ID is supplied by the renderer process, so should not be trusted. 288 // this ID is supplied by the renderer process, so should not be trusted.
284 // Besides, the corresponding process has probably died at this point. The ID 289 // Besides, the corresponding process has probably died at this point. The ID
285 // may even have been reused by a new process. 290 // may even have been reused by a new process.
286 virtual void PluginCrashed(const base::FilePath& plugin_path, 291 virtual void PluginCrashed(const base::FilePath& plugin_path,
287 base::ProcessId plugin_pid) {} 292 base::ProcessId plugin_pid) {}
288 293
289 // Notification that the given plugin has hung or become unhung. This 294 // Notification that the given plugin has hung or become unhung. This
290 // notification is only for Pepper plugins. 295 // notification is only for Pepper plugins.
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
386 void ResetWebContents(); 391 void ResetWebContents();
387 392
388 WebContentsImpl* web_contents_; 393 WebContentsImpl* web_contents_;
389 394
390 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver); 395 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
391 }; 396 };
392 397
393 } // namespace content 398 } // namespace content
394 399
395 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 400 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
OLDNEW
« no previous file with comments | « content/common/frame_messages.h ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698