Commit d00d41de authored by Hiroshige Hayashizaki's avatar Hiroshige Hayashizaki Committed by Commit Bot

Remove ScriptSourceCode(ScriptResource*) constructor

No longer used after
https://chromium-review.googlesource.com/801665.

To make upcoming refactoring easy.

Bug: 788828
Change-Id: Ie5ac6409c3d1c7ee113c90cff834a5ae6be53537
Reviewed-on: https://chromium-review.googlesource.com/786890
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: default avatarJeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521020}
parent ddc1a75c
......@@ -23,9 +23,6 @@ ScriptSourceCode::ScriptSourceCode(
url_.RemoveFragmentIdentifier();
}
ScriptSourceCode::ScriptSourceCode(ScriptResource* resource)
: ScriptSourceCode(nullptr, resource) {}
ScriptSourceCode::ScriptSourceCode(ScriptStreamer* streamer,
ScriptResource* resource)
: source_(resource->SourceText()),
......
......@@ -46,14 +46,17 @@ class CORE_EXPORT ScriptSourceCode final {
DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();
public:
// We lose the encoding information from ScriptResource.
// Not sure if that matters.
explicit ScriptSourceCode(ScriptResource*);
// For inline scripts.
ScriptSourceCode(
const String& source,
ScriptSourceLocationType = ScriptSourceLocationType::kUnknown,
const KURL& = KURL(),
const TextPosition& start_position = TextPosition::MinimumPosition());
// For external scripts.
//
// We lose the encoding information from ScriptResource.
// Not sure if that matters.
ScriptSourceCode(ScriptStreamer*, ScriptResource*);
~ScriptSourceCode();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment