Fixed a bug in DesktopMediaID class

Fixed an apparent bug introduced in
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. "other." is missed in front of web_content_id

BUG=557222
Committed: https://crrev.com/11f2c00dc0e05b7208a3d1730539866e30e289b9
Cr-Commit-Position: refs/heads/master@{#372353}

[GeorgeZ, 2016-01-27 19:40:23]

Description was changed from

==========
fixed a bug

BUG=55722
==========

to

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

BUG=557222
==========

[GeorgeZ, 2016-01-27 20:31:23]

gyzhou@chromium.org changed reviewers:
+ jochen@chromium.org

[GeorgeZ, 2016-01-27 20:31:23]

Hi Jochen,

Would you please have a review of this simple CL.

Thanks,

George

[jochen (gone - plz use gerrit), 2016-01-28 12:14:21]

can you update the CL description to have the one-line summary as the first
line?

is it possible to add a test for this?

[GeorgeZ, 2016-01-28 15:24:33]

Description was changed from

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

BUG=557222
==========

to

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. The code was   
return std::tie(type, id, web_contents_id) <
         std::tie(other.type, other.id, web_contents_id); 
and it suppose to be
  return std::tie(type, id, web_contents_id) <
         std::tie(other.type, other.id,    other.web_contents_id);

other. is missed in front of web_content_id

BUG=557222
==========

[GeorgeZ, 2016-01-28 15:34:13]

On 2016/01/28 12:14:21, jochen wrote:
> can you update the CL description to have the one-line summary as the first
> line?
> 
> is it possible to add a test for this?

Jochen,

I updated the CL description as required.

This bug was actually caught by by ongoing development of new class
TabDesktopMediaList's unit tests. After the CL for TabDesktopMediaList is
created and committed, unit tests will cover this bug.

Since this is an apparently bug, I think it is better to fix it earlier than
latter.

Thanks,

George

[GeorgeZ, 2016-01-28 15:35:00]

Description was changed from

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. The code was   
return std::tie(type, id, web_contents_id) <
         std::tie(other.type, other.id, web_contents_id); 
and it suppose to be
  return std::tie(type, id, web_contents_id) <
         std::tie(other.type, other.id,    other.web_contents_id);

other. is missed in front of web_content_id

BUG=557222
==========

to

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. "other." is
missed in front of web_content_id

BUG=557222
==========

[jochen (gone - plz use gerrit), 2016-01-29 09:17:00]

lgtm

[GeorgeZ, 2016-01-29 15:59:32]

The CQ bit was checked by gyzhou@chromium.org

[commit-bot: I haz the power, 2016-01-29 16:00:36]

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

[commit-bot: I haz the power, 2016-01-29 16:13:09]

Description was changed from

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. "other." is
missed in front of web_content_id

BUG=557222
==========

to

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. "other." is
missed in front of web_content_id

BUG=557222
==========

[commit-bot: I haz the power, 2016-01-29 16:13:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-29 16:14:06]

Description was changed from

==========
Fixed an apparent bug introduced in 
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. "other." is
missed in front of web_content_id

BUG=557222
==========

to

==========
Fixed an apparent bug introduced in
https://codereview.chromium.org/1503563004/

This is a bug in DesktopMediaID class's overloaded operator <. "other." is
missed in front of web_content_id

BUG=557222

Committed: https://crrev.com/11f2c00dc0e05b7208a3d1730539866e30e289b9
Cr-Commit-Position: refs/heads/master@{#372353}
==========

[commit-bot: I haz the power, 2016-01-29 16:14:07]

Patchset 1 (id:??) landed as
https://crrev.com/11f2c00dc0e05b7208a3d1730539866e30e289b9
Cr-Commit-Position: refs/heads/master@{#372353}