diff options
| author | kelvinp <kelvinp@chromium.org> | 2015-08-14 16:45:32 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-14 23:46:01 +0000 |
| commit | 248594dba42ad74eb89a6663304462e6538ba568 (patch) | |
| tree | d7ef5bcc673cd0e647d473f7575cbacc58e117b9 /remoting/remoting_webapp_files.gypi | |
| parent | ad61982619147865d7feb3b92b6c28d6487a58a2 (diff) | |
| download | chromium_src-248594dba42ad74eb89a6663304462e6538ba568.zip chromium_src-248594dba42ad74eb89a6663304462e6538ba568.tar.gz chromium_src-248594dba42ad74eb89a6663304462e6538ba568.tar.bz2 | |
Record the XMPP error in remoting.FormatIq.
This CL
1. Records the XMPP error type, jingle_action and error condition
in remoting.FormatIq.
This error is later extracted by the ClientSession to upload along
with session state change events.
We choose to record the error in the webapp instead of the plugin to
avoid adding new contract between the webapp and the plugin just for
the sake of debugging, especially when we are so close to branch point.
2. Removes some legacy fields in the FormatIq to avoid parse errors.
3. Parses the error before the original jingle message. This ensures
addition of unrecognized Jingle fields like QUIC will not break the
error parsing.
BUG=509070
Review URL: https://codereview.chromium.org/1289333004
Cr-Commit-Position: refs/heads/master@{#343532}
Diffstat (limited to 'remoting/remoting_webapp_files.gypi')
| -rw-r--r-- | remoting/remoting_webapp_files.gypi | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/remoting/remoting_webapp_files.gypi b/remoting/remoting_webapp_files.gypi index 70f6819..e41ddd6 100644 --- a/remoting/remoting_webapp_files.gypi +++ b/remoting/remoting_webapp_files.gypi @@ -93,6 +93,7 @@ 'webapp/base/js/dns_blackhole_checker_unittest.js', 'webapp/base/js/error_unittest.js', 'webapp/base/js/fallback_signal_strategy_unittest.js', + 'webapp/base/js/format_iq_unittest.js', 'webapp/base/js/identity_unittest.js', 'webapp/base/js/ipc_unittest.js', 'webapp/base/js/l10n_unittest.js', |