| Index: chrome/browser/ui/tab_helpers.h
|
| diff --git a/chrome/browser/ui/tab_helpers.h b/chrome/browser/ui/tab_helpers.h
|
| index 1c678f2b55a08b530b7c09e7db86c4f0b2ee0b15..7e32de6cf81a4dcf4ef225bce9c267d1e0859c80 100644
|
| --- a/chrome/browser/ui/tab_helpers.h
|
| +++ b/chrome/browser/ui/tab_helpers.h
|
| @@ -25,6 +25,8 @@ class PrerenderContents;
|
| // (or the equivalent on Android). The TabHelpers class allows specific classes
|
| // to attach the set of tab helpers that is used for tab contents.
|
| //
|
| +// https://www.chromium.org/developers/design-documents/tab-helpers
|
| +//
|
| // TODO(avi): This list is rather large, and for most callers it's due to the
|
| // fact that they need tab helpers attached early to deal with arbitrary
|
| // content loaded into a WebContents that will later be added to the tabstrip.
|
| @@ -50,6 +52,10 @@ class TabHelpers {
|
| // the full set of tab helpers to deal with it.
|
| friend class prerender::PrerenderContents;
|
|
|
| + // FYI: Do NOT add any more friends here. The functions above are the ONLY
|
| + // ones that need to call AttachTabHelpers; if you think you do, re-read the
|
| + // design document linked above, especially the section "Reusing tab helpers".
|
| +
|
| // Adopts the specified WebContents as a full-fledged browser tab, attaching
|
| // all the associated tab helpers that are needed for the WebContents to
|
| // serve in that role. It is safe to call this on a WebContents that was
|
|
|