aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar jcgregorio <jcgregorio@google.com>2014-11-13 08:06:40 -0800
committerGravatar Commit bot <commit-bot@chromium.org>2014-11-13 08:06:40 -0800
commit3b27adef0a52f6d321fdee7412ef69e7a7284bcc (patch)
tree44edab4ef6bc06603d150d35575cbef69833c82e
parent1c6e7571d40e31247609fd7469f8422018304310 (diff)
Revert of Make nanobench and dm be usable from Chromium build (patchset #5 id:80001 of https://codereview.chromium.org/657373002/)
Reason for revert: Causing breakages on Mac build. Original issue's description: > Make nanobench and dm be usable from Chromium build > > Move the app logic for each app as follows: > > <app>.cpp -- the file which contains main(). Embedders that compile > their own apps, such as ios shell, upcoming Chromium dm etc, do not use this. > > <app>_main.cpp -- the main logic of the Skia test application. This will be > used by Skia -compiled apps as well as embedder -compiled apps. > > <app>_main.h -- the API for the main logic. This will be > used by Skia -compiled apps as well as embedder -compiled apps. > > This way (the upcoming) Chromium dm can setup its Chromium-specific setup > in custom main(), and then call dm_main(), without the need of any > SK_BUILD_FOR_XXXX defines controlling whether the tool defines main or not. > > BUG=skia:2992 > > Committed: https://skia.googlesource.com/skia/+/c092d3bdab5f723576cc0346cea3ee282a9cb444 TBR=mtklein@chromium.org,mtklein@google.com,borenet@google.com,kkinnunen@nvidia.com NOTREECHECKS=true NOTRY=true BUG=skia:2992 Review URL: https://codereview.chromium.org/724073002
-rw-r--r--bench/nanobench.cpp8
-rw-r--r--bench/nanobench.h13
-rw-r--r--bench/nanobench_main.cpp18
-rw-r--r--dm/DM.cpp10
-rw-r--r--dm/dm.h14
-rw-r--r--dm/dm_main.cpp16
-rw-r--r--gyp/bench.gyp1
-rw-r--r--gyp/dm.gyp3
-rw-r--r--gyp/most.gyp2
-rw-r--r--tools/iOSShell.cpp5
10 files changed, 22 insertions, 68 deletions
diff --git a/bench/nanobench.cpp b/bench/nanobench.cpp
index baa109ede7..1f5ae60f82 100644
--- a/bench/nanobench.cpp
+++ b/bench/nanobench.cpp
@@ -572,7 +572,9 @@ private:
int fCurrentSKP;
};
+int nanobench_main();
int nanobench_main() {
+ SetupCrashHandler();
SkAutoGraphics ag;
#if SK_SUPPORT_GPU
@@ -750,3 +752,9 @@ int nanobench_main() {
return 0;
}
+#if !defined SK_BUILD_FOR_IOS
+int main(int argc, char** argv) {
+ SkCommandLineFlags::Parse(argc, argv);
+ return nanobench_main();
+}
+#endif
diff --git a/bench/nanobench.h b/bench/nanobench.h
deleted file mode 100644
index 4616747382..0000000000
--- a/bench/nanobench.h
+++ /dev/null
@@ -1,13 +0,0 @@
-/*
- * Copyright 2014 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-#ifndef nanobench_DEFINED
-#define nanobench_DEFINED
-
-// API for nanobench app.
-int nanobench_main();
-
-#endif
diff --git a/bench/nanobench_main.cpp b/bench/nanobench_main.cpp
deleted file mode 100644
index a598adc264..0000000000
--- a/bench/nanobench_main.cpp
+++ /dev/null
@@ -1,18 +0,0 @@
-/*
- * Copyright 2014 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-
-#include "CrashHandler.h"
-#include "SkCommandLineFlags.h"
-#include "nanobench.h"
-
-int main(int argc, char * const argv[]) {
- SetupCrashHandler();
- SkCommandLineFlags::Parse(argc, const_cast<char**>(argv));
- return nanobench_main();
-
-}
diff --git a/dm/DM.cpp b/dm/DM.cpp
index 08a458756f..52c86d5f68 100644
--- a/dm/DM.cpp
+++ b/dm/DM.cpp
@@ -1,6 +1,7 @@
// Main binary for DM.
// For a high-level overview, please see dm/README.
+#include "CrashHandler.h"
#include "LazyDecodeBitmap.h"
#include "SkCommonFlags.h"
#include "SkForceLinking.h"
@@ -186,7 +187,9 @@ static void append_matching_factories(Registry* head, SkTDArray<typename Registr
}
}
+int dm_main();
int dm_main() {
+ SetupCrashHandler();
SkAutoGraphics ag;
SkTaskGroup::Enabler enabled(FLAGS_threads);
@@ -241,3 +244,10 @@ int dm_main() {
report_failures(failures);
return failures.count() > 0;
}
+
+#if !defined(SK_BUILD_FOR_IOS) && !defined(SK_BUILD_FOR_NACL)
+int main(int argc, char** argv) {
+ SkCommandLineFlags::Parse(argc, argv);
+ return dm_main();
+}
+#endif
diff --git a/dm/dm.h b/dm/dm.h
deleted file mode 100644
index 99b6e10a66..0000000000
--- a/dm/dm.h
+++ /dev/null
@@ -1,14 +0,0 @@
-/*
- * Copyright 2014 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-#ifndef dm_DEFINED
-#define dm_DEFINED
-
-// API for dm app.
-
-int dm_main();
-
-#endif
diff --git a/dm/dm_main.cpp b/dm/dm_main.cpp
deleted file mode 100644
index ac8e0f3ce0..0000000000
--- a/dm/dm_main.cpp
+++ /dev/null
@@ -1,16 +0,0 @@
-/*
- * Copyright 2014 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-#include "CrashHandler.h"
-#include "SkCommandLineFlags.h"
-#include "dm.h"
-
-int main(int argc, char * const argv[]) {
- SetupCrashHandler();
- SkCommandLineFlags::Parse(argc, const_cast<char**>(argv));
- return dm_main();
-}
diff --git a/gyp/bench.gyp b/gyp/bench.gyp
index e91c4f29f5..26849d511a 100644
--- a/gyp/bench.gyp
+++ b/gyp/bench.gyp
@@ -13,7 +13,6 @@
'../bench/GMBench.cpp',
'../bench/RecordingBench.cpp',
'../bench/SKPBench.cpp',
- '../bench/nanobench_main.cpp',
'../bench/nanobench.cpp',
],
'includes': [
diff --git a/gyp/dm.gyp b/gyp/dm.gyp
index 0e5eebd416..e4d6bd6708 100644
--- a/gyp/dm.gyp
+++ b/gyp/dm.gyp
@@ -8,9 +8,6 @@
'includes': [
'dm.gypi',
],
- 'sources': [
- '../dm/dm_main.cpp',
- ],
'conditions': [
['skia_android_framework', {
'libraries': [ '-lskia' ],
diff --git a/gyp/most.gyp b/gyp/most.gyp
index 61efe1e455..e22629ae74 100644
--- a/gyp/most.gyp
+++ b/gyp/most.gyp
@@ -29,7 +29,7 @@
'dependencies': [ 'android_system.gyp:SampleApp_APK' ],
}],
['skia_os == "ios"', {
- 'dependencies!': [ 'SampleApp.gyp:SampleApp', 'dm.gyp:dm', 'bench.gyp:*' ],
+ 'dependencies!': [ 'SampleApp.gyp:SampleApp' ],
'dependencies': ['iOSShell.gyp:iOSShell' ],
}],
['skia_os == "mac" or skia_os == "linux"', {
diff --git a/tools/iOSShell.cpp b/tools/iOSShell.cpp
index 99d5ebd29a..8656e645aa 100644
--- a/tools/iOSShell.cpp
+++ b/tools/iOSShell.cpp
@@ -13,8 +13,6 @@
#include "SkCommonFlags.h"
#include "SkGraphics.h"
#include "SkWindow.h"
-#include "dm.h"
-#include "nanobench.h"
#include "sk_tool_utils.h"
//////////////////////////////////////////////////////////////////////////////
@@ -58,6 +56,9 @@ void ShellWindow::onSizeChange() {
DEFINE_bool(dm, false, "run dm");
DEFINE_bool(nanobench, false, "run nanobench");
+int nanobench_main();
+int dm_main();
+
IOS_launch_type set_cmd_line_args(int argc, char *argv[], const char* resourceDir) {
SkCommandLineFlags::Parse(argc, argv);
SetResourcePath(resourceDir);