Drop editing's createFullMarkup() function

Drop editing's createFullMarkup() function

Drop editing's createFullMarkup() function and inline it into the caller
instead. There is only one caller and that caller passes a Document to the
function so the function ends up doing a simple call to
createMarkup(node).

R=adamk@chromium.org, leviw@chromium.org

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

[adamk, 2014-07-30 17:16:18]

https://codereview.chromium.org/430823002/diff/1/Source/web/WebLocalFrameImpl...
File Source/web/WebLocalFrameImpl.cpp (right):

https://codereview.chromium.org/430823002/diff/1/Source/web/WebLocalFrameImpl...
Source/web/WebLocalFrameImpl.cpp:1466: return createMarkup(frame()->document(),
IncludeNode, 0);
Why do you need the third argument? It looks like it defaults to null.

[Inactive, 2014-07-30 17:22:30]

https://codereview.chromium.org/430823002/diff/1/Source/web/WebLocalFrameImpl...
File Source/web/WebLocalFrameImpl.cpp (right):

https://codereview.chromium.org/430823002/diff/1/Source/web/WebLocalFrameImpl...
Source/web/WebLocalFrameImpl.cpp:1466: return createMarkup(frame()->document(),
IncludeNode, 0);
On 2014/07/30 17:16:18, adamk wrote:
> Why do you need the third argument? It looks like it defaults to null.

You're right. It seems I don't need IncludeNode either. I merely copied the line
from createFullMarkup() and did not check if the arguments were needed. I'll fix
this.

[Inactive, 2014-07-30 17:25:10]

https://codereview.chromium.org/430823002/diff/1/Source/web/WebLocalFrameImpl...
File Source/web/WebLocalFrameImpl.cpp (right):

https://codereview.chromium.org/430823002/diff/1/Source/web/WebLocalFrameImpl...
Source/web/WebLocalFrameImpl.cpp:1466: return createMarkup(frame()->document(),
IncludeNode, 0);
On 2014/07/30 17:16:18, adamk wrote:
> Why do you need the third argument? It looks like it defaults to null.

Done.

[adamk, 2014-07-30 17:30:24]

lgtm

[Inactive, 2014-07-30 17:31:15]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-07-30 17:31:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/430823002/20001

[commit-bot: I haz the power, 2014-07-30 18:33:02]

Change committed as 179244