Fix bug causing tab favicon media indicator to not turn off.

chrome/browser/ui/tabs/tab_utils.cc

Index: chrome/browser/ui/tabs/tab_utils.cc
diff --git a/chrome/browser/ui/tabs/tab_utils.cc b/chrome/browser/ui/tabs/tab_utils.cc
index 93609fe822f8ef7a55997f481f0bb04c00e7072b..5b28bf39c996b8d01e48b5651c14d35551c78687 100644
--- a/chrome/browser/ui/tabs/tab_utils.cc
+++ b/chrome/browser/ui/tabs/tab_utils.cc
@@ -17,7 +17,7 @@ bool ShouldShowProjectingIndicator(content::WebContents* contents) {
   int render_view_id = contents->GetRenderViewHost()->GetRoutingID();
   scoped_refptr<MediaStreamCaptureIndicator> capture_indicator =
       MediaInternals::GetInstance()->GetMediaStreamCaptureIndicator();
-  return capture_indicator->IsProcessCapturingTab(render_process_id,
+  return capture_indicator->IsRenderViewMirroring(render_process_id,

[justinlin, 2013/01/23 19:30:43]

Hmm, not sure about this naming change. Isn't it actually whether the render
process is capturing that we care about?

[miu, 2013/01/23 20:33:28]

Done.  I thought a bit more about the naming here.  Originally, these two
methods were named almost the same way; but the problem I had is that they
indicate two different things.  The two methods can be described as:

1. Returns true if the render view is capturing user media (e.g., webcam).
2. Returns true if the render view itself is being mirrored.

In case #1, the render view is a *consumer* of some external media source.  So,
I went with naming this method IsCapturingUserMedia(rp_id, rv_id).

In case #2, the render view is itself a *source* of media (A/V is being
generated from it).  So, the method name should help communicate that flow of
media is in the opposite direction.  I renamed the method to
IsSupplyingMirroredMedia(rp_id, rv_id).

Do these names sound good to you?

BTW--Also did a little tweak here to make things fit nicer within 80-char lines.
 ;-)

                                                   render_view_id);
 }
 
@@ -29,9 +29,9 @@ bool ShouldShowRecordingIndicator(content::WebContents* contents) {
   // The projecting indicator takes precedence over the recording indicator, but
   // if we are projecting and we don't handle the projecting case we want to
   // still show the recording indicator.
-  return capture_indicator->IsProcessCapturing(render_process_id,
-                                               render_view_id) ||
-         capture_indicator->IsProcessCapturingTab(render_process_id,
+  return capture_indicator->IsRenderViewCapturing(render_process_id,
+                                                  render_view_id) ||
+         capture_indicator->IsRenderViewMirroring(render_process_id,
                                                   render_view_id);
 }