diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-29 05:22:09 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-29 05:22:09 +0000 |
| commit | 067f519a2e675f84736aa317192bb0dc5418c123 (patch) | |
| tree | 08a4a8bea4f9421c5612e2af0bb84bf0ab63ddb2 /content/renderer/render_process_impl.cc | |
| parent | 7d73f676c7a12a52236136c8681dfeb95dd550b6 (diff) | |
| download | chromium_src-067f519a2e675f84736aa317192bb0dc5418c123.zip chromium_src-067f519a2e675f84736aa317192bb0dc5418c123.tar.gz chromium_src-067f519a2e675f84736aa317192bb0dc5418c123.tar.bz2 | |
Remove SetJavaScriptFlags() wrapper function from webkit.
This function is just an helper/wrapper function around v8::V8::SetFlagsFromString() and used only by content.
John preferred to just remove the wrapper and call the v8 API directly in content.
Also removing it from webkit_glue.h helps with the removal of src/webkit/ directory.
BUG=265753
TEST=content_shell and content_unittests
R=jochen@chromium.org, jam@chromium.org
TBR=darin
Review URL: https://codereview.chromium.org/147973002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247626 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/renderer/render_process_impl.cc')
| -rw-r--r-- | content/renderer/render_process_impl.cc | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/content/renderer/render_process_impl.cc b/content/renderer/render_process_impl.cc index 50a4b6e..1ad3d5e 100644 --- a/content/renderer/render_process_impl.cc +++ b/content/renderer/render_process_impl.cc @@ -30,7 +30,7 @@ #include "skia/ext/platform_canvas.h" #include "third_party/WebKit/public/web/WebFrame.h" #include "ui/surface/transport_dib.h" -#include "webkit/glue/webkit_glue.h" +#include "v8/include/v8.h" #if defined(OS_MACOSX) #include "base/mac/mac_util.h" @@ -69,17 +69,23 @@ RenderProcessImpl::RenderProcessImpl() #endif // Out of process dev tools rely upon auto break behavior. - webkit_glue::SetJavaScriptFlags("--debugger-auto-break"); + std::string auto_break_flag("--debugger-auto-break"); + v8::V8::SetFlagsFromString(auto_break_flag.c_str(), + static_cast<int>(auto_break_flag.size())); #if defined(OS_ANDROID) - if (base::android::SysUtils::IsLowEndDevice()) - webkit_glue::SetJavaScriptFlags("--optimize-for-size"); + if (base::android::SysUtils::IsLowEndDevice()) { + std::string optimize_flag("--optimize-for-size"); + v8::V8::SetFlagsFromString(optimize_flag.c_str(), + static_cast<int>(optimize_flag.size())); + } #endif const CommandLine& command_line = *CommandLine::ForCurrentProcess(); if (command_line.HasSwitch(switches::kJavaScriptFlags)) { - webkit_glue::SetJavaScriptFlags( + std::string flags( command_line.GetSwitchValueASCII(switches::kJavaScriptFlags)); + v8::V8::SetFlagsFromString(flags.c_str(), static_cast<int>(flags.size())); } // Turn on cross-site document blocking for renderer processes. |