Reverted the code for the non-interactive audible tab notification

Reverted the code for the non-interactive audible tab notification

Depends on https://codereview.chromium.org/1159113006 (just landed)
Reverts:
- https://codereview.chromium.org/975943003
- https://codereview.chromium.org/1008093002
- https://chrome-internal-review.googlesource.com/#/c/204475/
- https://chrome-internal-review.googlesource.com/#/c/210391/

BUG=470600
Committed: https://crrev.com/cc85bbd51f2dd545ffeadde5b0c739aceca5443b
Cr-Commit-Position: refs/heads/master@{#346205}

[whywhat, 2015-07-08 17:25:50]

avayvod@chromium.org changed reviewers:
+ jam@chromium.org, qinmin@chromium.org, timav@chromium.org

[whywhat, 2015-07-08 17:25:51]

Please, take a look.

Min and Tima to check if I haven't missed anything
John for OWNERS lgtm

[Ted C, 2015-07-08 17:28:45]

On 2015/07/08 17:25:51, whywhat wrote:
> Please, take a look.
> 
> Min and Tima to check if I haven't missed anything
> John for OWNERS lgtm

Don't we want to keep the notification for non-controllable audio?

[whywhat, 2015-07-08 17:36:59]

On 2015/07/08 at 17:28:45, tedchoc wrote:
> On 2015/07/08 17:25:51, whywhat wrote:
> > Please, take a look.
> > 
> > Min and Tima to check if I haven't missed anything
> > John for OWNERS lgtm
> 
> Don't we want to keep the notification for non-controllable audio?

Currently, non-controllable audio is only the audio that's shorter than 5
seconds, we should not bother the user with the notification in this case.
All the future media session types we're going to support will only be able to
play in the foreground and then having a notification is silly (since it only
allows user to go to the notification area from the tab that's in the foreground
and then back to the tab).

[qinmin, 2015-07-08 18:46:21]

On 2015/07/08 17:36:59, whywhat wrote:
> On 2015/07/08 at 17:28:45, tedchoc wrote:
> > On 2015/07/08 17:25:51, whywhat wrote:
> > > Please, take a look.
> > > 
> > > Min and Tima to check if I haven't missed anything
> > > John for OWNERS lgtm
> > 
> > Don't we want to keep the notification for non-controllable audio?
> 
> Currently, non-controllable audio is only the audio that's shorter than 5
> seconds, we should not bother the user with the notification in this case.
> All the future media session types we're going to support will only be able to
> play in the foreground and then having a notification is silly (since it only
> allows user to go to the notification area from the tab that's in the
foreground
> and then back to the tab).

wait, so if a user starts playing audio stream in foreground, and creates
another new tab will pause the audio?
And what about multi window case?

[jam, 2015-07-09 16:42:54]

On 2015/07/08 17:25:51, whywhat wrote:
> Please, take a look.
> 
> Min and Tima to check if I haven't missed anything
> John for OWNERS lgtm

owners of what? 

can you find a better owner than me? who reviewed this initially.

[whywhat, 2015-08-26 16:12:45]

The CQ bit was checked by avayvod@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-08-26 16:12:59]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1215713021/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1215713021/40001

[commit-bot: I haz the power, 2015-08-26 16:35:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-26 16:35:43]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[whywhat, 2015-08-26 19:07:58]

avayvod@chromium.org changed reviewers:
+ dtrainor@chromium.org, sievers@chromium.org, tedchoc@chromium.org
- jam@chromium.org, timav@chromium.org

[whywhat, 2015-08-26 19:07:59]

PTaL (note this is the revert of the previous four patches you approved before):

tedchoc:
  chrome/android/java/AndroidManifest.xml
 
chrome/android/java/src/org/chromium/chrome/browser/ChromeWebContentsDelegateAndroid.java
 
chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
 
chrome/android/java/src/org/chromium/chrome/browser/media/MediaCaptureNotificationService.java
 
chrome/android/java/src/org/chromium/chrome/browser/media/MediaNotificationService.java
  chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java
  chrome/android/java/strings/android_chrome_strings.grd
  chrome/browser/android/chrome_web_contents_delegate_android.cc
  content/browser/android/media_players_observer.cc
  content/browser/android/media_players_observer.h
dtrainor:
  chrome/android/java/AndroidManifest.xml
 
chrome/android/java/src/org/chromium/chrome/browser/ChromeWebContentsDelegateAndroid.java
 
chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
 
chrome/android/java/src/org/chromium/chrome/browser/media/MediaCaptureNotificationService.java
 
chrome/android/java/src/org/chromium/chrome/browser/media/MediaNotificationService.java
  chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java
  chrome/android/java/strings/android_chrome_strings.grd
  chrome/app/android/chrome_main_delegate_android.cc
  chrome/browser/android/chrome_web_contents_delegate_android.cc
qinmin:
  content/browser/media/android/browser_media_player_manager.cc
  content/browser/media/android/browser_media_player_manager.h
  media/base/android/audio_decoder_job.h
  media/base/android/media_codec_player_unittest.cc
  media/base/android/media_decoder_job.h
  media/base/android/media_player_android.cc
  media/base/android/media_player_android.h
  media/base/android/media_player_bridge.cc
  media/base/android/media_player_manager.h
  media/base/android/media_source_player.cc
  media/base/android/media_source_player_unittest.cc
sievers:
  content/browser/android/media_players_observer.cc
  content/browser/android/media_players_observer.h
  content/browser/media/android/browser_media_player_manager.cc
  content/browser/media/android/browser_media_player_manager.h
  content/browser/media/audio_state_provider.cc
  content/browser/media/audio_state_provider.h
  content/browser/media/audio_stream_monitor.cc
  content/browser/media/audio_stream_monitor.h
  content/browser/media/audio_stream_monitor_unittest.cc
  content/browser/media/media_web_contents_observer.cc
  content/browser/media/media_web_contents_observer.h
  content/browser/web_contents/web_contents_impl.cc
  content/browser/web_contents/web_contents_impl.h
  content/browser/web_contents/web_contents_impl_unittest.cc

[whywhat, 2015-08-27 23:31:43]

Ping!

[qinmin, 2015-08-27 23:45:45]

lgtm

[Ted C, 2015-08-27 23:58:44]

lgtm

https://codereview.chromium.org/1215713021/diff/40001/content/browser/media/a...
File content/browser/media/android/browser_media_player_manager.cc (right):

https://codereview.chromium.org/1215713021/diff/40001/content/browser/media/a...
content/browser/media/android/browser_media_player_manager.cc:10: #include
"content/browser/media/android/browser_demuxer_android.h"
indented too much

[David Trainor- moved to gerrit, 2015-08-28 05:09:06]

lgtm

[whywhat, 2015-08-28 11:47:00]

Fixed indent

[whywhat, 2015-08-28 11:49:05]

The CQ bit was checked by avayvod@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-08-28 11:49:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1215713021/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1215713021/60001

[commit-bot: I haz the power, 2015-08-28 12:54:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-28 12:54:11]

Dry run: This issue passed the CQ dry run.

[no sievers, 2015-08-28 17:50:28]

content lgtm

[whywhat, 2015-08-28 19:05:06]

The CQ bit was checked by avayvod@chromium.org

[whywhat, 2015-08-28 19:05:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, dtrainor@chromium.org, qinmin@chromium.org
Link to the patchset: https://codereview.chromium.org/1215713021/#ps60001
(title: "Fixed indent")

[commit-bot: I haz the power, 2015-08-28 19:05:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1215713021/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1215713021/60001

[commit-bot: I haz the power, 2015-08-28 19:11:21]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-08-28 19:12:23]

Patchset 4 (id:??) landed as
https://crrev.com/cc85bbd51f2dd545ffeadde5b0c739aceca5443b
Cr-Commit-Position: refs/heads/master@{#346205}