aboutsummaryrefslogtreecommitdiff
path: root/src/volume-warning.vala
diff options
context:
space:
mode:
authorcharles kerr <charlesk@canonical.com>2015-12-20 10:57:36 -0600
committercharles kerr <charlesk@canonical.com>2015-12-20 10:57:36 -0600
commit293a8bb879ffdf974a204bc4f6e3c84b51c21793 (patch)
treed3ecf5f48efee5fb2f39226ce0e45c079923637f /src/volume-warning.vala
parent0c6cf868928329320466ba2b2e4c00b3f344a1d6 (diff)
downloadayatana-indicator-sound-293a8bb879ffdf974a204bc4f6e3c84b51c21793.tar.gz
ayatana-indicator-sound-293a8bb879ffdf974a204bc4f6e3c84b51c21793.tar.bz2
ayatana-indicator-sound-293a8bb879ffdf974a204bc4f6e3c84b51c21793.zip
change volume-warning to subclass from object instead of volume-control
Now we're starting to get somewhere...
Diffstat (limited to 'src/volume-warning.vala')
-rw-r--r--src/volume-warning.vala44
1 files changed, 9 insertions, 35 deletions
diff --git a/src/volume-warning.vala b/src/volume-warning.vala
index f202242..28bdbcd 100644
--- a/src/volume-warning.vala
+++ b/src/volume-warning.vala
@@ -25,7 +25,7 @@ using Gee;
[CCode(cname="pa_cvolume_set", cheader_filename = "pulse/volume.h")]
extern unowned PulseAudio.CVolume? vol_set2 (PulseAudio.CVolume? cv, uint channels, PulseAudio.Volume v);
-public class VolumeWarning : VolumeControl
+public class VolumeWarning : Object
{
// true if the warning dialog is currently active
public bool active { get; public set; default = false; }
@@ -33,13 +33,6 @@ public class VolumeWarning : VolumeControl
// true iff we're playing unapproved loud multimedia over headphones
public bool high_volume { get; protected set; default = false; }
- // FIXME: this is temporarily necessary while bootstrapping this
- // code because VolumeWarning is still subclassed from VolumeControl,
- // but TBH we don't need any concept of mute here.
- public override void set_mute (bool mute) {
- warning("set_mute not supported for VolumeWarning");
- }
-
/* this is static to ensure it being freed after @context (loop does not have ref counting) */
private static PulseAudio.GLibMainLoop loop;
@@ -59,7 +52,7 @@ public class VolumeWarning : VolumeControl
private bool _pulse_use_stream_restore = false;
private int32 _active_sink_input = -1;
private string[] _valid_roles = {"multimedia", "alert", "alarm", "phone"};
- public override string stream {
+ private string stream {
get {
if (_active_sink_input == -1)
return "alert";
@@ -83,10 +76,11 @@ public class VolumeWarning : VolumeControl
private bool _active_port_headphone = false;
private VolumeControl.ActiveOutput _active_output = VolumeControl.ActiveOutput.SPEAKERS;
+ private IndicatorSound.Options _options;
public VolumeWarning (IndicatorSound.Options options)
{
- base(options);
+ _options = options;
_volume.volume = 0.0;
_volume.reason = VolumeControl.VolumeReasons.PULSE_CHANGE;
@@ -201,18 +195,7 @@ public class VolumeWarning : VolumeControl
break;
case Context.SubscriptionEventType.SOURCE:
- break;
-
case Context.SubscriptionEventType.SOURCE_OUTPUT:
- switch (t & Context.SubscriptionEventType.TYPE_MASK)
- {
- case Context.SubscriptionEventType.NEW:
- c.get_source_output_info (index, source_output_info_cb);
- break;
-
- case Context.SubscriptionEventType.REMOVE:
- break;
- }
break;
}
}
@@ -233,7 +216,6 @@ public class VolumeWarning : VolumeControl
if (active_output_now != active_output_before &&
(active_output_now != VolumeControl.ActiveOutput.CALL_MODE &&
active_output_before != VolumeControl.ActiveOutput.CALL_MODE)) {
- this.active_output_changed (active_output_now);
if (active_output_now == VolumeControl.ActiveOutput.SPEAKERS) {
_high_volume_approved = false;
}
@@ -421,16 +403,6 @@ public class VolumeWarning : VolumeControl
}
}
- private void source_output_info_cb (Context c, SourceOutputInfo? i, int eol)
- {
- if (i == null)
- return;
-
- var role = i.proplist.gets (PulseAudio.Proplist.PROP_MEDIA_ROLE);
- if (role == "phone" || role == "production")
- this.active_mic = true;
- }
-
private bool _connected_to_pulse = false;
private void context_state_callback (Context c)
@@ -495,7 +467,7 @@ public class VolumeWarning : VolumeControl
warning( "pa_context_connect() failed: %s\n", PulseAudio.strerror(context.errno()));
}
- public override VolumeControl.ActiveOutput active_output
+ private VolumeControl.ActiveOutput active_output
{
get
{
@@ -575,7 +547,7 @@ public class VolumeWarning : VolumeControl
}
}
- public override VolumeControl.Volume volume {
+ private VolumeControl.Volume volume {
get {
return _volume;
}
@@ -620,6 +592,8 @@ public class VolumeWarning : VolumeControl
if (high_volume != new_high_volume) {
debug("changing high_volume from %d to %d", (int)high_volume, (int)new_high_volume);
high_volume = new_high_volume;
+ if (high_volume && !active)
+ show(_volume);
}
}
private bool calculate_high_volume() {
@@ -789,7 +763,7 @@ public class VolumeWarning : VolumeControl
private VolumeControl.Volume _cancel_volume = null;
private VolumeControl.Volume _ok_volume = null;
- public void show(VolumeControl.Volume volume) {
+ private void show(VolumeControl.Volume volume) {
// the volume to use if user hits 'cancel'
_cancel_volume = new VolumeControl.Volume();