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

Unified Diff: chrome/browser/extensions/api/tabs/tabs_api.cc

Issue 987583004: Add audible, muted to Tab, c.t.query, c.t.update, and c.t.onUpdated where relevant (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@patch1
Patch Set: rebase Created 5 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/tabs/tabs_api.cc
diff --git a/chrome/browser/extensions/api/tabs/tabs_api.cc b/chrome/browser/extensions/api/tabs/tabs_api.cc
index eb9cfa9275b1385148a7e2d12fa16af92716c663..3ca5535293545a261444699215bfc5950454d61c 100644
--- a/chrome/browser/extensions/api/tabs/tabs_api.cc
+++ b/chrome/browser/extensions/api/tabs/tabs_api.cc
@@ -20,6 +20,7 @@
#include "base/strings/string16.h"
#include "base/strings/string_number_conversions.h"
#include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
#include "base/strings/utf_string_conversions.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/apps/scoped_keep_alive.h"
@@ -46,6 +47,7 @@
#include "chrome/browser/ui/host_desktop.h"
#include "chrome/browser/ui/panels/panel_manager.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/tabs/tab_utils.h"
#include "chrome/browser/ui/window_sizer/window_sizer.h"
#include "chrome/browser/web_applications/web_app.h"
#include "chrome/common/chrome_switches.h"
@@ -952,6 +954,16 @@ bool TabsQueryFunction::RunSync() {
continue;
}
+ if (!MatchesBool(params->query_info.audible.get(),
+ chrome::IsPlayingAudio(web_contents))) {
+ continue;
+ }
+
+ if (!MatchesBool(params->query_info.muted.get(),
+ chrome::IsTabAudioMuted(web_contents))) {
+ continue;
+ }
+
if (!title.empty() && !base::MatchPattern(web_contents->GetTitle(),
base::UTF8ToUTF16(title)))
continue;
@@ -1228,6 +1240,27 @@ bool TabsUpdateFunction::RunAsync() {
tab_index = tab_strip->GetIndexOfWebContents(contents);
}
+ if (params->update_properties.muted.get()) {
+ if (chrome::IsTabAudioMutingFeatureEnabled()) {
+ if (!chrome::CanToggleAudioMute(contents)) {
+ WriteToConsole(
+ content::CONSOLE_MESSAGE_LEVEL_WARNING,
+ base::StringPrintf(
+ "Cannot update mute state for tab %d, tab has audio or video "
+ "currently being captured",
+ tab_id));
+ } else {
+ chrome::SetTabAudioMuted(contents, *params->update_properties.muted,
+ extension()->id());
+ }
+ } else {
+ WriteToConsole(content::CONSOLE_MESSAGE_LEVEL_WARNING,
+ base::StringPrintf(
+ "Failed to update mute state, --%s must be enabled",
+ switches::kEnableTabAudioMuting));
+ }
+ }
+
if (params->update_properties.opener_tab_id.get()) {
int opener_id = *params->update_properties.opener_tab_id;
« no previous file with comments | « chrome/browser/extensions/api/tab_capture/tab_capture_apitest.cc ('k') | chrome/browser/extensions/api/tabs/tabs_constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698