Remove WebContents::Was{Hidden,Shown}() from the content public interface

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 4f8a3f26c139feb35a84f65b01cf1d0530e1133f..61838e4a778fdb9c8f55a6f4154e914387019d42 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -382,9 +382,9 @@ class WebContents : public PageNavigator,
   virtual base::TimeTicks GetLastActiveTime() const = 0;
   virtual void SetLastActiveTime(base::TimeTicks last_active_time) = 0;
 
-  // Invoked when the WebContents becomes shown/hidden.
-  virtual void WasShown() = 0;
-  virtual void WasHidden() = 0;
+  // Hides the RenderWidgetHostView if it is not being used for capture, and
+  // emits WebContentsObserver::WasHidden().
+  virtual void MarkBackgrounded() = 0;

[jam, 2016/02/29 17:36:36]

what's the other analogous method?

i.e. it seems this is for undoing a previous call.where is that call?
shouldn't they be together?

[tapted, 2016/02/29 22:42:32]

Ah, that's the thing :). `MarkBackgrounded` actually shouldn't exist, and
embedders don't really need the analogous method. That's because they're
currently using calls to WebContents::WasHidden() shortly after creating a
WebContents, but with that WebContents created with
WebContents::InitParams::initially_hidden set to false, rather than true. For
Aura platforms, the `WasShown` call to balance a WasHidden is always made by the
content layer, by WebContentsView. (and http://crrev.com/1733173002 makes that
true for Mac as well as Aura).

Exposing `WasHidden` can lead to bugs - e.g. http://crbug.com/590476 which came
up recently. Part of the issue is that calls to `WasHidden` (or MarkBackgrounded
which this CL renames it to) can be called on WebContents that are already
hidden.

This CL annotates all the `WasHidden` calls that Chrome was making. I think they
can all be removed with appropriate use of
WebContents::InitParams::initially_hidden . (Or perhaps
`InitParams::initial_state = {BACKGROUND_PRELOAD, FOREGROUND}` if that's
clearer. Maybe there's a use case for `UNLOADED` as well, but there doesn't seem
to be).

 
   // Returns true if the before unload and unload listeners need to be
   // fired. The value of this changes over time. For example, if true and the