Set whether context menu appears on mouse up from WebPreferences

Set whether context menu appears on mouse up from WebPreferences

Whether the context menu appears on mouse up can depend on run-time
settings (in the case of Opera). This reads the setting to
WebPreferences, so that the browser has the possibility of
communicating the appropriate setting to the renderer. If not
explicitly set, it defaults, as before, to 'true' on Windows and
'false' on other platforms.

BUG=

Committed: https://crrev.com/9880491900bb5568e0ec7d152f914ce20f875462
Cr-Commit-Position: refs/heads/master@{#319040}

[Arjan van Leeuwen, 2014-12-23 13:35:00]

arjanl@opera.com changed reviewers:
+ ben@chromium.org

[Arjan van Leeuwen, 2014-12-23 13:35:00]

This small patch shouldn't affect Chrome, but makes some features in Opera
easier to implement (mouse gestures).

[Arjan van Leeuwen, 2015-02-23 07:34:23]

arjanl@opera.com changed reviewers:
+ avi@chromium.org, rbyers@chromium.org

[Arjan van Leeuwen, 2015-02-23 07:34:44]

Adding some more reviewers.

[Avi (use Gerrit), 2015-02-23 16:38:17]

lgtm

If it helps you guys, sure.

[Arjan van Leeuwen, 2015-02-23 16:40:28]

The CQ bit was checked by arjanl@opera.com

[commit-bot: I haz the power, 2015-02-23 16:41:43]

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

[commit-bot: I haz the power, 2015-02-23 16:44:45]

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

[commit-bot: I haz the power, 2015-02-23 16:44:46]

Try jobs failed on following builders:
  ios_dbg_simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[Arjan van Leeuwen, 2015-02-24 08:00:05]

The CQ bit was checked by arjanl@opera.com

[Arjan van Leeuwen, 2015-02-24 08:00:06]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org
Link to the patchset: https://codereview.chromium.org/819293002/#ps20001 (title:
"Fix merge issue in earlier patch")

[commit-bot: I haz the power, 2015-02-24 08:00:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/819293002/20001

[commit-bot: I haz the power, 2015-02-24 08:19:31]

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

[commit-bot: I haz the power, 2015-02-24 08:19:34]

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

[Arjan van Leeuwen, 2015-02-24 08:27:54]

arjanl@opera.com changed reviewers:
+ nasko@chromium.org, tsepez@chromium.org

[Arjan van Leeuwen, 2015-02-24 08:28:56]

Looks like I also need an LGTM from +tsepez or +nasko for the change in
common_param_traits_macros.h.

[nasko, 2015-02-24 16:26:03]

LGTM, assuming the change in common_param_traits_macros.h is made.

https://codereview.chromium.org/819293002/diff/20001/content/public/common/co...
File content/public/common/common_param_traits_macros.h (right):

https://codereview.chromium.org/819293002/diff/20001/content/public/common/co...
content/public/common/common_param_traits_macros.h:113:
IPC_STRUCT_TRAITS_MEMBER(context_menu_on_mouse_up)
Move this to be in the same order as web_preferences.h, after default_encoding.

[Tom Sepez, 2015-02-25 17:46:57]

LGTM, deferring to nasko.

[Arjan van Leeuwen, 2015-03-02 10:09:54]

Now with order changed as requested.

[Arjan van Leeuwen, 2015-03-04 08:22:07]

The CQ bit was checked by arjanl@opera.com

[Arjan van Leeuwen, 2015-03-04 08:22:08]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
avi@chromium.org, nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/819293002/#ps40001 (title:
"Fix order to be like WebPreferences")

[commit-bot: I haz the power, 2015-03-04 08:22:25]

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

[commit-bot: I haz the power, 2015-03-04 09:23:13]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-04 09:23:45]

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