[compiler-dispatcher] make it so that we can always parse on bg threads

src/compiler-dispatcher/compiler-dispatcher.cc

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/compiler-dispatcher/compiler-dispatcher.h"
 
 #include "include/v8-platform.h"
 #include "include/v8.h"
 #include "src/base/platform/time.h"
 #include "src/cancelable-task.h"
@@ skipping 50 matching lines @@
   }
   return job->status() != CompileJobStatus::kFailed;
 }
 
 bool IsFinished(CompilerDispatcherJob* job) {
   return job->status() == CompileJobStatus::kDone ||
          job->status() == CompileJobStatus::kFailed;
 }
 
 bool CanRunOnAnyThread(CompilerDispatcherJob* job) {
-  return (job->status() == CompileJobStatus::kReadyToParse &&
-          job->can_parse_on_background_thread()) ||
-         (job->status() == CompileJobStatus::kReadyToCompile &&
-          job->can_compile_on_background_thread());
+  return (job->status() == CompileJobStatus::kReadyToParse) ||

[vogelheim, 2017/01/13 15:20:22]

style nitpick: parentheses aren't really needed. See IsFinished above.

[jochen (gone - plz use gerrit), 2017/01/13 16:14:40]

done

+         (job->status() == CompileJobStatus::kReadyToCompile);

[rmcilroy, 2017/01/13 15:56:22]

This now implicitly assumes the compiler dispatcher only compiles Ignition
bytecode. Could we DCHECK this when enabling the dispatcher?

[jochen (gone - plz use gerrit), 2017/01/13 16:14:40]

done

 }
 
 void DoNextStepOnBackgroundThread(CompilerDispatcherJob* job) {
   DCHECK(CanRunOnAnyThread(job));
   switch (job->status()) {
     case CompileJobStatus::kReadyToParse:
       job->Parse();
       break;
 
     case CompileJobStatus::kReadyToCompile:
@@ skipping 537 matching lines @@
       lock.reset();
       DoNextStepOnMainThread(isolate_, job->second.get(),
                              ExceptionHandling::kSwallow);
     }
   }
   if (jobs_.size() > too_long_jobs) ScheduleIdleTaskIfNeeded();
 }
 
 }  // namespace internal
 }  // namespace v8