DevTools: add tests for device mode's responsive mode

DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002

BUG=651123
Committed: https://crrev.com/ae4bf8b7cae4ad6190ce8efadf39c28eed0d2cc8
Cr-Commit-Position: refs/heads/master@{#427876}

[luoe, 2016-10-25 18:52:56]

luoe@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2016-10-26 01:56:15]

Description was changed from

==========
DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002
==========

to

==========
DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002

BUG=651123
==========

[dgozman, 2016-10-26 18:35:23]

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/device-mode/device-mode-responsive-expected.txt
(right):

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/device-mode/device-mode-responsive-expected.txt:1:
1
Missing output?

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeToolbar.js
(right):

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeToolbar.js:117:
* @this {WebInspector.DeviceModeToolbar}
Don't need @this annotations on prototype functions.

[luoe, 2016-10-26 18:56:51]

My bad, I left out the test results.  They should be included now.  Before the
ceil() -> floor() change, this test fails, showing a different scale in the
diff.

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/device-mode/device-mode-responsive-expected.txt
(right):

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/device-mode/device-mode-responsive-expected.txt:1:
1
Sorry, I checked the test results but didn't run --reset-result.  I should
double check in the future!

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeToolbar.js
(right):

https://codereview.chromium.org/2446113004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeToolbar.js:117:
* @this {WebInspector.DeviceModeToolbar}
On 2016/10/26 18:35:22, dgozman wrote:
> Don't need @this annotations on prototype functions.

Done.

[dgozman, 2016-10-26 21:55:40]

lgtm. Thanks!

[luoe, 2016-10-26 22:07:31]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 22:08:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 23:24:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 23:24:58]

Dry run: This issue passed the CQ dry run.

[luoe, 2016-10-26 23:26:33]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2016-10-26 23:27:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 23:33:32]

Description was changed from

==========
DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002

BUG=651123
==========

to

==========
DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002

BUG=651123
==========

[commit-bot: I haz the power, 2016-10-26 23:33:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-26 23:37:58]

Description was changed from

==========
DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002

BUG=651123
==========

to

==========
DevTools: add tests for device mode's responsive mode

This adds tests that reflect changes after this was fixed:
https://codereview.chromium.org/2449763002

BUG=651123

Committed: https://crrev.com/ae4bf8b7cae4ad6190ce8efadf39c28eed0d2cc8
Cr-Commit-Position: refs/heads/master@{#427876}
==========

[commit-bot: I haz the power, 2016-10-26 23:37:59]

Patchset 2 (id:??) landed as
https://crrev.com/ae4bf8b7cae4ad6190ce8efadf39c28eed0d2cc8
Cr-Commit-Position: refs/heads/master@{#427876}