Chromium Code Reviews| Index: third_party/WebKit/Source/modules/mediastream/MediaConstraintsImpl.cpp |
| diff --git a/third_party/WebKit/Source/modules/mediastream/MediaConstraintsImpl.cpp b/third_party/WebKit/Source/modules/mediastream/MediaConstraintsImpl.cpp |
| index ee8148fb7b52fc73a0bb5768f49e01e67328534b..c49865f25ee17426da4f17d13412c1eec61db5d7 100644 |
| --- a/third_party/WebKit/Source/modules/mediastream/MediaConstraintsImpl.cpp |
| +++ b/third_party/WebKit/Source/modules/mediastream/MediaConstraintsImpl.cpp |
| @@ -122,6 +122,11 @@ const char kCpuOveruseEncodeUsage[] = "googCpuOveruseEncodeUsage"; |
| const char kHighStartBitrate[] = "googHighStartBitrate"; |
| const char kPayloadPadding[] = "googPayloadPadding"; |
| // End of names from libjingle |
| +// Names that have been used in the past, but should now be ignored. |
| +// Kept around for backwards compatibility. |
| +// https://crbug.com/579729 |
| +const char kGoogLeakyBucket[] = "googLeakyBucket"; |
| +const char kEnableAutoThrottling[] = "enableAutoThrottling"; |
| // Names used for testing. |
| const char kTestConstraint1[] = "valid_and_supported_1"; |
| @@ -352,6 +357,9 @@ static void parseOldStyleNames(const WebVector<WebMediaConstraint>& oldNames, We |
| result.googHighStartBitrate.setExact(atoi(constraint.m_value.utf8().c_str())); |
| } else if (constraint.m_name.equals(kPayloadPadding)) { |
| result.googPayloadPadding.setExact(toBoolean(constraint.m_value)); |
| + } else if (constraint.m_name.equals(kGoogLeakyBucket) |
| + || constraint.m_name.equals(kEnableAutoThrottling)) { |
| + WTF_LOG(Media, "Obsolete constraint %s passed", constraint.m_name.utf8().c_str()); |
|
jochen (gone - plz use gerrit)
2016/01/28 08:54:37
is it reasonable to log something to the console (
|
| } else if (constraint.m_name.equals(kTestConstraint1) |
| || constraint.m_name.equals(kTestConstraint2)) { |
| // These constraints are only for testing parsing. |