diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2018-08-23 20:58:14 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-23 20:58:14 +0000 |
| commit | 203339a2aa88bb31576233220d7ced73920a596f (patch) | |
| tree | 895fc5fcc07107c8cc6f81b9c6e0f12faef9fc1e /fxjs/cfxjs_engine_embeddertest.cpp | |
| parent | 3b45012d57884b06915eb5a1f54fbba04a45e807 (diff) | |
| download | pdfium-203339a2aa88bb31576233220d7ced73920a596f.tar.xz | |
Fix shadowed variables
This CL fixes instances of variable shadowing that are
discovered by turning on -Wshadow.
BUG=pdfium:1137
Change-Id: I418d50de89ecbeb12e85b23a358bc61e8f16e888
Reviewed-on: https://pdfium-review.googlesource.com/41150
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'fxjs/cfxjs_engine_embeddertest.cpp')
| -rw-r--r-- | fxjs/cfxjs_engine_embeddertest.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/fxjs/cfxjs_engine_embeddertest.cpp b/fxjs/cfxjs_engine_embeddertest.cpp index ff866d3ee0..a9422ece98 100644 --- a/fxjs/cfxjs_engine_embeddertest.cpp +++ b/fxjs/cfxjs_engine_embeddertest.cpp @@ -51,10 +51,12 @@ TEST_F(CFXJSEngineEmbedderTest, MultipleEngines) { engine2.InitializeEngine(); v8::Context::Scope context_scope(GetV8Context()); - Optional<IJS_Runtime::JS_Error> err = engine()->Execute(WideString(kScript0)); - EXPECT_FALSE(err); - CheckAssignmentInEngineContext(engine(), kExpected0); - + { + Optional<IJS_Runtime::JS_Error> err = + engine()->Execute(WideString(kScript0)); + EXPECT_FALSE(err); + CheckAssignmentInEngineContext(engine(), kExpected0); + } { // engine1 executing in engine1's context doesn't affect main. v8::Context::Scope context_scope1(engine1.GetV8Context()); |