Remove invert viewport scroll order setting from Blink

Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724
Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}

Committed: https://crrev.com/993ed1a7d9f986c1c87f0368003b2baf0589ffb0
Cr-Commit-Position: refs/heads/master@{#367373}

[bokan, 2015-11-13 03:01:37]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

BUG=
==========

to

==========
Remove invert viewport scroll order setting from Blink

BUG=443724
==========

[bokan, 2015-11-13 03:07:48]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

BUG=443724
==========

to

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724
==========

[bokan, 2015-11-13 03:08:29]

bokan@chromium.org changed reviewers:
+ avi@chromium.org, rbyers@chromium.org, rjkroege@chromium.org

[bokan, 2015-11-13 03:08:30]

rbyers@chromium.org for public/web
avi@chromium.org: for content
rjkroege@chromium.org: for html_viewer

[bokan, 2015-11-13 03:10:21]

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

[commit-bot: I haz the power, 2015-11-13 03:11:08]

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

[rjkroege, 2015-11-13 03:18:53]

On 2015/11/13 03:11:08, commit-bot: I haz the power wrote:
> Dry run: CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1435233002/1
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1435233002/1

components/html_viewer/blink_settings_impl.cc lgtm

[commit-bot: I haz the power, 2015-11-13 03:33:33]

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

[commit-bot: I haz the power, 2015-11-13 03:33:34]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Avi (use Gerrit), 2015-11-13 15:54:06]

lgtm

🎉👍

[Rick Byers, 2015-11-14 03:57:39]

LGTM

[bokan, 2015-11-17 16:46:42]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2015-11-17 16:46:57]

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

[commit-bot: I haz the power, 2015-11-17 17:37:41]

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

[commit-bot: I haz the power, 2015-11-17 17:37:42]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[bokan, 2015-12-04 20:31:25]

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

[commit-bot: I haz the power, 2015-12-04 20:33:12]

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

[commit-bot: I haz the power, 2015-12-04 21:36:23]

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

[commit-bot: I haz the power, 2015-12-04 21:36:24]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[bokan, 2015-12-07 21:24:58]

The CQ bit was checked by bokan@chromium.org

[bokan, 2015-12-07 21:24:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
rjkroege@chromium.org, avi@chromium.org, rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1435233002/#ps40001
(title: "Fixed broken unit tests")

[commit-bot: I haz the power, 2015-12-07 21:26:50]

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

[commit-bot: I haz the power, 2015-12-07 23:26:30]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724
==========

to

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724
==========

[commit-bot: I haz the power, 2015-12-07 23:26:30]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-07 23:27:17]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724
==========

to

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}
==========

[commit-bot: I haz the power, 2015-12-07 23:27:27]

Patchset 3 (id:??) landed as
https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}

[Lambros, 2015-12-08 00:04:43]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1509783002/ by lambroslambrou@chromium.org.

The reason for reverting is: Speculatively reverting, as it might have caused
compile failures:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/5...

ninja explain: output
obj/third_party/WebKit/Source/core/gen/blink/bindings/core/v8/webcore_testing.V8Internals.o
older than most recent input
../../out/Release/gen/blink/core/testing/InternalSettingsGenerated.h (1449527421
vs 1449531710)
ninja explain:
obj/third_party/WebKit/Source/core/gen/blink/bindings/core/v8/webcore_testing.V8Internals.o
is dirty
ninja explain: output
obj/third_party/WebKit/Source/core/gen/blink/bindings/core/v8/webcore_testing.V8InternalSettings.o
older than most recent input
../../out/Release/gen/blink/core/testing/InternalSettingsGenerated.h (1449527414
vs 1449531710)
ninja explain:
obj/third_party/WebKit/Source/core/gen/blink/bindings/core/v8/webcore_testing.V8InternalSettings.o
is dirty
ninja explain: obj/third_party/WebKit/Source/core/libwebcore_testing.a is dirty
ninja explain: webkit_unit_tests is dirty
ninja explain: obj/third_party/WebKit/public/all_blink.actions_depends.stamp is
dirty
ninja explain: obj/third_party/WebKit/Source/core/libwebcore_testing.a is dirty
ninja explain: content_shell is dirty
ninja explain: obj/build/blink_tests.actions_depends.stamp is dirty
ninja explain: obj/third_party/WebKit/Source/core/libwebcore_testing.a is dirty
[1/7] CXX
obj/third_party/WebKit/Source/core/gen/blink/bindings/core/v8/webcore_testing.V8Internals.o
[2/7] CXX
obj/third_party/WebKit/Source/core/gen/blink/bindings/core/v8/webcore_testing.V8InternalSettings.o
[3/7] AR obj/third_party/WebKit/Source/core/libwebcore_testing.a
[4/7] LINK webkit_unit_tests
[5/7] LINK content_shell
[6/7] STAMP obj/third_party/WebKit/public/all_blink.actions_depends.stamp
[7/7] STAMP obj/build/blink_tests.actions_depends.stamp
Failing build because ninja reported work to do.
This means that after completing a compile, another was run and
it resulted in still having work to do (that is, a no-op build
wasn't a no-op). Consult the first "ninja explain:" line for a
likely culprit.
.

[bokan, 2016-01-04 18:32:05]

Attempting to reland now that crbug.com/531579 is purportedly fixed.

[bokan, 2016-01-04 18:32:14]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}
==========

to

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}
==========

[bokan, 2016-01-04 18:32:22]

The CQ bit was checked by bokan@chromium.org

[bokan, 2016-01-04 18:32:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
rjkroege@chromium.org, avi@chromium.org, rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1435233002/#ps100001
(title: "Build fix after rebase")

[commit-bot: I haz the power, 2016-01-04 18:34:48]

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

[commit-bot: I haz the power, 2016-01-04 21:35:40]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}
==========

to

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}
==========

[commit-bot: I haz the power, 2016-01-04 21:35:40]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-01-04 21:36:24]

Description was changed from

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}
==========

to

==========
Remove invert viewport scroll order setting from Blink

Note: This setting was already removed on the Chromium side
in r348988 and set the Blink setting to always be true. This
patch just removes the now unused Blink branches.

BUG=443724

Committed: https://crrev.com/82a9c12ebb952e773ee0db1f07aca6fb4d2b6dc9
Cr-Commit-Position: refs/heads/master@{#363625}

Committed: https://crrev.com/993ed1a7d9f986c1c87f0368003b2baf0589ffb0
Cr-Commit-Position: refs/heads/master@{#367373}
==========

[commit-bot: I haz the power, 2016-01-04 21:36:24]

Patchset 6 (id:??) landed as
https://crrev.com/993ed1a7d9f986c1c87f0368003b2baf0589ffb0
Cr-Commit-Position: refs/heads/master@{#367373}