Get rid of a redundant argument DOWNSTREAM from VisiblePosition constructor call sites

Get rid of a redundant argument DOWNSTREAM from VisiblePosition constructor call sites

This patch gets rid of a redundant argument DOWNSTREAM from |VisiblePosition|
constructor call sites to simplify source code for improving code health,
since default value of affinity is |DOWNSTREAM| in each constructor
declarations.

This patch is a preparation of introducing |TextAffinity| enum class as
replacement of |EAffinity|, http://crbug.com/522038.

BUG=522038
TEST=n/a; no behavior changes

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200796

[yosin_UTC9, 2015-08-19 02:00:50]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org

[yosin_UTC9, 2015-08-19 02:00:58]

PTAL

[tkent, 2015-08-19 02:09:12]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-08-19 02:09:13]

lgtm

[commit-bot: I haz the power, 2015-08-19 02:09:28]

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

[commit-bot: I haz the power, 2015-08-19 02:52:53]

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

[commit-bot: I haz the power, 2015-08-19 02:52:54]

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

[yosin_UTC9, 2015-08-19 03:18:40]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-08-19 03:18:54]

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

[commit-bot: I haz the power, 2015-08-19 03:30:51]

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

[commit-bot: I haz the power, 2015-08-19 03:30:52]

Failed to apply patch for Source/core/editing/commands/CompositeEditCommand.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/editing/commands/CompositeEditCommand.cpp
  Hunk #1 FAILED at 1142.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/editing/commands/CompositeEditCommand.cpp.rej

Patch:       Source/core/editing/commands/CompositeEditCommand.cpp
Index: Source/core/editing/commands/CompositeEditCommand.cpp
diff --git a/Source/core/editing/commands/CompositeEditCommand.cpp
b/Source/core/editing/commands/CompositeEditCommand.cpp
index
72de46aacb88b1b1d262ca0f97a7ffbd0770f80a..ed72b6e60c20c5b44e41126234053a3a7dae8782
100644
--- a/Source/core/editing/commands/CompositeEditCommand.cpp
+++ b/Source/core/editing/commands/CompositeEditCommand.cpp
@@ -1142,7 +1142,7 @@ void CompositeEditCommand::moveParagraphWithClones(const
VisiblePosition& startO
 
     cloneParagraphUnderNewElement(start, end, outerNode, blockElement);
 
-    setEndingSelection(VisibleSelection(start, end, DOWNSTREAM));
+    setEndingSelection(VisibleSelection(start, end));
     deleteSelection(false, false, false);
 
     // There are bugs in deletion when it removes a fully selected table/list.

[yosin_UTC9, 2015-08-19 03:38:13]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2015-08-19 03:38:14]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1285233004/#ps20001
(title: "2015-08-19T12:36:19 Rebase")

[commit-bot: I haz the power, 2015-08-19 03:38:29]

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

[commit-bot: I haz the power, 2015-08-19 04:38:51]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200796